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

Should assertions have their own CSV file? #415

Closed
jscholes opened this issue Apr 6, 2021 · 3 comments
Closed

Should assertions have their own CSV file? #415

jscholes opened this issue Apr 6, 2021 · 3 comments
Labels
Agenda+Test Writing For discussion during the next teleconference related to test writing and manual test runs enhancement New feature or request tests About assistive technology tests

Comments

@jscholes
Copy link
Contributor

jscholes commented Apr 6, 2021

With 6 assertions, tests.csv will currently have 15 headed columns. That is approaching unfriendly spreadsheet territory for me, particularly given that the number of columns will grow with the number of assertions (some of our tests have 8, I think).

I propose that a new file be used, assertions.csv, with the following columns:

  1. testId
  2. assertion1
  3. ...
  4. assertionN

This would make tests.csv more manageable, particularly if some of the suggestions from #364 were implemented as well (e.g. to cut out the title column altogether).

@jscholes jscholes added enhancement New feature or request tests About assistive technology tests Agenda+Test Writing For discussion during the next teleconference related to test writing and manual test runs labels Apr 6, 2021
@sinabahram
Copy link

This will also have other advantages like reducing git noise, potential conflicts due to topological changes in the original spreadsheet, and more. +1

@jscholes
Copy link
Contributor Author

jscholes commented Apr 8, 2021

Another potential avenue: have a separate assertions.csv. In that fiel, have rows with assertion ID, text and priority (which could be built into text). Then, in tests.csv, we could just reference an array of assertion IDs to facilitate reuse.

@mcking65
Copy link
Contributor

mcking65 commented Sep 7, 2023

resolved by #974.

@mcking65 mcking65 closed this as completed Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+Test Writing For discussion during the next teleconference related to test writing and manual test runs enhancement New feature or request tests About assistive technology tests
Projects
None yet
Development

No branches or pull requests

3 participants