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.

Sign up for GitHub

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

Jump to bottom

Add jest for javascript testing #2138

Open
wants to merge 14 commits into
base: legacy
Choose a base branch
from
Open

Add jest for javascript testing #2138

wants to merge 14 commits into from

Conversation

jaltekruse
Copy link
Contributor

@jaltekruse jaltekruse commented May 16, 2023 •
edited
Loading

  • Jest is a framework for writing javascript unit tests
  • This adds jest as well as the react-testing-library, which allows rendering react components, interacting with and inspecting parts of the render tree. These tests can run faster than cypress tests as they don't communicate with the PHP server, and they also don't require a full browser, they render into a lightweight virtual DOM.
  • To run tests, inside the devcontainer run: npx jest
  • To debug an individual test, this PR also adds a VSCode Jest extension that allows you to run and debug individual tests
  • This includes a sample test that can mock data coming from the server and it renders a component that is embedded as a child route and expects some outletContext data as input

jaltekruse added 10 commits May 15, 2023 11:21
@jaltekruse
WIP - cypress level test
4f05577
@jaltekruse
getting jest set up
dd0fe03
@jaltekruse
WIP - testing stuff
e8aeacf
@jaltekruse
WIP - trying to get a test running with react-router and recoil stuff…
fcb31e7 
… together

- current error - TypeError: Cannot read properties of undefined (reading 'subscribe')
@jaltekruse
First working test with react-router
8b06564 
- I forgot to stage the right file last commit, so I didn't save my in-progress state, but I did end up copying it to start with the simpler version and fix it
@jaltekruse
Factoring out a little bit of code
ae682c5
@jaltekruse
Make the utility function do a little more, delete the bad version of…
971d8fa 
… the test
@jaltekruse
Small cleanups, delete incomplete test for some recoil code
00e685f
@jaltekruse
Rename test, add vscode config for debugging jest tests, remove incom…
e5ca25a 
…plete cypress test
@jaltekruse
revert changes to courseActions
c1b2cdd 
- was a minor fix to get the code to run from a unit test, will
  worry about this later when I finish that test and fix the bug
@jaltekruse jaltekruse force-pushed the add-jest-for-javascript-testing branch from 80afbb1 to 40c0125 Compare May 16, 2023 22:41
@jaltekruse jaltekruse mentioned this pull request May 17, 2023
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lyanthropos Lyanthropos Awaiting requested review from Lyanthropos

@dqnykamp dqnykamp Awaiting requested review from dqnykamp

@kevincharles kevincharles Awaiting requested review from kevincharles

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jaltekruse