GSOC: Add env file merger, e2e test database cleanup, and signup flow test without email verification#4210
Open
clairep94 wants to merge 8 commits into
Open
Conversation
clairep94
commented
Jul 18, 2026
Comment on lines
-20
to
-36
| Project.find({}, {}, { | ||
| timeout: true | ||
| }).cursor().eachAsync((project) => { | ||
| console.log(project.name); | ||
| if (project.name.length < 256) { | ||
| console.log('Project name is okay.'); | ||
| return Promise.resolve(); | ||
| Project.find( | ||
| {}, | ||
| {}, | ||
| { | ||
| timeout: true | ||
| } | ||
| project.name = project.name.substr(0, 255); | ||
| project.slug = slugify(project.name, '_'); | ||
| return project.save().then(() => { | ||
| console.log('Updated sketch slug to: ' + project.slug); | ||
| ) | ||
| .cursor() | ||
| .eachAsync((project) => { | ||
| console.log(project.name); | ||
| if (project.name.length < 256) { | ||
| console.log('Project name is okay.'); | ||
| return Promise.resolve(); | ||
| } | ||
| project.name = project.name.substr(0, 255); | ||
| project.slug = slugify(project.name, '_'); | ||
| return project.save().then(() => { | ||
| console.log('Updated sketch slug to: ' + project.slug); | ||
| }); | ||
| }) | ||
| .then(() => { | ||
| console.log('Done iterating over every sketch.'); | ||
| process.exit(0); | ||
| }); | ||
| }).then(() => { | ||
| console.log('Done iterating over every sketch.'); | ||
| process.exit(0); | ||
| }); No newline at end of file |
Collaborator
Author
There was a problem hiding this comment.
these lines are all due to auto-format on save
…e2e workflow steps to use command; Update docs
clairep94
marked this pull request as ready for review
July 18, 2026 21:42
clairep94
force-pushed
the
clairep-setup_test_db_e2e
branch
from
July 18, 2026 21:44
795b00a to
ca076c2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue:
Fixes #
GSOC E2E test setup -- configures a way to have separate e2e ENV variables, test db and server
Demo:
Screen.Recording.2026-07-18.at.21.30.30.mov
When an e2e app server has not been started from another terminal you see this warning:
Changes:
loadEnv.jsfunction which helps merge.env.e2e&.env(+ enables merging env files in the future).envis the base &.env.e2econtains overrides such as Ports (9xxx for e2e vs 8xxx) and database URL --> prevents developers from accidentally hitting their regular db when running e2e tests locally.env.e2efile that has direct copies from.env.example+ overridden values. If the required env values change in the future, we do not need to change them in multiple places.signup.spec.ts-- email verification will follow up a subsequent PRI have verified that this pull request:
npm run lint)npm run test)npm run typecheck)developbranch.Fixes #123