test: tidy up functional tests

TimKnight-DWP · TimKnight-DWP · commit 8ebfe1a9fa9a · 2024-04-24T10:27:13.000+01:00
Signed-off-by: Tim Knight &lt;tim.knight1@engineering.digital.dwp.gov.uk&gt;
diff --git a/tests/functional/api/test_deploy_tokens.py b/tests/functional/api/test_deploy_tokens.py
@@ -16,7 +16,7 @@ def test_project_deploy_tokens(gl, project):
 
     deploy_token = project.deploytokens.get(deploy_token.id)
     assert deploy_token.name == "foo"
-    assert deploy_token.expires_at == today + "T00:00:00.000Z"
+    assert deploy_token.expires_at == f"{today}T00:00:00.000Z"
     assert deploy_token.scopes == ["read_registry"]
     assert deploy_token.username == "bar"
 
diff --git a/tests/functional/cli/test_cli_repository.py b/tests/functional/cli/test_cli_repository.py
@@ -77,6 +77,9 @@ def test_list_merge_request_commits(gitlab_cli, merge_request, project):
 
 
 def test_commit_merge_requests(gitlab_cli, project, merge_request):
+    """This tests the `project-commit merge-requests` command and also tests
+    that we can print the result using the `json` formatter"""
+
     # Pause to let GL catch up (happens on hosted too, sometimes takes a while for server to be ready to merge)
     time.sleep(30)
 
diff --git a/tests/functional/cli/test_cli_v4.py b/tests/functional/cli/test_cli_v4.py
@@ -2,7 +2,7 @@
 import os
 import time
 
-branch = "branch-cli-v4"
+branch = "BRANCH-cli-v4"
 
 
 def test_create_project(gitlab_cli):
@@ -243,6 +243,8 @@ def test_accept_request_merge(gitlab_cli, project):
     # Pause to let GL catch up (happens on hosted too, sometimes takes a while for server to be ready to merge)
     time.sleep(30)
 
+    mr = project.mergerequests.get(mr.iid)
+
     if mr.detailed_merge_status == "not_approved":
         approve_cmd = [
             "project-merge-request",
diff --git a/tests/functional/conftest.py b/tests/functional/conftest.py
@@ -389,9 +389,6 @@ def _make_merge_request(*, source_branch: str, create_pipeline: bool = False):
             }
         )
 
-        # Helps with Debugging why MRs fail to merge resulting in 405 from downstream tests
-        project.approvalrules.list()
-
         if create_pipeline:
             project.files.create(
                 {

Original file line number	Diff line number	Diff line change
`@@ -389,9 +389,6 @@ def _make_merge_request(*, source_branch: str, create_pipeline: bool = False):`
`389`	`389`	`}`
`390`	`390`	`)`
`391`	`391`
`392`		`- # Helps with Debugging why MRs fail to merge resulting in 405 from downstream tests`
`393`		`- project.approvalrules.list()`
`394`		`-`
`395`	`392`	`if create_pipeline:`
`396`	`393`	`project.files.create(`
`397`	`394`	`{`