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

Progress bar/completion status stops if test is marked as FAIL. #1103

Open
Monoloqu3 opened this issue Jun 2, 2023 · 4 comments
Open

Progress bar/completion status stops if test is marked as FAIL. #1103

Monoloqu3 opened this issue Jun 2, 2023 · 4 comments

Comments

@Monoloqu3
Copy link

Monoloqu3 commented Jun 2, 2023

It's a bit confusing that the progress bar/completion status is stopped even though the test suite is still running/finished testing.
I'm not sure if this is a bug or it was planed to be like that.
Has anyone found a workaround for this?

Here is screenshot showing how it looks like in my case: https://imgur.com/Myvn1Jk
The status says "Ran 5 of 6 phases" even though it has already ran 6 of 6 phases.

@Monoloqu3 Monoloqu3 changed the title Progress bar/completion status stops if any of the tests are marked as FAIL. Progress bar/completion status stops if test is marked as FAIL. Jun 2, 2023
@kehrazy
Copy link
Contributor

kehrazy commented Jun 4, 2023

if (this.test.status === TestStatus.waiting) {
return 0;
} else if (this.test.status === TestStatus.pass) {
return this.test.phases.length;
}

Looks like this is intended behaviour.

I assume the logic here is that we can easily look into our GUI and find out easily when did our tests fail.

@dbhatman
Copy link
Collaborator

dbhatman commented Jun 5, 2023

If I understand correctly, this is a bug as it looks like the total count of run phases is incorrect?

From a quick look https://github.com/google/openhtf/blame/5f11f03af022f7d041d14f135512ca9e71d8359b/openhtf/output/web_gui/src/app/stations/station/test-summary.component.ts#L47 should change from PhaseStatus.fail to PhaseStatus.waiting as fail is a completed phase indicator. If you have a chance please give that a shot and feel free to send a PR. Otherwise I'll see if I can find some time to test this out.

@Monoloqu3
Copy link
Author

If I understand correctly, this is a bug as it looks like the total count of run phases is incorrect?

From a quick look https://github.com/google/openhtf/blame/5f11f03af022f7d041d14f135512ca9e71d8359b/openhtf/output/web_gui/src/app/stations/station/test-summary.component.ts#L47 should change from PhaseStatus.fail to PhaseStatus.waiting as fail is a completed phase indicator. If you have a chance please give that a shot and feel free to send a PR. Otherwise I'll see if I can find some time to test this out.

Thanks! I will this soltution and let you know about the result.

@Monoloqu3
Copy link
Author

If I understand correctly, this is a bug as it looks like the total count of run phases is incorrect?

From a quick look https://github.com/google/openhtf/blame/5f11f03af022f7d041d14f135512ca9e71d8359b/openhtf/output/web_gui/src/app/stations/station/test-summary.component.ts#L47 should change from PhaseStatus.fail to PhaseStatus.waiting as fail is a completed phase indicator. If you have a chance please give that a shot and feel free to send a PR. Otherwise I'll see if I can find some time to test this out.

Could be a nooby issue, but it seems like in web_gui folder I don't have "src" folder. I installed OpenHTF using pip.
This is how my path looks like: C:\Users\name\venv\openhtf\Lib\site-packages\openhtf\output\web_gui

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants