Skip to content
This repository has been archived by the owner on Aug 17, 2023. It is now read-only.

Test Seeding Type Filter #233

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

saiatl2354
Copy link

Pull Request Description

Closes #185


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.

@braughtg
Copy link
Member

braughtg commented May 9, 2023

This looks like a good start. You appear to be testing that the appropriate summary table(s) appear. What you need to be testing is that the contents of the main report table has only the selected type(s) of seedings in it.

@saiatl2354
Copy link
Author

Check whether the seeding types shown in the dropdown are within the date range

I attempted to check whether the seeding types that are in the date range are shown in the dropdown. I did this by selecting the date range and checking the rpt button before checking that the dropdown has the value "ALL" and shows the "children" within that date range. I am unsure as to whether it is neccesary to test the ALL as it was done before, but our goal is to show all of the seeding types that are within the date range for this test. Below is what I did for the test, it passed, however, I am not exactly sure whether it is testing the correct features.
Screen Shot 2023-05-10 at 9 32 44 AM

@braughtg
Copy link
Member

our goal is to show all of the seeding types that are within the date range for this test.

To show that the dropdown has all of the seeding types you will need tests that generate different reports:

  • One that has only direct seedings. Then check that the dropdown contains direct seedings, but not tray seedings.
  • One that has only tray seedings. Then check that the dropdown contains tray seedings, but not direct seedings.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Seeding Report: Test Seeding Type Filter
3 participants