-
-
Notifications
You must be signed in to change notification settings - Fork 186
Closed
Labels
🏷 status: label work requiredNeeds proper labelling before it can be worked onNeeds proper labelling before it can be worked on💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🤖 aspect: dxConcerns developers' experience with the codebaseConcerns developers' experience with the codebase
Description
The current unit and e2e tests were written before the current Outreachy internship round, and need to be updated for the current codebase. We need unit tests for components and utils, but some components won't need tests because they mock Vocabulary components, and will be deprecated soon.
Unit tests are done with Jest, and e2e tests are done with nightwatch.
This task is partially blocking #77. The reason for the block was that the tests written at the time the PR was created were old and would always fail. Once we've got some tests updated, we can merge it.
The components that need tests are:
- AttributionDetailsStep (Add unit and e2e tests for AttributionDetailsStep component #112)
- CopyrightWaiverStep (Add unit and e2e tests for CopyrightWaiverStep component #113)
- DropdownStep (Add unit and e2e tests for DropdownStep component #114)
- FirstStep (Add unit and e2e tests for FirstStep component #115)
- HelpSection (Add unit and e2e tests for the HelpSection component #116)
- LicenseCode (Add unit and e2e tests for the LicenseCode component #117)
- LicenseCopy (Add unit and e2e tests for the LicenseCopy component #122)
- LicenseDetailsCard (Add unit and e2e tests for the LicenseDetailsCard component #118)
- LicenseDropdown (Add unit and e2e tests for the LicenseDropdown component #120)
- LicenseUseCard (Add unit and e2e tests for the LicenseUseCard component #121)
- Step (Add unit and e2e tests for the Step component #123)
- Stepper (Improve unit and e2e tests for the Stepper component #126)
- App (Add unit and e2e tests for the App component #124)
- All methods in
./src/utils/license-utilities.js(Add unit tests for license-uitilities.js #125)
Note that this specific issue is not up for grabs, and is meant to track meta discussion about the larger task of creating tests as a whole.
Metadata
Metadata
Assignees
Labels
🏷 status: label work requiredNeeds proper labelling before it can be worked onNeeds proper labelling before it can be worked on💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🤖 aspect: dxConcerns developers' experience with the codebaseConcerns developers' experience with the codebase