Skip to content

Speed up BBS e2e tests by reducing repo migrations #1470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
brianaj merged 12 commits into from
Dec 5, 2025
+0 −6
Merged

Speed up BBS e2e tests by reducing repo migrations #1470

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
eb712b1
split up bbs e2e tests
brianaj Dec 5, 2025
508a775
Fix constructor signatures in new BBS test files
brianaj Dec 5, 2025
0b7f4c5
update workflow_dispatch to manually test
brianaj Dec 5, 2025
36d99c5
Revert "update workflow_dispatch to manually test"
brianaj Dec 5, 2025
3d61641
only migrate one repo
brianaj Dec 5, 2025
123e629
add back changes workflow_dispatch for manual testing
brianaj Dec 5, 2025
d579b1c
add remaining changes
brianaj Dec 5, 2025
ebdf768
revert to one test
brianaj Dec 5, 2025
528fe64
revert yaml
brianaj Dec 5, 2025
a5a9467
Merge branch 'main' into brianaj/1387-speed-up-bbs-e2e-tests
brianaj Dec 5, 2025
9c9f8f8
actually revert yaml changes completely
brianaj Dec 5, 2025
9b466e8
Merge branch 'brianaj/1387-speed-up-bbs-e2e-tests' of https://github.…
brianaj Dec 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions src/OctoshiftCLI.IntegrationTests/BbsToGithub.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,7 @@ public async Task Basic(string bbsServer, bool useSshForArchiveDownload, Archive
var bbsProjectKey = $"E2E-{TestHelper.GetOsName().ToUpper()}";
var githubTargetOrg = $"octoshift-e2e-bbs-{TestHelper.GetOsName()}";
var repo1 = $"{bbsProjectKey}-repo-1";
var repo2 = $"{bbsProjectKey}-repo-2";
var targetRepo1 = $"{bbsProjectKey}-e2e-{TestHelper.GetOsName().ToLower()}-repo-1";
var targetRepo2 = $"{bbsProjectKey}-e2e-{TestHelper.GetOsName().ToLower()}-repo-2";

var sourceBbsApi = new BbsApi(_sourceBbsClient, bbsServer, _logger);
var sourceHelper = new TestHelper(_output, sourceBbsApi, _sourceBbsClient, bbsServer);
Expand All @@ -95,8 +93,6 @@ await retryPolicy.Retry(async () =>
await sourceHelper.CreateBbsProject(bbsProjectKey);
await sourceHelper.CreateBbsRepo(bbsProjectKey, repo1);
await sourceHelper.InitializeBbsRepo(bbsProjectKey, repo1);
await sourceHelper.CreateBbsRepo(bbsProjectKey, repo2);
await sourceHelper.InitializeBbsRepo(bbsProjectKey, repo2);
});

var sshPort = Environment.GetEnvironmentVariable("SSH_PORT_BBS");
Expand Down Expand Up @@ -135,9 +131,7 @@ await _targetHelper.RunBbsCliMigration(
_targetHelper.AssertNoErrorInLogs(_startTime);

await _targetHelper.AssertGithubRepoExists(githubTargetOrg, targetRepo1);
await _targetHelper.AssertGithubRepoExists(githubTargetOrg, targetRepo2);
await _targetHelper.AssertGithubRepoInitialized(githubTargetOrg, targetRepo1);
await _targetHelper.AssertGithubRepoInitialized(githubTargetOrg, targetRepo2);

// TODO: Assert migration logs are downloaded
}
Expand Down
Loading