Enable Allure for integration tests (#66)

* Enable Allure for integration tests

* Remove %s from log message

* Fix relation in helpers.py for matrix-auth

Updated the relation in the integration helper to use the correct interface.

* Format add_relation call for better readability

* Fix typo in relation addition for matrix-auth

Author: swetha1654

.github/workflows/integration_test.yaml

@@ -11,3 +11,8 @@ jobs:
       juju-channel: 3.4/stable
       provider: lxd
       modules: '["test_charm.py", "test_ingress.py"]'
+  allure-report:
+    if: always() && !cancelled()
+    needs:
+      - integration-tests
+    uses: canonical/operator-workflows/.github/workflows/allure_report.yaml@main

src/irc.py

@@ -254,7 +254,7 @@ def _generate_app_registration_local(self) -> None:
     def _generate_media_proxy_key(self) -> None:
         """Generate the content of the media proxy key."""
         if IRC_BRIDGE_SIGNING_KEY_FILE_PATH.exists():
-            logger.info("Media proxy key file %s already exists, skipping")
+            logger.info("Media proxy key file already exists, skipping")
             return
         media_proxy_key_command = [
             "/bin/bash",

tests/integration/helpers.py

@@ -184,7 +184,9 @@ async def generate_anycharm_relation(
         to=machine,
     )
     await ops_test.model.wait_for_idle(apps=[any_charm.name])
-    await ops_test.model.add_relation(f"{any_charm.name}", f"{app.name}")
+    await ops_test.model.add_relation(
+        f"{any_charm.name}:provide-matrix-auth", f"{app.name}:matrix-auth"
+    )
 
 
 async def get_unit_address(application: Application) -> str:

tox.ini

@@ -102,11 +102,13 @@ commands =
 [testenv:integration]
 description = Run integration tests
 deps =
+    -r{toxinidir}/requirements.txt
+    allure-pytest>=2.8.18
+    git+https://github.com/canonical/data-platform-workflows@v24.0.0\#subdirectory=python/pytest_plugins/allure_pytest_collection_report
     juju
     pytest
     pytest-asyncio
     pytest-operator
-    -r{toxinidir}/requirements.txt
 commands =
     pytest -v --tb native --ignore={[vars]tst_path}unit --log-cli-level=INFO -s {posargs}