Issue 5333 - Allow cms to generate relative pages

tests/test_page_outputs.py

 import os
 import sys
 import runpy
 
 import mock
 import pytest
 
 from .conftest import ROOTPATH
 from .utils import get_dir_contents, exception_test
 from cms.sources import FileSource
 from cms.bin.test_server import DynamicServerHandler
 
 
-def get_expected_outputs(test_type):
-    expected_out_path = os.path.join(ROOTPATH, 'tests', 'expected_output',
-                                     test_type)
-    outputs = get_dir_contents(expected_out_path)
-    expected_out_path = os.path.join(ROOTPATH, 'tests', 'expected_output',

[Vasily Kuznetsov, 2018/11/07 18:03:02]

Reusing variables like this is not a very good practice. It can lead to people
introducing bugs in the future by accident.

Also, perhaps we can abstract away the code repetition here. Something like
this:

    root = os.path.join(ROOTPATH, 'tests', 'expected_output')
    outputs = get_dir_contents(os.path.join(root, 'common'))
    outputs.update(get_dir_contents(os.path.join(root, test_type)))

[rhowell, 2018/11/07 20:05:26]

Done.

-                                     'common')
-    outputs.update(get_dir_contents(expected_out_path))

[Vasily Kuznetsov, 2018/11/07 18:03:02]

It seems that the correct approach should be to load commons and then update
them with the outputs for our test type. We want specific expected outputs to
have precedence over the common ones. Or am I missing something?

[rhowell, 2018/11/07 20:05:26]

Done.

+def get_expected_outputs(test_type, with_common=True):
+    root = os.path.join(ROOTPATH, 'tests', 'expected_output')
+    outputs = {}
+    if with_common:
+        outputs = get_dir_contents(os.path.join(root, 'common'))
+    outputs.update(get_dir_contents(os.path.join(root, test_type)))
     return outputs.items()
 
 
 static_expected_outputs = get_expected_outputs('static')
 dynamic_expected_outputs = get_expected_outputs('dynamic')
-relative_expected_outputs = get_expected_outputs('relative')

[Vasily Kuznetsov, 2018/11/07 18:03:02]

My proposal was to also skip the common expected outputs for relative tests.
This could be accomplished by adding a boolean flag to `get_expected_outputs` so
that its signature becomes something like `get_expected_outputs(test_type,
with_common=True)`. Then we can use `get_expected_outputs('relative', False)` to
only get the few tests we actually want to do for relative links.

[rhowell, 2018/11/07 20:05:25]

Done.

+relative_expected_outputs = get_expected_outputs('relative', False)
 
 
 def generate_static_pages(temp_site, tmpdir_factory, *extra_args):
     dst_path = str(tmpdir_factory.mktemp('static'))
     sys.argv = ['filler', temp_site, dst_path] + list(extra_args)
     with mock.patch('cms.sources.FileSource.version', 1):
         runpy.run_module('cms.bin.generate_static_pages',
                          run_name='__main__')
     return get_dir_contents(dst_path)
 
@@ skipping 44 matching lines @@
 
 
 @pytest.mark.parametrize('page', ['en/translate', '/en/translate'])
 def test_dynamic_server_handler_with_conflicts(page, temp_site_with_conflicts):
     handler = DynamicServerHandler('localhost', 5000,
                                    str(temp_site_with_conflicts))
     environ = {'PATH_INFO': page}
     exp_msg = 'The requested page conflicts with another page.'
 
     exception_test(handler, Exception, exp_msg, environ, lambda x, y: None)