Show More
@@ -3,10 +3,10 b'' | |||
|
3 | 3 | """ |
|
4 | 4 | from test_pr import load_results, post_logs, post_results_comment, print_results |
|
5 | 5 | |
|
6 | num, results, pr = load_results() | |
|
6 | num, results, pr, unavailable_pythons = load_results() | |
|
7 | 7 | results_urls = post_logs(results) |
|
8 | print_results(pr, results_urls) | |
|
9 | post_results_comment(pr, results_urls, num) | |
|
8 | print_results(pr, results_urls, unavailable_pythons) | |
|
9 | post_results_comment(pr, results_urls, num, unavailable_pythons) | |
|
10 | 10 | |
|
11 | 11 | print() |
|
12 | 12 | print("Posted test results to pull request") |
@@ -128,7 +128,7 b' def run_tests(venv):' | |||
|
128 | 128 | # Restore $PATH |
|
129 | 129 | os.environ["PATH"] = orig_path |
|
130 | 130 | |
|
131 | def markdown_format(pr, results_urls): | |
|
131 | def markdown_format(pr, results_urls, unavailable_pythons): | |
|
132 | 132 | def format_result(py, passed, gist_url, missing_libraries): |
|
133 | 133 | s = "* %s: " % py |
|
134 | 134 | if passed: |
@@ -151,11 +151,11 b' def markdown_format(pr, results_urls):' | |||
|
151 | 151 | "Not available for testing: " + ", ".join(unavailable_pythons)] |
|
152 | 152 | return "\n".join(lines) |
|
153 | 153 | |
|
154 | def post_results_comment(pr, results, num): | |
|
155 | body = markdown_format(pr, results) | |
|
154 | def post_results_comment(pr, results, num, unavailable_pythons=unavailable_pythons): | |
|
155 | body = markdown_format(pr, results, unavailable_pythons) | |
|
156 | 156 | gh_api.post_issue_comment(gh_project, num, body) |
|
157 | 157 | |
|
158 | def print_results(pr, results_urls): | |
|
158 | def print_results(pr, results_urls, unavailable_pythons=unavailable_pythons): | |
|
159 | 159 | print("\n") |
|
160 | 160 | if pr['mergeable']: |
|
161 | 161 | print("**Test results for commit %s merged into master**" % pr['head']['sha'][:7]) |
@@ -174,7 +174,7 b' def print_results(pr, results_urls):' | |||
|
174 | 174 | |
|
175 | 175 | def dump_results(num, results, pr): |
|
176 | 176 | with open(os.path.join(basedir, 'lastresults.pkl'), 'wb') as f: |
|
177 | pickle.dump((num, results, pr), f) | |
|
177 | pickle.dump((num, results, pr, unavailable_pythons), f) | |
|
178 | 178 | |
|
179 | 179 | def load_results(): |
|
180 | 180 | with open(os.path.join(basedir, 'lastresults.pkl'), 'rb') as f: |
General Comments 0
You need to be logged in to leave comments.
Login now