Skip to content

Commit e02f1b3

Browse files
committed
fix: Add external_id to mock repos in SCM onboarding acceptance tests
The new externalId field in OrganizationIntegrationReposEndpoint requires external_id in the repository data. The acceptance test mocks were missing this field, causing KeyError → 500 errors.
1 parent c33ccd6 commit e02f1b3

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

tests/acceptance/test_scm_onboarding.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ def test_scm_onboarding_happy_path(self) -> None:
5252
"name": "sentry",
5353
"identifier": "getsentry/sentry",
5454
"default_branch": "master",
55+
"external_id": "12345",
5556
},
5657
]
5758

@@ -162,6 +163,7 @@ def test_scm_onboarding_with_integration_install(self) -> None:
162163
"name": "sentry",
163164
"identifier": "getsentry/sentry",
164165
"default_branch": "master",
166+
"external_id": "12345",
165167
},
166168
]
167169

@@ -273,6 +275,7 @@ def test_scm_onboarding_detection_error_falls_back_to_manual_picker(self) -> Non
273275
"name": "sentry",
274276
"identifier": "getsentry/sentry",
275277
"default_branch": "master",
278+
"external_id": "12345",
276279
},
277280
]
278281

0 commit comments

Comments
 (0)