search

vgteam / sequenceTubeMap

displays multiple genomic sequences in the form of a tube map
MIT License
180 stars 25 forks source link

fix for test reporters #263

Closed rmccrear closed 1 year ago

rmccrear commented 1 year ago

The last merge broke the cli command npm run test -- ./src/whatever.test.js.

This fixes the problem by reverting test and creating a new command ci:test to be run in the CI environment on GitHub.

ci:test will run with the custom test reporters, while test will run as it did before.

This fixes the issue mentioned in https://github.com/vgteam/sequenceTubeMap/pull/261#issuecomment-1532042284

rmccrear commented 1 year ago

I also tried adding some nice html reporting with junit reporter (It looks like: this.)

But I found that it requires write permissions when running the action in order to save the report for display. So, I reverted that commit.

adamnovak commented 1 year ago

I feel like there's got to be an option we can add to the end of that command line that will make it stop parsing --reporters and accept more options afterward.