Skip to content
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.

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

Isaiah/playwright tests #537

Merged
merged 98 commits into from
Nov 15, 2024
Merged

Isaiah/playwright tests #537

merged 98 commits into from
Nov 15, 2024

Conversation

IsaiahA21
Copy link
Collaborator

I added playwright test coverage for HomePage, Nav bar, and Project page.
Lmk if the tests are adequate.

@IsaiahA21 IsaiahA21 linked an issue Sep 7, 2024 that may be closed by this pull request
Copy link

vercel bot commented Sep 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tsu-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 15, 2024 0:45am

Copy link
Collaborator

@Anidion Anidion left a comment

Choose a reason for hiding this comment

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

Great work! This is a really good start
If you're interested in learning about CI/CD that can be your next task, but up to you :)

playwright.config.ts Show resolved Hide resolved
tests/navigationBar/NavBar.test.ts Outdated Show resolved Hide resolved
tests/navigationBar/NavBar.test.ts Outdated Show resolved Hide resolved
tests/navigationBar/NavBar.test.ts Outdated Show resolved Hide resolved
…tent-updates to include Playwright Recorder

Signed-off-by: IsaiahA21 <[email protected]>
…which will regenerated each time. \n Hence, added data-testid to PhotoGallery.tsx, Profile.tsx, FounderSection.styles.ts to images could be uniquely identified based on an id.

Signed-off-by: IsaiahA21 <[email protected]>
content-updates.md Outdated Show resolved Hide resolved
Sahitiakella and others added 2 commits November 14, 2024 15:04
Signed-off-by: s-akella04 <[email protected]>
Co-authored-by: s-akella04 <[email protected]>
Signed-off-by: IsaiahA21 <[email protected]>
Signed-off-by: Sukritib13 <[email protected]>
Signed-off-by: IsaiahA21 <[email protected]>
IsaiahA21 and others added 20 commits November 14, 2024 15:04
Co-authored-by: Ben Schmidt <[email protected]>
Signed-off-by: IsaiahA21 <[email protected]>
…tent-updates to include Playwright Recorder

Signed-off-by: IsaiahA21 <[email protected]>
…which will regenerated each time. \n Hence, added data-testid to PhotoGallery.tsx, Profile.tsx, FounderSection.styles.ts to images could be uniquely identified based on an id.

Signed-off-by: IsaiahA21 <[email protected]>
Signed-off-by: IsaiahA21 <[email protected]>
* handling empty PM list

* Update src/components/TeamSection/TeamInformation.ts

Co-authored-by: Ben Schmidt <[email protected]>
Signed-off-by: Morteza Faraji <[email protected]>

---------

Signed-off-by: Morteza Faraji <[email protected]>
Co-authored-by: Ben Schmidt <[email protected]>
* 524 - initail changes, added Alumni tab, reusable components

* 524-Adding back Slider to UI, addressing comments. Signed-off-by: Sukriti, Website Developer

* Addressing comments - creating helper functions, fixing styles

Signed-off-by: Sukritib13 <[email protected]>

* addressing comments - using var for bg color, using type for readability

Signed-off-by: Sukritib13 <[email protected]>

* using types and correcting case

Signed-off-by: Sukritib13 <[email protected]>

* using memoization

Signed-off-by: Sukritib13 <[email protected]>

---------

Signed-off-by: Sukritib13 <[email protected]>
Co-authored-by: brian nguyen <[email protected]>
Copy link
Collaborator

@Anidion Anidion left a comment

Choose a reason for hiding this comment

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

Looks good!

@IsaiahA21 IsaiahA21 merged commit c36fab4 into main Nov 15, 2024
4 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.

Add tests
6 participants