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

[test] Fix non-deterministic test in plist to html #3545

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

csordasmarton
Copy link
Contributor

In one test case related to plist to html conversion we iterate
over multiple plist files, we convert it to HTML files and we also
skip duplicates.

So let's assume that I have two plist files a.plist and b.plist and both
plist files contains the same results. Let's assume that I process the a.plist
file first and convert the reports to HTML. Next if I process the b.plist I will
do not generate HTML file from it because the same reports are already in the HTML
file what we generated for a.plist and we do not generate empty HTML files anymore.

To solve this problem in the test case we will check if we have report, and in this
case the HTML file has to be exist otherwise it hasn't.

In one test case related to plist to html conversion we iterate
over multiple plist files, we convert it to HTML files and we also
skip duplicates.

So let's assume that I have two plist files a.plist and b.plist and both
plist files contains the same results. Let's assume that I process the a.plist
file first and convert the reports to HTML. Next if I process the b.plist I will
do not generate HTML file from it because the same reports are already in the HTML
file what we generated for a.plist and we do not generate empty HTML files anymore.

To solve this problem in the test case we will check if we have report, and in this
case the HTML file has to be exist otherwise it hasn't.
@csordasmarton csordasmarton added bugfix 🔨 test ☑️ Adding or refactoring tests labels Dec 9, 2021
@csordasmarton csordasmarton added this to the release 6.18.1 milestone Dec 9, 2021
@csordasmarton csordasmarton requested a review from bruntib December 9, 2021 13:27
@bruntib bruntib merged commit 100357d into Ericsson:master Dec 9, 2021
@csordasmarton csordasmarton deleted the fix_plist_to_html_test branch January 28, 2022 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix 🔨 test ☑️ Adding or refactoring tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants