-
Notifications
You must be signed in to change notification settings - Fork 99
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
Add listpage and detailspage tests #3024
Conversation
Codecov Report
@@ Coverage Diff @@
## gsa-21.04 #3024 +/- ##
=============================================
+ Coverage 54.28% 57.12% +2.84%
=============================================
Files 1076 1076
Lines 26245 26246 +1
Branches 7490 7490
=============================================
+ Hits 14246 14992 +746
+ Misses 10882 10181 -701
+ Partials 1117 1073 -44
Continue to review full report at Codecov.
|
76d1c3c
to
7a4fb3a
Compare
gsa/src/web/entity/__tests__/box.js
Outdated
@@ -17,22 +17,29 @@ | |||
*/ | |||
import React from 'react'; | |||
|
|||
import Date, {setLocale} from 'gmp/models/date'; | |||
import date, {setLocale} from 'gmp/models/date'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was upper case on purpose, because it's a component model, just like we have Result, Target etc in upper case as well. Putting this into lower case might give the impression, this is some kind of transformation function.
c559f98
to
5ffef5c
Compare
Add listpage and detailspage tests (backport #3024)
What:
Add tests from master branch and port all changes from #2660
Why:
How:
Checklist: