Skip to content

[Onboarding Flow] Finishing up the test suite for onboarding#35

Merged
MartinBraquet merged 6 commits into
CompassConnections:mainfrom
O-Bots:main
Mar 4, 2026
Merged

[Onboarding Flow] Finishing up the test suite for onboarding#35
MartinBraquet merged 6 commits into
CompassConnections:mainfrom
O-Bots:main

Conversation

@O-Bots

@O-Bots O-Bots commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Added data-test attributes where necessary to create stable locators
  • Added compatibility question tests and verification's
  • Updated tests to cover Keywords and Headline changes recently made
  • Updated tests to cover all of the big5 personality traits

O-Bots added 5 commits March 2, 2026 22:55
Changed type ChildrenExpectation so that it can be used for database verification
Added more compatibility questions
Added compatibility question tests and verifications
Updated tests to cover Keywords and Headline changes recently made
Updated tests to cover all of the big5 personality traits
@vercel

vercel Bot commented Mar 4, 2026

Copy link
Copy Markdown

@O-Bots is attempting to deploy a commit to the Compass Team on Vercel.

A member of the Team first needs to authorize it.

@O-Bots
O-Bots requested a review from MartinBraquet March 4, 2026 01:38
@codecov

codecov Bot commented Mar 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread tests/e2e/web/pages/profilePage.ts Outdated
Comment thread tests/e2e/web/pages/profilePage.ts Outdated
Comment thread tests/e2e/web/pages/profilePage.ts Outdated
Comment thread tests/e2e/web/pages/profilePage.ts Outdated

@MartinBraquet MartinBraquet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks awesome, compat prompts during onboarding now also covered!
Just one suggestion for next time: to abide by the repo formatting rules, could you just run

yarn lint-fix
yarn prettier

before making a PR?
Normally they should run as a pre commit, but seems like they didn't (since I see some trailing semi-colons).

@MartinBraquet
MartinBraquet merged commit e115df8 into CompassConnections:main Mar 4, 2026
4 of 5 checks passed
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