Navigation Menu

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

ignore json files which do not contain test/fixtures #7187

Merged
merged 3 commits into from Jul 22, 2022

Conversation

AlexKamaev
Copy link
Contributor

skip JSON files if they do not contain 'steps' statement

@AlexKamaev AlexKamaev temporarily deployed to CI July 21, 2022 08:48 Inactive
@AlexKamaev AlexKamaev temporarily deployed to CI July 21, 2022 10:10 Inactive
@AndreyBelym AndreyBelym temporarily deployed to CI July 22, 2022 08:25 Inactive
@AndreyBelym AndreyBelym merged commit 5d0c87a into DevExpress:master Jul 22, 2022
@github-actions
Copy link

Release v1.20.1-rc.1 addresses this.

@IssacH
Copy link

IssacH commented Jul 28, 2022

@AlexKamaev @AndreyBelym , recently our testcafe test cases are broken since we will put all of the json data and test specs under a 'tests' folder. We will specify the whole 'tests' folder as target for testcafe. I am just wondering if it is related with this issue? If so in our project we always use a fixed version for testcafe(1.14.2), not sure if there is any code change recently that caused the problem. BTW we'd like to know when there will be a stable version of testcafe to consume? Thanks a lot!

@need-response-app need-response-app bot added the STATE: Need response An issue that requires a response or attention from the team. label Jul 28, 2022
@AlexKamaev
Copy link
Contributor Author

AlexKamaev commented Jul 29, 2022

@IssacH The behavior you described looks exactly like the issue addressed in this PR. However, this issue cannot affect the TestCafe 1.14.2 version. At present, I cannot give you a precise estimate on when the next version will be released. However, please take a look at the 1.20.1-rc.1 version that contains a fix for the issue.

@need-response-app need-response-app bot removed the STATE: Need response An issue that requires a response or attention from the team. label Jul 29, 2022
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

Successfully merging this pull request may close these issues.

None yet

3 participants