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.

Sign up for GitHub

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

Jump to bottom

Dev::UnitTest::Run: add the option --file #2490

Closed
bschmalhofer opened this issue Aug 24, 2023 · 4 comments
Closed

Dev::UnitTest::Run: add the option --file #2490

bschmalhofer opened this issue Aug 24, 2023 · 4 comments
Assignees
@bschmalhofer
Labels
enhancement New feature or request

Comments

@bschmalhofer
Copy link
Contributor

The options --directory and --test already exist and work fine. But sometimes it would be nice to simple pass a single file to the command.
@bschmalhofer bschmalhofer added the enhancement New feature or request label Aug 24, 2023
@bschmalhofer bschmalhofer self-assigned this Aug 24, 2023
@bschmalhofer
Copy link
Contributor Author

See also #2454

@bschmalhofer
Copy link
Contributor Author

bschmalhofer commented Aug 26, 2023 •
edited
Loading

  • While we are at it, also support that the files indicated with --test can have a trailing .t.

bschmalhofer added a commit that referenced this issue Aug 26, 2023
@bschmalhofer
Issue #2490: Fix the description of the option --test
f52ddd0 
It was stated that scripts matching a pattern are run.
But actually the complete file name, and optionally parent, dirs must be given.
That the trailing '.t' can be passed is a new feature.
bschmalhofer added a commit that referenced this issue Aug 26, 2023
@bschmalhofer
Issue #2490: allow the trailing '.t' for --test option
6b89a69 
Also, some cleanup regarding the pattern matching of the file list.
bschmalhofer added a commit that referenced this issue Aug 26, 2023
@bschmalhofer
Issue #2490: support passing an explcit file or dir
a5a7803 
in the case all other options to Dev::UnitTest::Run,
 that select the tests scripts, are ignored
@bschmalhofer
Copy link
Contributor Author

bschmalhofer commented Aug 26, 2023 •
edited
Loading

TODO:

  • Support a single argument specifying the test script or a dir
  • Passing a directory should run the test scripts in that directory
  • Passing the argument should ignores all other file selection options. No error will be reported when those options are passed

bschmalhofer added a commit that referenced this issue Aug 26, 2023
@bschmalhofer
Merge pull request #2493 from RotherOSS/issue-#2490-dev_unittest_run_…
5e28664 
…file

Issue #2490 dev unittest run file
@bschmalhofer
Copy link
Contributor Author

A quick manual test looked fine. PR is merged. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bschmalhofer bschmalhofer

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bschmalhofer