search

jlaska / pytest-github

Plugin for py.test that associates tests with github issues using a marker
MIT License
18 stars 9 forks source link

Update --github-summary to show issue state #9

Closed jladdjr closed 6 years ago

jladdjr commented 6 years ago

Updates the output of --github-summary to show two separate lists of issues - Unresolved Issues and Resolved Issues. Issues are deemed to be resolved if they are closed or contain a label specified by --github-completed. Also updates the output to sort the issues by issue number.

Sample output:

Unresolved Issues
https://github.com/pytest-github/open/issues/1
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_bar
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz
https://github.com/pytest-github/open/issues/2
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_bar
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz
https://github.com/pytest-github/open/issues/3
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_bar
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz

Resolved Issues
https://github.com/pytest-github/closed/issues/4
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_foo
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz
https://github.com/pytest-github/closed/issues/5
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_foo
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz
https://github.com/pytest-github/closed/issues/6
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_foo
 - test_param_github_summary_multiple_issues0/test_param_github_summary_multiple_issues.py:test_baz
coveralls commented 6 years ago

Coverage Status

Coverage increased (+1.1%) to 95.833% when pulling 159eed62038583f024f52ddfc5903a866e27547f on jladdjr:github_summary_issue_state into 6449a30468653451f2aea270748124b233900169 on jlaska:master.

unlikelyzero commented 6 years ago

LGTM. It looks like pytest doesn't have another error code to include in the 'skipped' or 'ignored' state

jlaska commented 6 years ago

@jladdjr thanks for this update. Just one comment, but otherwise looks good.

jladdjr commented 6 years ago

Awesome, updated, think it's good to go

jlaska commented 6 years ago

Merging ... should we update and push out a new release?

jladdjr commented 6 years ago

@jlaska - heck yeah!