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.

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

Db test waits #646

Merged
merged 5 commits into from
Apr 12, 2023
Merged

Conversation

braughtg
Copy link
Member

Pull Request Description

Modified the dbtest examples so that they use the cy.waitForPage() pattern that will be typical of real use cases.


Licensing Certification

FarmData2 is a Free Cultural Work and all accepted contributions are licensed as described in the LICENSE.md file. This requires that the contributor holds the rights to do so. By submitting this pull request I certify that I satisfy the terms of the Developer Certificate of Origin for its contents.

pranavm2109 and others added 5 commits April 5, 2023 17:14
* created file to test seeding input type defaults

* Filled out before each

* Our team's test shows up in correct folder so we can run in Cypress

* Added test to check if the Tray element is enabled

* Added test to check if the Direct element is enabled

* tested that neither the tray nor the direct element is selected

* tested whether the message to prompt tray/direct element selection is visible

* Tested that the form elements for Tray and Direct are not visible

* Finished testing visibility of form elements - had to add one more

* Deleted unnecessary (duplicate) test file from field kit

* Adjusted the comment in the describe and added a descriptive comment at the top of the file.

* merged second and third individual test into one test

* changed data-cy attribute of message to prompt selection of tray or direct element

---------

Co-authored-by: Alexandrialexie <brownale@dickinson.edu>
Co-authored-by: andrewscheiner53 <scheinea@dickinson.edu>
* added spec.js file

* Added it block for testing header

* checked the number of workers field is empty and enabled

* checked the number of time worked field is empty and enabled

* Added the "check correct dropdown for selected time unit"
"Checks Time units dropdown"
"checks time units default is minutes"

* couple of spacing and alignment issues fixed.

Signed-off-by: Foogabob <80643562+Foogabob@users.noreply.github.com>

* changed a test name to be more descriptive

Signed-off-by: Foogabob <80643562+Foogabob@users.noreply.github.com>

---------

Signed-off-by: Foogabob <80643562+Foogabob@users.noreply.github.com>
Co-authored-by: EliasBerhe <eliastes123@gmail.com>
Co-authored-by: Foogabob <locker@dickinson.edu>
Co-authored-by: Foogabob <80643562+Foogabob@users.noreply.github.com>
* Initial commit, adding seeding input data cypress test

* subtask #1 Assert header has 'Data'

* Check the crop dropdown list size

* Testing crop dropdown

* Testing for crop dropdown length

* Testing for crop dropdown length

* test input date element enabled and has correct default value

* Created test to check if crop selected has been enabled

* Cleaning cypress test

* Add test to check if any option was selected

* Remove dropdown element reference via children function

* combined similar 'it' blocks

* Created test to check that the crop drop down does not have a selected value

---------

Co-authored-by: nathang15 <nguyenat@dickinson.edu>
Co-authored-by: infantlikesprogramming <hadu@dickinson.edu>
@braughtg braughtg merged commit 6e189cd into DickinsonCollege:main Apr 12, 2023
braughtg added a commit that referenced this pull request Apr 12, 2023
@braughtg braughtg mentioned this pull request Apr 12, 2023
braughtg added a commit that referenced this pull request Apr 12, 2023
Reverts #646

This was to be merged into the FD2School_FarmData2 fork instead of into
the upstream.
@braughtg braughtg deleted the dbTest-waits branch April 12, 2023 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants