-
Notifications
You must be signed in to change notification settings - Fork 172
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
Report name patch #51
Open
JamesMTSloan
wants to merge
16
commits into
oldani:master
Choose a base branch
from
JamesMTSloan:report_name_patch
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- support for combining reports into single report (combine_reports kwarg) - support for test cases with underscores in name - optional timestamping of filenames (add_timestamp kwarg) - optional automatic opening of generated reports in browser tab (paths to reports are returned from test result) (open_in_browser kwarg) - support for optional user variables to be passed to jinja2 template (template_args kwarg) - added tracebacks to reports - added stdout to reports (so tests that pass can also be expanded - print relative paths to generated reports - made default output directory the current working directory so there are now no required args also: - updated and adjusted readme - changed use of deprecated _TextTestResult -> TextTestResult - changed format of template slightly
…oid clashes from duplicate names) - simplified test method names
…f a string to allow the user to customise their template more easily. also some other small tweaks.
reduced prepended paths to testcase names except for when common testcase names need to be distinguished
…ting test suites, including case where name collision between similarly named test cases is handled. combined reports and separate reports are shown.
added support for skipped tests skip reasons changed template to support subtests in sub-tables fixed bug where non-combined tests had summaries with details from all tests tweaked format of HTML has that info buttons line up better
…a stub in the report. this needed some rather ugly filtering but unittest seems to insist on calling addSuccess() in this case removed support for std.out in subtests for now as this was being accumulated across subtests
#2) * Updated results.py to use the copy library so that it can be run using Python 2.7 * changed import order
# Conflicts: # tests/test.py # tests/test2.py
@JamesMTSloan thanks you for the help here. Could you rebase this, to merge it? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
bugfix for #49