Skip to content

Commit

Permalink
chore: Delete unnecessary line, more explanation in test docstring
Browse files Browse the repository at this point in the history
  • Loading branch information
alithethird committed Sep 23, 2024
1 parent ed4e8d7 commit dfe603f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions tests/integration/test_charm.py
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,10 @@ async def test_setup_discourse(
@pytest.mark.abort_on_fail
async def test_db_migration(model: Model, ops_test: OpsTest, pytestconfig: Config, run_action):
"""
arrange: preload postgres with a mock db
act: deploy and integrate with discourse
assert: discourse must be active idle
arrange: preload postgres with a mock db that is created in Discource v3.2.0
act: deploy and integrate with discourse v3.3.0
assert: discourse must be active idle, previously it was creating migration
errors related to not being able to delete some columns because of triggers
"""
postgres_app = await model.deploy(
"postgresql-k8s",
Expand All @@ -109,7 +110,6 @@ async def test_db_migration(model: Model, ops_test: OpsTest, pytestconfig: Confi
return_code, _, _ = await ops_test.juju(
"scp",
"--container",
# postgres_app.name,
"postgresql",
"./mock_db",
f"{postgres_app.units[0].name}:.",
Expand Down

0 comments on commit dfe603f

Please sign in to comment.