Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
Ignore empty fixture directories and fix fixtures in the parser #9113
Okay this PR started because I was annoyed that sometimes when I added fixtures somewhere but then ended up reverting, the empty folder was keep but the fixture runner was still generating an
So I went in and changed the behavior of the fixture runner to ignore folders if there is neither
I checked which folders are ignored and noticed that there are some test fixtures in babylon which are never executed because the folder structure was wrong or the files were named incorrectly.
I fixed all of these cases which are in the second commit.
I noticed another potential issue with regular expressions where we do not check if character ranges are valid, but I did not want to dive into this one yet. So the tests for this are currently skipped.
I removed the other change and moved it into a PR #9114