Skip to content

Conversation

Konboi
Copy link
Contributor

@Konboi Konboi commented Aug 3, 2021

I want to record test results without a datetime suffix.
Because I need to send test results as the same name to use CircleCi's test in parallel (https://circleci.com/docs/2.0/parallelism-faster-jobs/#splitting-by-timing-data)

However, the xmlrunner adds a date-time suffix to a test suite automatically. Although there are the same test cases, the CircleCI realizes different test cases.

So I add an outsuffix option to control a suffix from the command.

e.g)

xmlrunner --output ./test-results/  --outsuffix="" tests/**/test_*.py

$ xmlrunner -h
usage: xmlrunner [-h] [-o DIR | --output-file FILENAME] [--outsuffix STRING]

optional arguments:
  -h, --help            show this help message and exit
  -o DIR, --output DIR  Directory for storing XML reports ('.' default)
  --output-file FILENAME
                        Filename for storing XML report
  --outsuffix STRING    Output suffix (timestamp is default)

How about this option? Will you please review it?

@Konboi
Copy link
Contributor Author

Konboi commented Aug 4, 2021

@dnozay I added test code. Could you review this?

@Konboi
Copy link
Contributor Author

Konboi commented Sep 6, 2021

@dnozay or anyone What is this status...? Is there anything need to do?

@dnozay
Copy link
Member

dnozay commented Sep 14, 2021

@Konboi , sorry, yes, could you please rebase so it runs the checks?
thank you

@Konboi Konboi force-pushed the add-outsuffix-option branch from 758c93e to f4e9173 Compare September 15, 2021 02:32
@Konboi
Copy link
Contributor Author

Konboi commented Sep 15, 2021

@dnozay Thank you for your reply. I rebased.
Could you approve to run github actions?

@Konboi
Copy link
Contributor Author

Konboi commented Sep 29, 2021

@dnozay Could you approve to run Github actions...?

@dnozay dnozay merged commit 0ef90a6 into xmlrunner:master Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants