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

cli: add --audit and --auditList options #600

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bzoz
Copy link
Contributor

@bzoz bzoz commented Aug 20, 2018

In short: on Windows running citgm-all --auditList platform will list all modules that are flaky/skipped on win32. Running citgm-all --audit platform will run the smoke tests for all the skipped/flaky modules.

Adds citgm-all --auditList option which will list all modules that are flaky or skipped on current machine, including the the reason a module is skipped:

  • version
  • distro
  • release
  • platform
  • endian
  • fips

Running citgm-all --auditList reson1 reasn2... will show only the modules that are skipped or flaky because they match given reason.

Calling citgm-all --audit will run the smoke tests for all modules that are skipped or flaky. Tested modules can also be filtered by providing list of "reasons" to test.

Checklist
  • npm test passes
  • tests are included
  • documentation is changed or added
  • contribution guidelines followed here

The test for current master do not pass neither on my WSL installation nor on my Win10 box. I've added some tests, I'll add more when I get test\bin\test-citgm-all.js work on Windows.

Adds `citgm-all --auditList` option which will list all modules that
are flaky or skipped on current machine, including the the matched
reason: "version", "distro", "release", "platform", "endian" or "fips".

Running `citgm-all --auditList reson1 reasn2...` will show only the
modules that are skipped or flaky because they match given reason.

Calling `citgm-all --audit` will run the smoke tests for all modules
that are skipped or flaky. Tested modules can also be filtered by
providing list of "reasons" to test.
profnandaa

This comment was marked as off-topic.

@MylesBorins
Copy link
Contributor

Should we close this?

@bzoz
Copy link
Contributor Author

bzoz commented Apr 6, 2019

I think we should land this, but I didn't have time recently to write tests.

@BridgeAR BridgeAR added the WIP Work in progress label Apr 13, 2019
@MylesBorins MylesBorins changed the base branch from master to main August 14, 2020 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants