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

Seeding Report: Test Seeding Type Filter #28

Open
4 tasks
braughtg opened this issue Jan 28, 2024 · 5 comments · May be fixed by #151
Open
4 tasks

Seeding Report: Test Seeding Type Filter #28

braughtg opened this issue Jan 28, 2024 · 5 comments · May be fixed by #151
Labels
Good Second Issue An issue that is a little deeper than a good first issue. testing Issue related to testing FarmData2 functionality

Comments

@braughtg
Copy link
Member

The tests should generate seeding reports as appropriate to check that:

  • - when “All” is selected the table shows both direct seedings and tray seedings.
  • - when “Direct” is selected the table shows only direct seedings.
  • - when “Tray” is selected the table only shows tray seedings.
  • - only the seeding types available in the date range are shown in the dropdown.

Some additional notes relevant to this issue:

  • The .spec.js file containing your test should be stored in an appropriate location and have a short but descriptive name. Use the locations and an naming from the "Good First issues" as examples.
  • The .spec.js file should include a comment at the top that describes what the file as a whole is testing.
  • The message for the describe should describe in a short phrase what the file is testing.
  • After logging in and visiting the desired page the beforeEach method should call cy.waitForPage(). This will ensure that the page is fully loaded (e.g. that all the Maps used by the page are loaded) before performing any tests.
  • It is not necessary to include a separate it for each of the things to be tested.
    • You should decide how to divide the things being tested into its so that each it tests a cohesive set of things.
    • The message for each it should describe in a short phrase what the it is testing.
  • The .spec.js files in the farmdata2/farmdata2_modules/fd2_example/ sub-tabs (e.g. ui, api) may provide some helpful examples.
  • These tests should utilize logs that are in the sample database. Information about the data contained in the sample database can be found in the "The Sample Database" section of the docker/sampleDB/README.md file.

Original issue by braughtg
Tuesday Apr 04, 2023 at 16:45 GMT

@braughtg braughtg added Good Second Issue An issue that is a little deeper than a good first issue. testing Issue related to testing FarmData2 functionality labels Jan 28, 2024
@cschnydman
Copy link

We (Chloe, Megan, Ryan) will take this.

@cschnydman cschnydman linked a pull request Apr 12, 2024 that will close this issue
@megantriplett
Copy link

Ready to review @wpgoble

@SpencerGoodman
Copy link

Hello team (Chloe, Megan, Ryan). My team has been working with these tables and had some trouble accessing different rows/columns. What data-cy attributes did you use to access these items? I just want to confirm that my group did this correctly.

@cschnydman
Copy link

Hello team (Chloe, Megan, Ryan). My team has been working with these tables and had some trouble accessing different rows/columns. What data-cy attributes did you use to access these items? I just want to confirm that my group did this correctly.

There are different ways to do it but the way we chose to is using the table data data-cy.
For example, "data-cy=td-r2c3," accesses row 2, column 3 of the sample report

@SpencerGoodman
Copy link

Hello team (Chloe, Megan, Ryan). My team has been working with these tables and had some trouble accessing different rows/columns. What data-cy attributes did you use to access these items? I just want to confirm that my group did this correctly.

There are different ways to do it but the way we chose to is using the table data data-cy. For example, "data-cy=td-r2c3," accesses row 2, column 3 of the sample report

That is very helpful. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Good Second Issue An issue that is a little deeper than a good first issue. testing Issue related to testing FarmData2 functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants