Skip to content

Update deployWorkspaceProject tests to reflect the new deployment modes#973

Merged
MicroFish91 merged 72 commits intomainfrom
mwf/advanced-tests-09012025
Sep 16, 2025
Merged

Update deployWorkspaceProject tests to reflect the new deployment modes#973
MicroFish91 merged 72 commits intomainfrom
mwf/advanced-tests-09012025

Conversation

@MicroFish91
Copy link
Copy Markdown
Contributor

@MicroFish91 MicroFish91 commented Sep 15, 2025

Renamed & Updated:

  • albumapi-js => basic-js
  • monorepo-basic-1 => monorepo-admincreds
  • monorepo-basic-2 => monorepo-identity

New:

  • advanced-js

@MicroFish91 MicroFish91 requested a review from a team as a code owner September 15, 2025 21:10
}
});

async function setupResources(): Promise<void> {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Defaulting behavior is a bit more predictable now, so we can remove this.

import { generateAdvancedJSTests } from "./polyrepo/advancedJSScenario";
import { generateBasicJSTests } from "./polyrepo/basicJSScenario";

export function generateTestScenarios(): DeployWorkspaceProjectTestScenario[] {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed this file, basically the same logic though (although referencing updated project names)

nturinski
nturinski previously approved these changes Sep 15, 2025
Base automatically changed from mwf/walkthrough-082925 to main September 16, 2025 01:35
@MicroFish91 MicroFish91 dismissed nturinski’s stale review September 16, 2025 01:35

The base branch was changed.

@MicroFish91 MicroFish91 merged commit 4ab17d5 into main Sep 16, 2025
2 checks passed
@MicroFish91 MicroFish91 deleted the mwf/advanced-tests-09012025 branch September 16, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants