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

Reinstate unit test coverage for Redux store reducers #3797

Closed
Tracked by #3794
ikem-legend opened this issue Sep 15, 2021 · 0 comments · Fixed by #3851
Closed
Tracked by #3794

Reinstate unit test coverage for Redux store reducers #3797

ikem-legend opened this issue Sep 15, 2021 · 0 comments · Fixed by #3851
Assignees
@gadjacobs
Labels
type: test
Milestone
Sprint 59

Comments

@ikem-legend
Copy link
Member

Description

The following files have been added to the coverage ignore list as a compromise to meet the deadlines. Since we have released v2.0.0, we'd like to increase the test coverage of the following files:

Reducers

  • src/store/reducers/settings.js
  • src/store/reducers/bookmarks.js
  • src/store/reducers/network.js

Acceptance Criteria

  • All unit tests must meet the coverage threshold
  • All functions must be unit tested
@ManuGowda ManuGowda mentioned this issue Sep 15, 2021
Closed
8 tasks
@sridharmeganathan sridharmeganathan added this to the Sprint 59 milestone Sep 29, 2021
@gadjacobs gadjacobs mentioned this issue Oct 12, 2021
Merged
gadjacobs added a commit that referenced this issue Oct 12, 2021
@gadjacobs
Merge #3797
8ee04a8
reyraa added a commit that referenced this issue Oct 12, 2021
@reyraa
Merge pull request #3851 from LiskHQ/3797-unit-test-coverage-for-redu…
93d3fe0 
…x-reducers

Unit test coverage for redux reducers - Closes #3797
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gadjacobs gadjacobs

Labels
type: test
Projects
None yet
Milestone
Sprint 59
Development

Successfully merging a pull request may close this issue.

Unit test coverage for redux reducers - Closes #3797 LiskHQ/lisk-desktop
3 participants
@gadjacobs @ikem-legend @sridharmeganathan