Skip to content

Conversation

raviqqe
Copy link

@raviqqe raviqqe commented Jul 12, 2025

The test file is created like two years ago. But the filename doesn't seem to be intentional as paths starting with . are treated as hidden paths on Unix-like systems, such as macOS and Linux.

Maybe, we should also fix the file traversal logic to require at least one (non-period) character before the .feature file extension?

Edit: There are actually some other files named .feature... Should we fix them as well?

@raviqqe raviqqe changed the title Fix test filename Fix test filenames Jul 12, 2025
@tyranron tyranron added enhancement Improvement of existing features or bugfix k::testing Related to testing and/or automated tests labels Sep 10, 2025
@tyranron tyranron added this to the 0.22.0 milestone Sep 10, 2025
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@raviqqe this is the intended file layout and behavior. We want to be sure that this works, because do use the similar pattern a lot in our codebases.

The fact that files starting with . are treated as hidden, should not be a problem for you. IDEs shows them pretty well, and it's a very common practice in programming world (.gitignore and etc).

@tyranron tyranron added wontfix This will not be worked on invalid This doesn't seem right labels Sep 10, 2025
@tyranron tyranron closed this Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement of existing features or bugfix invalid This doesn't seem right k::testing Related to testing and/or automated tests wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants