-
Notifications
You must be signed in to change notification settings - Fork 14
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
[Labels] Cannot specify Full-Class ordering from another file #50
Comments
Just to be clear (I may nor understand the issue correctly):
Is that correct, or do you mean something else? |
|
Ok, I understand that you want to use just the module name as marker to achieve the same, but that could be a bit tricky. I'm not sure that I could get this working without breaking anything else without major refactorings. You wrote in the other issue that you already adapted the code and could make a PR - I would be interested in that! |
I guess I should try again with About my patch, alright, I'll open a Draft PR to discuss this tonight (it may not suit your intended design though, we'll see :D) |
Hello,
In order to order the tests of a personal project, I've attempted to order tests on the basis of files (or the classes within it), in order to keep the ordering specification short and concise. Note that I've also encountered issues ordering tests in a similar way to the documentation, by specifying a class name as the label.
Alas, it seems that is does not work this way (or is not intended to ?), as I only receive a Warning telling me that the mark is ignored, in light of the failure to match a node_id with the provided label.
Looking a bit into the code, I noticed that the code path handling Class-based dependencies has the constraint of containing exactly two
::
strings, meaning that it must probably be a test-name from within a specific test-class.I believe that it would be quite practical to be able to order tests relatively to test classes, not only from the same file, but also from other test files, as the documentation does not hint at the impossibility to do so.
Thank you for maintaining this piece of code, in hope that my feedback here may help the project and other users !
The text was updated successfully, but these errors were encountered: