Skip to content

Comments

Refactor randomize test users and db#648

Closed
victor-enogwe wants to merge 7 commits intomainfrom
refactor-randomize-test-users-and-db
Closed

Refactor randomize test users and db#648
victor-enogwe wants to merge 7 commits intomainfrom
refactor-randomize-test-users-and-db

Conversation

@victor-enogwe
Copy link
Contributor

What does this PR do?

Fixes the flaky tests.

##Use Case

closes #647

@victor-enogwe victor-enogwe deleted the refactor-randomize-test-users-and-db branch July 22, 2025 18:37
const mongoUri = (global as unknown as { __MONGO_URI__: string }).__MONGO_URI__;

process.env["BASEURL"] = "https://foo.yourdomain.app";
process.env["BASEURL"] = "http;s://foo.yourdomain.app";
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Disregard, realized this PR never got merged


export type Direction_Migrate = "forward" | "back";
export type Direction_Migrate = "forward" | "back" | "up" | "down";

Copy link
Contributor

@tyler-dane tyler-dane Jul 23, 2025

Choose a reason for hiding this comment

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

This is good to fix...but we shouldn't sneak in out-of-scope work into massive PRs. It makes it harder to follow

@victor-enogwe

Copy link
Contributor

Choose a reason for hiding this comment

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

Disregard, realized this PR never got merged

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.

Isolate test runs from side effects

2 participants