[LNT] Python 3 support: don't assume URL params order

[thopre]

Summary

Test server/ui/V4Pages.py tests on 2 occasions an address redirection
with parameters in it. However order of those parameters changes between
Python 2 and Python 3. This commit adapts the tests to accept all
possible orders.

[cmatthews]

We should probably use the url parser on these, and check them formally. This is okay for now though.

[thopre]

Committed as 6505c12235d59535dbfabc01e481c2e53f26a7a9

tests/server/ui/V4Pages.py

check_code(client, '/v4/nts/9999?json=true',
                expected_code=HTTP_NOT_FOUND)
     check_code(client, '/v4/nts/9999/report',
                expected_code=HTTP_NOT_FOUND)
     check_code(client, '/v4/nts/9999/text_report',
                expected_code=HTTP_NOT_FOUND)
 
     # Get a graph page. This has been changed to redirect.
     check_redirect(client, '/v4/nts/1/graph?test.3=2',
-                   'v4/nts/graph\?plot\.0=1\.3\.2&highlight_run=1$')
+                   'v4/nts/graph\?'
+                   '(plot\.0=1\.3\.2&highlight_run=1'
+                   '|highlight_run=1&plot\.0=1\.3\.2)$')
 
     # Get a run that contains generic producer information
     check_producer_label(client, '/v4/nts/7',
                          ['Current', '152293', '2012-05-10T16:28:23',
                           '0:00:35', 'Producer'])
     check_producer_label(client, '/v4/nts/8',
                          ['Current', '152294', '2012-05-11T16:28:23',
                           '0:00:35', 'Producer'])
def main():
     resp = check_code(client, '/__health')
     assert resp.get_data(as_text=True) == "Ok"
     resp = check_code(client, '/ping')
     assert resp.get_data(as_text=True) == "pong"
 
     # Check we can convert a sample into a graph page.
     graph_to_sample = check_code(client, '/db_default/v4/nts/graph_for_sample/10/compile_time?foo=bar',
                                  expected_code=HTTP_REDIRECT)
-    assert graph_to_sample.headers['Location'] == "http://localhost/db_default/v4/nts/graph?foo=bar&plot.0=2.6.0"
+    assert graph_to_sample.headers['Location'] in (
+            "http://localhost/db_default/v4/nts/graph?foo=bar&plot.0=2.6.0",
+            "http://localhost/db_default/v4/nts/graph?plot.0=2.6.0&foo=bar")
 
     # Check that is we ask for a sample or invalid field, we explode with 400s.
     check_code(client, '/db_default/v4/nts/graph_for_sample/10000/compile_time?foo=bar',
                expected_code=HTTP_NOT_FOUND)
     check_code(client, '/db_default/v4/nts/graph_for_sample/10/not_a_metric?foo=bar',
                expected_code=HTTP_BAD_REQUEST)
 
     # check get_geomean_comparison_result with empty unchanged_tests
     check_html(client, '/v4/nts/11')
     check_html(client, '/v4/nts/12')
 
 
 if __name__ == '__main__':
     main()