This repository has been archived by the owner on Jan 9, 2023. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
test: migrate tests to React Testing Library #2516
Merged
matteovivona
merged 568 commits into
HospitalRun:master
from
OpenSourceRaidGuild:master
Jan 15, 2021
Merged
Changes from 22 commits
Commits
Show all changes
568 commits
Select commit
Hold shift + click to select a range
5eea518
Merge pull request #161 from emma-r-slight/emma-labsList
nobrayner 0a9570f
test(duplicatenewpatientmodal.test.tsx): update tests to use RTL
emma-r-slight 865dcfd
Merge remote-tracking branch 'origin/master' into codyarose_Diagnosis…
codyarose 7c00669
Merge pull request #165 from codyarose/codyarose_Labs
nobrayner af591cb
Merge pull request #160 from codyarose/codyarose_DiagnosisForm
nobrayner 8159479
fix(searchpatients.test): add "await" to async tests
emma-r-slight c4e194b
Merge pull request #170 from emma-r-slight/emma-duplicatePatient
nobrayner 0fc87c2
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans 120995a
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans 1b2f258
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans 147013b
Convert MedicationsList.test.tsx to RTL
codyarose a264672
Merge pull request #162 from kcdraidgroup/juhanakristian-improve-visi…
JacobMGEvans ed71f41
Merge pull request #153 from kcdraidgroup/juhanakristian-152
JacobMGEvans d84a4e0
Merge pull request #151 from kcdraidgroup/juhanakristian-150
JacobMGEvans f55e09b
Merge pull request #173 from codyarose/codyarose_MedicationsList
JacobMGEvans 53ed6e2
Convert ImportantPatientInfo.test.tsx to RTL
codyarose b4f1dbc
Tests Setup and Cleanup for GeneralInformation test conversion
JacobMGEvans f760ef8
Merge pull request #176 from codyarose/codyarose_ImportantPatientInfo
JacobMGEvans 3748223
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans bceb054
test(patients): convert to rtl
nobrayner 5ef29a1
test(relatedpersonstab): convert to rtl
nobrayner 94fe727
fix(tests): cleaning up mocks so they don't leak
nobrayner 92c027d
test(view appointments): make tests time agnostic
nobrayner ffa4d9e
Convert AddRelatedPersonModal.test.tsx to RTL
codyarose 00064ec
test(query-hooks): update remaining query hooks to use executeQuery u…
mpeyper cc46717
test(contactinfo.test.tsx): update test to use RTL
emma-r-slight c9f73f3
Merge pull request #182 from mpeyper/mpeyper
JacobMGEvans d64822e
Merge pull request #183 from emma-r-slight/emma-ContactInfo
JacobMGEvans 0e7d25e
GeneralInfo Error test converted
JacobMGEvans 2594cde
Update AddRelatedPersonModal to fix save test
codyarose ad4c873
More than half of non editable input scenerios are converted
JacobMGEvans 67101bd
Merge remote-tracking branch 'origin/master' into codyarose_AddRelate…
codyarose 9fa9fd3
fix(test): failing in ci
nobrayner f028b10
fix(test): increase timeout
nobrayner 46c77dc
Mostly done with converting non-editable describe block
JacobMGEvans b3be647
Merge pull request #181 from codyarose/codyarose_AddRelatedPersonModal
nobrayner 059bac4
Merge pull request #180 from kcdraidgroup/nobrayner-163
nobrayner f12f344
fix(test): incorrect date formats
nobrayner 0638d44
fix(test): add typing delay so search gets called correctly
nobrayner baf9c2b
Abstracted assertion pattern that was repeating
JacobMGEvans ce51bca
fix(tests): incorrectly resetting canvas mock
nobrayner ce82875
test(appointment detail form): convert to rtl
nobrayner 988ad76
GeneralInformation RTL Conversion complete
JacobMGEvans bfd107f
Merge pull request #185 from kcdraidgroup/nobrayner-184
nobrayner 31a0a70
match the i18 key environment
JacobMGEvans cb00401
Datepicker element in CI not found
JacobMGEvans a58d186
fix(test): increase findBy timeouts for dates
nobrayner 7fe5ee0
fix(test): prettier error
nobrayner d2866f2
test(appointments): convert to rtl
nobrayner 42f5f8d
Merge pull request #187 from kcdraidgroup/nobrayner-186
nobrayner 9e69c0b
fix(test): pass correctly formatted dates
nobrayner 791bfdc
Merge pull request #179 from kcdraidgroup/jacobmgevans
nobrayner b028a4d
Convert NewPatient.test.tsx to RTL
codyarose 18083f6
test(mutation-hooks): cleaned up excessive use of act in mutation hoo…
mpeyper 6fd366f
test(hooks): clean up hook tests
mpeyper 4aaa334
test(network-status-message): remove unnecessary usage of @testing-li…
mpeyper 0dbddc1
Merge pull request #189 from mpeyper/mpeyper
JacobMGEvans b564ccf
chore(new patient): comment out test for now
nobrayner ffeae97
Merge pull request #188 from codyarose/codyarose_NewPatient
JacobMGEvans 2e0a242
Refactor test setup for file
JacobMGEvans 6bb0bfa
Refactored test setup Sidebar file
JacobMGEvans 855611f
Merge pull request #190 from kcdraidgroup/jacobmgevans
JacobMGEvans 1a49469
chore(enzyme): remove enzyme
nobrayner 775c8d6
chore(test): cleanup
nobrayner 2a8f982
chore(deps): add react-test-renderer
nobrayner 01c1b52
fix(test): attempt to fix bad datetime test
nobrayner 1957426
Merge pull request #191 from kcdraidgroup/remove-enzyme
nobrayner f6d8271
Merge commit '95ad3252e548e33c86bfde9304d07cfab608e81b' into aprillion
Aprillion 2899482
Merge pull request #193 from kcdraidgroup/aprillion
Aprillion 6bbd731
fix(test): editing address in GeneralInformation
Aprillion 733b26e
fix(test): `NaN` is an invalid value
Aprillion 53be3d1
Merge pull request #196 from kcdraidgroup/aprillion
Aprillion c5fab6b
Partially related to issue #195, remove customHook redundant tests in…
JacobMGEvans 39d5aa4
Merge pull request #197 from kcdraidgroup/jacobmgevans
JacobMGEvans 4dff109
Remove redundant test and clean up ImagingRequestTable
codyarose ca47677
Added settings tests
JacobMGEvans 29c4c76
Merge pull request #199 from kcdraidgroup/jacobmgevans
JacobMGEvans 67d6d29
Replace IncidentRepository calls, improve table query
codyarose 26e35dd
Refactor setup to remove dupe code, removed redundant test, improved …
codyarose e1aa92b
Fixed updateTitle (setState) call not being within useEffect
codyarose 2192e60
Refactored to use screen instead of container where possible, inline …
codyarose 405ec6d
test(medicationlist.test.tsx): improve test coverage
emma-r-slight 49f96da
fix(test): improve CareGoalForm
Aprillion 63e7bb4
test(appointmentslist.test.tsx): update table test to use getByRole
emma-r-slight f18900b
test(viewallergy.test.tsx): remove act from file and add getbyrole
emma-r-slight abc6718
Merge pull request #208 from kcdraidgroup/aprillion
JacobMGEvans 081246c
Merge pull request #207 from emma-r-slight/emma-refactor-tests
JacobMGEvans 941c724
Merge pull request #205 from codyarose/codyarose_Incidents
JacobMGEvans 7a45922
Merge pull request #203 from codyarose/codyarose_ViewIncidentsTable
JacobMGEvans 4f154e3
Merge pull request #201 from codyarose/codyarose_ViewIncidents
JacobMGEvans 7a162f2
Merge pull request #200 from codyarose/codyarose
JacobMGEvans cb47735
fix(test): check ViewIncidents expected data
Aprillion 77266c0
fix(test): improve VisitForm
Aprillion 8a88313
Add aria-label to make form role available
codyarose 5b8d39c
Refactored to follow RTL setup convention, improved /incidents/new te…
codyarose 8da5bc4
fix(test): improve DateTimePickerWithLabelFormGroup
Aprillion f056391
fix(test): improve DatePickerWithLabelFormGroup
Aprillion afee1a2
Added name to getByRole query
codyarose a42b3ee
chore(test): delete VisitForm
Aprillion b055637
Merge pull request #213 from kcdraidgroup/aprillion-4
mpeyper 12d80ce
Merge pull request #211 from codyarose/codyarose
JacobMGEvans 7ebd924
Merge pull request #210 from kcdraidgroup/aprillion-2
JacobMGEvans 0cb3562
Merge pull request #209 from kcdraidgroup/aprillion-1
JacobMGEvans 13b8e41
Merge pull request #212 from kcdraidgroup/aprillion-3
JacobMGEvans 7d4a8b0
Merge pull request #214 from emma-r-slight/emma-improve-tests
JacobMGEvans db92fcb
Refactored setup function
codyarose fd539b9
fix(test): add visit modal should check fields are disabled
nobrayner 21aa5f1
Merge pull request #215 from codyarose/codyarose_ImagingRequestTable
JacobMGEvans 19d2517
Merge pull request #216 from kcdraidgroup/nobrayner-x
JacobMGEvans d4d8e96
fix(test): fix non-null assertion check warning
codyarose ed61bcf
test(ViewIncident.test.tsx): remove redundant breadcrumbs component test
codyarose 99a0752
actually remove, not just comment out
codyarose f7e8606
feat: add testid to visit field for testing
codyarose 1f1c109
refactor: improve queries - remove function call tests
codyarose 789fff8
Merge pull request #217 from codyarose/codyarose
nobrayner 5992dad
Merge pull request #218 from codyarose/codyarose_ViewIncidents
nobrayner a22a455
Merge pull request #219 from codyarose/codyarose_NewLabRequest
JacobMGEvans 9dc8111
Revert "refactor(NewLabRequest): improve queries - remove function ca…
JacobMGEvans f3d5f0e
Merge pull request #220 from kcdraidgroup/revert-219-codyarose_NewLab…
JacobMGEvans cde4a39
Remove querySelector from ViewApplointments
juhanakristian 2015a17
Merge pull request #221 from kcdraidgroup/juhanakristian-ViewAppointm…
JacobMGEvans 3add6f6
test(incidents): improve assertions & render
tigerabrodi fa037d6
Merge pull request #223 from kcdraidgroup/tigerabrodi-improve-inciden…
tigerabrodi a3c09a8
Merge branch 'master' of https://github.com/HospitalRun/hospitalrun-f…
JacobMGEvans c06b426
fix(test): report incident
nobrayner 3862c6e
fix(test): navbar
nobrayner 9940f72
fix(test): sidebar
nobrayner 4903ba5
Merge pull request #224 from kcdraidgroup/jacobmgevans
nobrayner 026e44d
feat: add testid to visit field for testing
codyarose 3aa4b89
refactor: improve queries - remove function call tests
codyarose 5d6952c
Merge remote-tracking branch 'origin/master' into codyarose_NewLabReq…
codyarose 8d98934
fix: merge conflict
nobrayner ac35b21
Merge pull request #225 from codyarose/codyarose_NewLabRequest
nobrayner 7c53673
Merge pull request #226 from kcdraidgroup/update-upstream
nobrayner e9ee1f8
test(report-incident): improvements
tigerabrodi 76f5280
test(view-lab): improvements
tigerabrodi 2bdc2e0
Merge pull request #228 from kcdraidgroup/tigerabrodi-report-incident…
tigerabrodi 4763780
test(medication-request-table): improvements
tigerabrodi 692893c
Merge pull request #230 from kcdraidgroup/tigerabrodi-improve-viewlab…
tigerabrodi 386574b
fixup! redo changes
tigerabrodi e94d817
Merge pull request #232 from kcdraidgroup/tigerabrodi-medication-requ…
JacobMGEvans f3139e8
Revert "test(report-incident): improvements"
JacobMGEvans 8693002
Merge pull request #238 from kcdraidgroup/revert-228-tigerabrodi-repo…
JacobMGEvans d736383
Removes render setup
juhanakristian 6ba9e6a
Replace render with setup
juhanakristian 94f5b9f
Remove unnecessary asyncs
juhanakristian 05d2e17
Merge pull request #235 from kcdraidgroup/juhanakristian-patientsearc…
JacobMGEvans 67c29c7
Merge pull request #237 from kcdraidgroup/juhanakristian-viewpatients…
JacobMGEvans f12b4db
fix(test): can create care goal
nobrayner 467f063
fix(test): increase timeout
nobrayner a9acd72
fix(test): better wait for assertion
nobrayner 3115437
fix(test): increase waitFor timeout
nobrayner 5e4eba4
fix(test): linting
nobrayner 4cf0e76
chore: use jest config instead of setup tests
nobrayner 31c2b1e
test: improve labs/Labs.test.tsx
codyarose 2e19466
fix(test): increase timeout
nobrayner c47ce48
Merge pull request #240 from kcdraidgroup/nobrayner-prfb-1
nobrayner a1b4eb2
test: improve labs/ViewLab.test~
codyarose b17f15e
Merge remote-tracking branch 'origin/master' into codyarose_ViewLab
codyarose 90a1757
Merge remote-tracking branch 'origin/master' into codyarose_Labs
codyarose 6aa7654
Merge pull request #244 from codyarose/codyarose_ViewLab
nobrayner 0c1ac98
Merge pull request #242 from codyarose/codyarose_Labs
nobrayner 8782d24
test: improve labs/ViewLabs.test
codyarose d5ae24a
Merge remote-tracking branch 'origin/master' into codyarose_ViewLabs
codyarose 118d285
test: search functionality test
codyarose e81e959
fix(test): use dashboard + url to check
nobrayner 86122a9
fix(test): replace spinner and snapshots with better user assertions
nobrayner fc0f308
fix(test): consistent it instead of test
nobrayner 6727764
fix(test): more descriptive test titles + test cleanup
nobrayner 1e7ff86
Merge pull request #248 from kcdraidgroup/nobrayner-prfb-2
nobrayner a2e644b
test(viewincidentdetails.test.tsx): add back deleted tests using RTL
emma-r-slight e707c77
test(viewincidentdetails.test.tsx): finish improving test description…
emma-r-slight d7659bc
feat: add testid's to NewMedicationRequest fields
codyarose 40df690
test: improve NewMedicationRequest test
codyarose afb2a77
Merge remote-tracking branch 'origin/master' into codyarose_NewMedica…
codyarose 13dea3e
test: remove act import
codyarose f3e9b12
Merge remote-tracking branch 'origin/master' into codyarose_ViewLabs
codyarose 6835c6e
test(improve test descriptions): improve Test descriptions
emma-r-slight f97d303
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
emma-r-slight 0ca15f2
Merge pull request #253 from emma-r-slight/emma-improve-test-desriptions
JacobMGEvans d34999a
Merge pull request #252 from codyarose/codyarose_NewMedicationRequest
JacobMGEvans 128a9ff
test(allergieslist.test): update alert test
emma-r-slight d7090b4
Merge pull request #254 from emma-r-slight/emma-improveTests
JacobMGEvans 99326ff
test: improve queries, remove unused mocks
codyarose f6ec2e5
Merge remote-tracking branch 'origin/master' into codyarose_Medications
codyarose 70a1bc0
test: readded mocks
codyarose 8722ac7
test: refactored setup function
codyarose 06e143c
test: refactor setup function, update queries
codyarose 78f0566
test: add modal open/close test to Allergies
codyarose 8a845dc
test: remove redundant close and save tests => covered in patients/al…
codyarose 7898ddf
test: add waitforelementtoberemoved
codyarose b0a87e1
Merge pull request #246 from codyarose/codyarose_ViewLabs
JacobMGEvans 1d3cf9e
Merge pull request #256 from codyarose/codyarose_Medications
JacobMGEvans 3131da3
Merge pull request #257 from codyarose/codyarose_Allergies
JacobMGEvans e72e76f
Remove eslint console ignores
JacobMGEvans e92f559
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans 051048e
Naming convention improved per Jacks suggestion
JacobMGEvans 03fcc58
Merge pull request #259 from kcdraidgroup/jacobmgevans
JacobMGEvans 88c3e47
Merge remote-tracking branch 'origin/master' into codyarose_Allergies…
codyarose 35af11a
Fix ViewLabs.test.tsx
JacobMGEvans c9e23d1
Merge pull request #258 from codyarose/codyarose_AllergiesList
JacobMGEvans 3d019ca
Merge branch 'master' of https://github.com/kcdraidgroup/hospitalrun-…
JacobMGEvans f824248
Removed negative confirmation
JacobMGEvans 92e996a
Progress save
JacobMGEvans 0ffa12f
test: refactor to check for title in /appointments
codyarose 012bd0a
Merge remote-tracking branch 'origin/master' into codyarose_HospitalRun
codyarose 7c884ab
Fixed ViewLabs Search functionality test
JacobMGEvans 6c62b5b
Merge pull request #260 from kcdraidgroup/jacobmgevans
JacobMGEvans 4943047
test(newimagingrequest.test): change test descriptions re PR review
emma-r-slight 0bf14fd
Merge pull request #261 from codyarose/codyarose_HospitalRun
nobrayner 63499e5
test: correct expect based on test description
codyarose 671b0b3
Merge remote-tracking branch 'origin/master' into codyarose_ViewAllergy
codyarose e80383e
feat: add testids to inputs whose labels are not properly connected
codyarose f342ec5
Merge pull request #262 from emma-r-slight/emma-improveTests
nobrayner 21487e5
test: clean up format imports
codyarose e5e83e3
Merge pull request #263 from codyarose/codyarose_ViewAllergy
nobrayner c8c61a3
Merge pull request #264 from codyarose/codyarose_InputTestIds
nobrayner 7ac2655
test: refactor to remove now redundant testids
codyarose 337cdb7
refactor: destructured lodash imports
codyarose 4ec48e8
Merge pull request #265 from codyarose/codyarose_DateFnsCleanup
nobrayner aa7b639
test: refactor patient form queries
codyarose 967e52c
test: refactor requests queries
codyarose 69c5e13
Merge remote-tracking branch 'origin/master' into codyarose_UseInputT…
codyarose 9821b83
test(noteslist.test): remove queryselector from test
emma-r-slight 402392c
test(viewpatient.test.tsx): improve test description remove querysel…
emma-r-slight 8ccb322
Merge pull request #267 from emma-r-slight/emma-improveTests
JacobMGEvans 8e9228a
Merge remote-tracking branch 'origin/master' into codyarose_UseInputT…
codyarose acafa59
test: refactor patient info queries
codyarose 21700c5
test: refactor setup and correct test
codyarose ac3efd2
test: refactor setup and queries
codyarose 0b5535a
test: refactor setup to remove wrapper
codyarose 84aba16
Copied tests from AddRelatedPersonModal.test.tsx to RelatedPersonsTab…
juhanakristian 8d27975
Merge pull request #268 from codyarose/codyarose_RemoveWrappers
nobrayner 07bbb35
Merge pull request #266 from codyarose/codyarose_UseInputTestids
nobrayner 45451f3
Deletes AddRelatedPersonModal.test.tsx. Cleanup and improve integrati…
juhanakristian a57cab7
Fix variable name
juhanakristian 998a785
Increase timeout
juhanakristian 8d82cd8
Update with Upstream Master
JacobMGEvans 0182da3
Fixed the change that utilized the requestedByFullName for defining t…
JacobMGEvans 3888dc5
Merge pull request #271 from kcdraidgroup/jacobmgevans
JacobMGEvans fbd79d9
removed comment
JacobMGEvans b6f8d3d
Merge pull request #272 from kcdraidgroup/jacobmgevans
JacobMGEvans 7fb20b0
test: add form labels and refactor queries
codyarose 758daba
test: refactor sidebar active link queries
codyarose 4860ebb
test: refactor to remove querySelector calls
codyarose 6f3567c
test: removed unused imports
codyarose 4e7e446
fix: increase timeout
nobrayner 1ea596f
test: add back more semantic spinner queries
codyarose ac4e1e9
Merge pull request #273 from codyarose/codyarose_RemoveQuerySelectors
nobrayner a5f3b13
Merge pull request #270 from kcdraidgroup/juhanakristian-addrelatedpe…
nobrayner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
mmm i don't know if i want my labs being requested by Dr Strange LOL