Noissue - Adds better locale support to test suite

tests/test_page_outputs.py

 import os
 import sys
 import time
 import runpy
 import signal
 import pytest
 import urllib2
 import subprocess
 from conftest import ROOTPATH
 
 
 def get_dir_contents(path):
     dirdata = {}
-    for dirpath, dirnames, filenames in sorted(os.walk(path)):

[Vasily Kuznetsov, 2017/08/11 07:08:08]

We're putting the result into a dictionary here. Does sorting actually make any
difference?

[Jon Sonesen, 2017/08/11 07:43:55]

There is no difference i meant to remove this thank you!

+    for dirpath, dirnames, filenames in os.walk(path):
         for output_file in filenames:
             filepath = os.path.join(dirpath, output_file)
             with open(filepath) as f:
                 locale = os.path.split(os.path.split(filepath)[0])[1]
                 dirdata[os.path.join(locale, output_file)] = f.read().strip()
     return dirdata
 
 
 def get_expected_outputs(test_type):
     expected_out_path = os.path.join(ROOTPATH, 'tests', 'expected_output')
@@ skipping 56 matching lines @@
 def test_dynamic(dynamic_server, filename, expected_output):
     response = urllib2.urlopen(dynamic_server + filename)
     assert response.read().strip() == expected_output
 
 
 def test_revision_arg(revision, output_pages):
     if revision is None:
         assert 'en/bar' in output_pages
     else:
         assert 'en/bar' not in output_pages