Add --use-existing feature to integ tests (#126)

Co-authored-by: arturo-seijas <[email protected]>

src/charm.py

@@ -492,6 +492,7 @@ def _set_workload_version(self) -> None:
             return
         env_settings = self._create_discourse_environment_settings()
         try:
+            logger.info("Setting workload version")
             get_version_process = container.exec(
                 [f"{DISCOURSE_PATH}/bin/rails", "runner", "puts Discourse::VERSION::STRING"],
                 environment=env_settings,
@@ -511,6 +512,7 @@ def _run_s3_migration(self) -> None:
         env_settings = self._create_discourse_environment_settings()
         try:
             self.model.unit.status = MaintenanceStatus("Running S3 migration")
+            logger.info("Running S3 migration")
             process = container.exec(
                 [f"{DISCOURSE_PATH}/bin/bundle", "exec", "rake", "s3:upload_assets"],
                 environment=env_settings,

tests/conftest.py

@@ -10,3 +10,9 @@ def pytest_addoption(parser):
     parser.addoption("--saml-email", action="store")
     parser.addoption("--saml-password", action="store")
     parser.addoption("--charm-file", action="store", default=None)
+    parser.addoption(
+        "--use-existing",
+        action="store_true",
+        default=False,
+        help="This will skip deployment of the charms. Useful for local testing.",
+    )

tests/integration/conftest.py

@@ -132,6 +132,11 @@ async def app_fixture(
     """Discourse charm used for integration testing.
     Builds the charm and deploys it and the relations it depends on.
     """
+    use_existing = pytestconfig.getoption("--use-existing", default=False)
+    if use_existing:
+        yield model.applications[app_name]
+        return
+
     postgres_app = await model.deploy(
         "postgresql-k8s",
         channel="14/edge",