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

Are silent SKIPs useful or detrimental to the quality of testing? #5853

Open
janekmi opened this issue Aug 7, 2023 · 0 comments
Open

Are silent SKIPs useful or detrimental to the quality of testing? #5853

janekmi opened this issue Aug 7, 2023 · 0 comments
Labels
Priority: 4 low QA: CI .github/ and utils/ related to automated testing Type: Question A question

Comments

@janekmi
Copy link
Contributor

janekmi commented Aug 7, 2023

QUESTION: Are silent SKIPs useful or detrimental to the quality of testing?

Facts

  • Currently, we have several require_* calls that generate silent SKIPs
  • Silent SKIPs were introduced to simplify eye-balling test logs. Since a smaller number of log lines are simpler to comprehend.
  • Silent SKIPS are sometimes used to substitute for the lack of a mechanism allowing to ask for running only tests that fulfil a given conditions e.g. are meant to be run for a given CPU architecture.
  • At the same time, silent SKIPs do not allow us to easily assess whether a given test was skipped rightfully or wrongly for the given configuration.

Discussion

In the case of a silent SKIP, the only observable fact is that the given record is missing. Note that you have to know what you are looking for to spot the given record is missing which imposes on the reader of the test logs an additional requirement of knowing what tests are meant to be run for the given configuration.

Possibly, this requirement is inevitable but till now it has not been accepted by our team.

Maybe silent SKIPs should be abandoned and instead, other mechanisms should be introduced to address the real issues silent SKIPs were used for.

Issues silent SKIPs address and alternative solutions for them

@janekmi janekmi added the Type: Question A question label Aug 7, 2023
@janekmi janekmi added QA: CI .github/ and utils/ related to automated testing Priority: 4 low labels Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 4 low QA: CI .github/ and utils/ related to automated testing Type: Question A question
Projects
None yet
Development

No branches or pull requests

1 participant