Skip to content

Commit 8d4bbeb

Browse files
committed
test(seer): populate GitLab config['path'] in autofix repo fixtures
get_repo_url_path() requires config['path'] for GitLab repos and raises if it's missing, which is correct for production where path_with_namespace is always populated. The test fixtures were creating GitLab repos without it; set it so the fixtures reflect production reality.
1 parent 4230854 commit 8d4bbeb

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

tests/sentry/autofix/test_utils.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,8 @@ def test_filters_out_unsupported_providers(self) -> None:
5858
external_id="456",
5959
integration_id=345,
6060
)
61+
# GitLab repos always have the URL-safe path_with_namespace in config["path"].
62+
gitlab_repo.update(config={"path": "getsentry/sentry-gitlab"})
6163
self.create_code_mapping(
6264
project=project, repo=gitlab_repo, stack_root="gitlab/", source_root="src/gitlab/"
6365
)
@@ -83,6 +85,8 @@ def test_includes_gitlab_repos_with_feature_flag(self) -> None:
8385
external_id="456",
8486
integration_id=345,
8587
)
88+
# GitLab repos always have the URL-safe path_with_namespace in config["path"].
89+
gitlab_repo.update(config={"path": "getsentry/sentry-gitlab"})
8690
self.create_code_mapping(
8791
project=project, repo=gitlab_repo, stack_root="gitlab/", source_root="src/gitlab/"
8892
)

0 commit comments

Comments
 (0)