Skip to content

Delete devel option, fix problems with tags and remove unneсessary test folders printing#41

Merged
imaliyov merged 2 commits intomainfrom
tags
Sep 14, 2023
Merged

Delete devel option, fix problems with tags and remove unneсessary test folders printing#41
imaliyov merged 2 commits intomainfrom
tags

Conversation

@hurricane642
Copy link
Copy Markdown
Collaborator

Hello everyone,
In this PR we are solving a problem with tags - previously, if certain tags were selected, a "no test folders" error would occur. The error was due to the fact that these same tags were tested at the test_perturbo_for_qe2pert stage, where the number of tests is less, and there really could be no tests with such tags. At the same time, tests for test_perturbo_for_qe2pert test selection are runned in any case - whether we run this test or not (this is how config.py works).
Now the problem is solved with 1 additional if.
Also, we removed the devel option and also removed the unnecessary printing of the line === Test folders == :
Finally, we've renamed epr_info.yaml file to the test_listing.yml file.

@hurricane642 hurricane642 added the bug Something isn't working label Sep 13, 2023
@hurricane642 hurricane642 self-assigned this Sep 13, 2023
@imaliyov imaliyov merged commit b309f6d into main Sep 14, 2023
@imaliyov imaliyov deleted the tags branch September 14, 2023 10:39
This was referenced Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants