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

Introduce updated cypress test #3167

Merged
merged 101 commits into from
Sep 1, 2022
Merged

Introduce updated cypress test #3167

merged 101 commits into from
Sep 1, 2022

Conversation

rabilrbl
Copy link
Contributor

@rabilrbl rabilrbl commented Jul 19, 2022 •
edited
Loading

Base #1804
Resolves #2601

Changes

  • Update to cypress 10
  • Introduce proper TS types in tests
  • Fix captcha issue on multiple login requests
  • Added API based login, improves testing speed

Setup

  • yarn install
  • Incase if post setup script on npm install fails.
    yarn cypress:install
  • Start dev server
    yarn start
  • Open cypress dashboard
    yarn cypress:open
  • From cypress 10 there is no way to run all spec test at once, hence each test should be triggered manually. Ref

Abhinandan Arya added 30 commits September 6, 2021 08:10
upgrade cypress to 8.3.1
d7d72b4
reinitialise cypress
ce20592 
1. keeping examples for reference
2. consistent with code structure in `src`
wrap login to separate cypress command
af34b00
refactor auth test - store user in fixture
1a19d76
wrap patient create into cy command
22a4acd
wrap facility crud as cy command
24ca0a4
refactor facility spec into its
3394f6b
facility spec refactored, session persist command added
818c2ca
refactored profile edit spec
7605fda
added locations, inventory tests
2d1417f 
but running them still pending
just initialise some test boilerplate
1e3fa62
add user_crud test boilerplate with todos
53d07f4
user spec doing
588a450
added sample test spec boilerplate
8d90146
persist localstorage for avoiding repeated logins
78835bc 
due to recaptcha prompt
fix error
d4891a1
fix user test as possible
f5c7455 
skipped it for some time
sample test basic test added
b1cbcdc
sample list basic test added
4798ee7
add facility inventory basic tests
e80b7d9
delete cypress examples
c1c1fc3
fixing death_report test
94c348e
add shifting manage, filter tests
6b44a2f
added boilerplate tests for external_results, resource
4c0afe1
change in convention
e1aca96 
every folder will have its same name spec file denoting basic features of that page
for example: `assets.spec.js` in `assets_spec`
add name to user select
5c1568c
fix shifting filter test
a446ca5
download button in shifting test fixed
23ebdcc 
bug found `Cannot read property of undefined(res) in BoardView.tsx` to be fix
fix download error -
8e1bdb4 
TypeError: Cannot read properties of undefined (reading 'data')
    at triggerDownload (BoardView.tsx:72)
fix download error -
edfc240 
TypeError: Cannot read properties of undefined (reading 'data')
    at triggerDownload (ResourceBoardView.tsx:61)
@nihal467
Copy link
Member

nihal467 commented Aug 25, 2022 •
edited
Loading

@rabilrbl

image

  • the sample_test is not working properly

image

  • the shifting filters are not working properly

image

  • profile edit is not working

image

  • why does the death report is having the patient as undefined?

@rabilrbl
Copy link
Contributor Author

rabilrbl commented Aug 26, 2022 •
edited
Loading

@nihal467
Errors

  • Sample_Test
  • Shifting Filter ( couldn't reproduce the error displayed, but fixed other errors found )
  • Profile Edit ( unable to reproduce all tests was successful )
    image
  • Death Report - The issue is caused by some component in front end, not cypress.

@nihal467
Copy link
Member

@rabilrbl

image

  • the shifting filters are not working properly, can you check it

image

  • the user_crud is malfunctioning, can you check it

@nihal467
Copy link
Member

test approved

@sonarcloud
Copy link

sonarcloud bot commented Sep 1, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 6 Security Hotspots
Code Smell A 7 Code Smells

No Coverage information No Coverage information
6.8% 6.8% Duplication

@khavinshankar khavinshankar merged commit d96bd98 into coronasafe:develop Sep 1, 2022
@Ashesh3 Ashesh3 mentioned this pull request Sep 19, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khavinshankar khavinshankar khavinshankar approved these changes

@vigneshhari vigneshhari Awaiting requested review from vigneshhari

@gigincg gigincg Awaiting requested review from gigincg

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

Successfully merging this pull request may close these issues.

Fix Cypress
4 participants
@rabilrbl @nihal467 @khavinshankar @gigincg