search

testomatio / check-tests

GitHub action with static analysis for JavaScript tests.
https://testomat.io
20 stars 8 forks source link

--no-empty option seems to be ignored on backend #135

Open tikolakin opened 11 months ago

tikolakin commented 11 months ago

Describe the bug There seems to be a no way to get rid of empty test suites

To Reproduce I have tried to import test with both options

Screenshot 2023-09-29 at 18 10 46 Screenshot 2023-09-29 at 18 09 18

After import - empty suite remains available

image

Expected behavior When noempty is sent as true - Testomatio removes empty suites

TetianaKhomenko commented 5 months ago

Hi @tikolakin! The issue was fixed. The fix will be published in the next release.

TetianaKhomenko commented 4 months ago

Hi @tikolakin the fix is live now. Please check it and reply if it works for you.

tikolakin commented 4 months ago

It doesn't work, after importing with --no-empty. Tested with https://github.com/testomatio/check-tests/releases/tag/0.8.24 Empty folders/suites are created in testomatio Also, recent changes seem to introduce a regression to the folder structure, prefixing it with the source folder. It wasn't like it before.

TetianaKhomenko commented 2 months ago

Hi @tikolakin! The issue should be fixed now. Can we ask you to check it on new imports?