1159786 - Write tests to cover code in th_publisher.post_treeherder_collection function

Status: RESOLVED WONTFIX

(Tree Management :: Treeherder, defect, P3)

Description

[Cameron Dawson [:camd]]

We had an error where this line:

    if not response or response.status != 200:

was accidentally set to:

    if not response or response.status == 200:

Which broke things, but didn't break any tests.  We need at least 1 test that this will break so we can catch it.  

However, due to the way ``thclient`` is written using ``httplib`` instead of ``requests``, this is not possible.  Our test framework doesn't support the same response values.  So waiting on that bug to be fixed.

Comment 1

[Ed Morley [:emorley]]

post_treeherder_collection no longer exists.