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

aggregateCSVReportsTwo should create two aggregate reports #272

Open
TGiles opened this issue Jul 23, 2021 · 2 comments
Open

aggregateCSVReportsTwo should create two aggregate reports #272

TGiles opened this issue Jul 23, 2021 · 2 comments
Labels
approved For work that @TGiles approves tests

Comments

@TGiles
Copy link
Owner

TGiles commented Jul 23, 2021

Describe the bug
There's an intermittent bug (that I can't reproduce locally) that involves aggregateCSVReportsTwo should create two aggregate reports.

To Reproduce
I wish I knew how to consistently reproduce. I have no idea why this is happening.

Expected behavior
The test should consistently pass (or consistently fail if new code breaks the existing behavior).

Additional context
If anyone else is able to reproduce this consistently, I'd be very grateful. I can't reproduce on my Windows box or Linux VM, only seems to happen on the CI servers.

@TGiles TGiles added approved For work that @TGiles approves tests labels Jul 23, 2021
@TGiles
Copy link
Owner Author

TGiles commented Aug 5, 2021

The failure rate seems to have gone down with the changes introduced in 57c916a but I'm going to keep this open for a bit longer to make sure.

@L00Cyph3r
Copy link
Contributor

What is the exact error you are getting? As far as I can see the only thing that could mean something is that _readReportDirectory() does a console.error which might trigger something. If you have an error or a link to a failed CI-job I can try to poke around a bit more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved For work that @TGiles approves tests
Projects
None yet
Development

No branches or pull requests

2 participants