Getting close.... but there are still some minor timing issues against the smoke build.
I am increasing a few timeout values within postgreSQL & commands files
It seems that at times, Cypress is running so fast that it does not type entire words (it sometimes misses the first couple of letters.
I've put a small change in the userManagement test file to try and revolve this.
- Adding a wait and clear in association with the type command
Still some timing issues against the smoke build, but less this time.
- Mainly adding timeouts against cy.visit
- Improving some structure with the close modal command in a few places
This time it is adding appropriate deleteAllApps commands and adding waits
- The waits seem to be essential for the smoke build.
- Tests pass locally, but timing issues causes failure on the smoke build
Changes have been made to the following test files to remove smoke build failures:
- addRadioButtons.spec.js
- accountPortals.spec.js
- userManagement.spec.js
- userSettings.spec.js
All tests pass locally, just having issues with timing on the CI run
Also reduced test retries (runs a total of 2 times instead of 3, the 3rd run does not add any value)
Account portal tests
- Visibility when accessing the portal with different permissions
Commands
- New command to logout when app grid is not present
- Need matching data-cy commands for this
Created a new folder called adminAndManagement
- contains user and portal based tests
Timeouts refactoring
- Replacing a large number of waits with timeouts - this will prevent less time waiting during all testing
New user test
- Edits user details (for now just adds first name and last name)
Commands updated
- Removing some waits and replacing with timeouts associated with 'get' Cypress functionality
We do not need to test templates this way via the UI
- There is currently a test which creates an app via a template
- These specific templates tests are time consuming and repetitive
- These template tests will be replaced by API testing
Commands.js
- Removing a publishApp function -> There was two, one is not needed
Mitch-Budibase
Dean
Andrew Kingston
Mihail Hadzhiev
MihailHadzhiev2022
Maurits Lourens