search

wbond / packagecontrol.io

The Package Control website
https://packagecontrol.io
Other
111 stars 46 forks source link

PR tests returning status code 500 don't have any further information #102

Closed FichteFoll closed 4 years ago

FichteFoll commented 6 years ago

Whenever a PR test fails to run, I only receive {"__status_code__": 500} as the answer, with no context information whatsoever. Going through run_repo_tests.py, it doesn't look like __status_code__ is ever set without at least also setting error and message, so I suspect that the result data gets overwritten when the status is set here, or later in the pipeline.

Example: https://github.com/wbond/package_control_channel/pull/6937, https://travis-ci.org/wbond/package_control_channel/builds/339309986#L452

wbond commented 4 years ago

A 500 should never be expected - it just means there was a programming error in the website code.