Treat pytest warnings as errors

Warnings are usually a sign of something that can be improved or even
wrong, like tasks not being properly awaited or cancelled, so it's a
good idea to treat them as errors to avoid them getting unnoticed.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

RELEASE_NOTES.md

@@ -15,15 +15,17 @@
 
 ### Cookiecutter template
 
-All upgrading should be done via the migration script or regenerating the templates.
+All upgrading should be done via the migration script or regenerating the templates. But you might still need to adapt your code:
+
+- `pytest` now uses `-Werror` by default, so if your tests run with warnings, they will now be turned to errors and you'll need to fix them.
 
 ## New Features
 
 <!-- Here goes the main new features and examples or instructions on how to use them -->
 
 ### Cookiecutter template
 
-<!-- Here new features for cookiecutter specifically -->
+- `pytest` now uses `-Werror` by default.
 
 ## Bug Fixes
 

cookiecutter/{{cookiecutter.github_repo_name}}/pyproject.toml

@@ -203,7 +203,7 @@ disable = [
 
 [tool.pytest.ini_options]
 {%- if cookiecutter.type != "api" %}
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["tests", "src"]
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"

pyproject.toml

@@ -205,7 +205,7 @@ module = [
 ignore_missing_imports = true
 
 [tool.pytest.ini_options]
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["src", "tests"]
 markers = [
   "integration: integration tests (deselect with '-m \"not integration\"')",

tests_golden/integration/test_cookiecutter_generation/actor/frequenz-actor-test/pyproject.toml

@@ -153,7 +153,7 @@ disable = [
 ]
 
 [tool.pytest.ini_options]
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["tests", "src"]
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"

tests_golden/integration/test_cookiecutter_generation/app/frequenz-app-test/pyproject.toml

@@ -152,7 +152,7 @@ disable = [
 ]
 
 [tool.pytest.ini_options]
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["tests", "src"]
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"

tests_golden/integration/test_cookiecutter_generation/lib/frequenz-test-python/pyproject.toml

@@ -149,7 +149,7 @@ disable = [
 ]
 
 [tool.pytest.ini_options]
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["tests", "src"]
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"

tests_golden/integration/test_cookiecutter_generation/model/frequenz-model-test/pyproject.toml

@@ -153,7 +153,7 @@ disable = [
 ]
 
 [tool.pytest.ini_options]
-addopts = "-W=all -vv"
+addopts = "-W=all -Werror -vv"
 testpaths = ["tests", "src"]
 asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"