This test spec file fires off against every PR. There is some timing issues currently.
I am making a few changes in relation to timeout values (a small restructure too) in hopes this helps with the PR builds.
Tests pass locally for me, so there may be more changes required against this spec file as I check its performance
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