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

feat(tests): front end tests for week 9 features #67

Merged
merged 6 commits into from
Oct 31, 2023

Conversation

connordoman
Copy link
Contributor

Front End Tests for Week 9

Resolves #65

Description of the change:

Wrote tests for the following:

  • Header.tsx
  • Footer.tsx
  • config.ts

Also took the time to refactor some test file names to better represent their origin.

Some components are only able to E2E tested currently.

Motivation for the change:

Testing is required for this course.

How to manually test:

To run these tests, navigate to front-end/ and run npm run test.


Also uncovered more need for integration and E2E testing so we'll have to integrate that very soon.

@connordoman connordoman added feat New feature or request area/front-end Front-end work test Unit/Integration tests labels Oct 31, 2023
@connordoman connordoman self-assigned this Oct 31, 2023
Copy link
Contributor

@MyStackOverflows MyStackOverflows left a comment

Choose a reason for hiding this comment

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

looks great, pretty comprehensive

@connordoman connordoman merged commit d4b926c into develop Oct 31, 2023
7 checks passed
@tthvo tthvo deleted the gh-65-frontend-tests branch November 30, 2023 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/front-end Front-end work feat New feature or request test Unit/Integration tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants