support schema v2 and dependencies on mappings

1 files changed, 71 insertions, 40 deletions

diff --git a/test/haketilo_test/unit/test_install.py b/test/haketilo_test/unit/test_install.py
index 29910cf..b1321ff 100644
--- a/test/haketilo_test/unit/test_install.py
+++ b/test/haketilo_test/unit/test_install.py
@@ -57,8 +57,38 @@ install_ext_data = {
 
 @pytest.mark.ext_data(install_ext_data)
 @pytest.mark.usefixtures('webextension')
-@pytest.mark.parametrize('complex_variant', [False, True])
-def test_install_normal_usage(driver, execute_in_page, complex_variant):
+@pytest.mark.parametrize('variant', [{
+    # The resource/mapping others depend on.
+    'root_resource_id': f'resource-abcd-defg-ghij',
+    'root_mapping_id':  f'mapping-abcd-defg-ghij',
+    # Those ids are used to check the alphabetical ordering.
+    'item_ids': [f'resource-{letters}' for letters in (
+        'a', 'abcd', 'abcd-defg-ghij', 'b', 'c',
+        'd', 'defg', 'e', 'f',
+        'g', 'ghij', 'h', 'i', 'j'
+    )],
+    'files_count': 9
+}, {
+    'root_resource_id': 'resource-a',
+    'root_mapping_id':  'mapping-a',
+    'item_ids':         ['resource-a'],
+    'files_count': 0
+}, {
+    'root_resource_id': 'resource-a-w-required-mapping-v1',
+    'root_mapping_id':  'mapping-a-w-required-mapping-v1',
+    'item_ids':         ['resource-a-w-required-mapping-v1'],
+    'files_count':      1
+}, {
+    'root_resource_id': 'resource-a-w-required-mapping-v2',
+    'root_mapping_id':  'mapping-a-w-required-mapping-v2',
+    'item_ids':         [
+        'mapping-a',
+        'resource-a',
+        'resource-a-w-required-mapping-v2'
+    ],
+    'files_count':      1
+}])
+def test_install_normal_usage(driver, execute_in_page, variant):
     """
     Test of the normal package installation procedure with one mapping and,
     depending on parameter, one or many resources.
@@ -67,41 +97,27 @@ def test_install_normal_usage(driver, execute_in_page, complex_variant):
 
     assert execute_in_page('returnval(shw());') == [[], False]
 
-    if complex_variant:
-        # The resource/mapping others depend on.
-        root_id = 'abcd-defg-ghij'
-        root_resource_id = f'resource-{root_id}'
-        root_mapping_id = f'mapping-{root_id}'
-        # Those ids are used to check the alphabetical ordering.
-        resource_ids = [f'resource-{letters}' for letters in (
-            'a', 'abcd', root_id, 'b', 'c',
-            'd', 'defg', 'e', 'f',
-            'g', 'ghij', 'h', 'i', 'j'
-        )]
-        files_count = 9
-    else:
-        root_resource_id = f'resource-a'
-        root_mapping_id = f'mapping-a'
-        resource_ids = [root_resource_id]
-        files_count = 0
-
     # Preview the installation of a resource, show resource's details, close
     # the details and cancel installation.
     execute_in_page('returnval(install_view.show(...arguments));',
-                    'https://hydril.la/', 'resource', root_resource_id)
+                    'https://hydril.la/', 'resource',
+                    variant['root_resource_id'])
 
     assert execute_in_page('returnval(shw());') == [['show'], True]
-    assert f'{root_resource_id}-2021.11.11-1'\
+    assert f'{variant["root_resource_id"]}-2021.11.11-1'\
         in containers['install_preview'].text
     assert_container_displayed('install_preview')
 
     entries = execute_in_page('returnval(ets().map(e => e.main_li.innerText));')
-    assert len(entries) == len(resource_ids)
+    assert len(entries) == len(variant['item_ids'])
+    resource_idx = variant['item_ids'].index(variant['root_resource_id'])
     # Verify alphabetical ordering.
-    assert all([id in text for id, text in zip(resource_ids, entries)])
+    assert all([id in text for id, text in
+                zip(variant['item_ids'], entries)])
 
-    assert not execute_in_page('returnval(ets()[0].old_ver);').is_displayed()
-    execute_in_page('returnval(ets()[0].details_but);').click()
+    assert not execute_in_page(f'returnval(ets()[{resource_idx}].old_ver);')\
+        .is_displayed()
+    execute_in_page(f'returnval(ets()[{resource_idx}].details_but);').click()
     assert 'resource-a' in containers['resource_preview_container'].text
     assert_container_displayed('resource_preview_container')
 
@@ -116,20 +132,24 @@ def test_install_normal_usage(driver, execute_in_page, complex_variant):
     # details, close the details and commit the installation.
     execute_in_page('returnval(install_view.show(...arguments));',
                     'https://hydril.la/', 'mapping',
-                    root_mapping_id, [2022, 5, 10])
+                    variant['root_mapping_id'], [2022, 5, 10])
 
     assert execute_in_page('returnval(shw(2));') == [['show'], True]
     assert_container_displayed('install_preview')
 
     entries = execute_in_page('returnval(ets().map(e => e.main_li.innerText));')
-    assert len(entries) == len(resource_ids) + 1
-    assert f'{root_mapping_id}-2022.5.10' in entries[0]
+    assert len(entries) == len(variant['item_ids']) + 1
+
+    all_item_ids = sorted([*variant['item_ids'], variant['root_mapping_id']])
+    mapping_idx = all_item_ids.index(variant["root_mapping_id"])
     # Verify alphabetical ordering.
-    assert all([id in text for id, text in zip(resource_ids, entries[1:])])
+    assert all([id in text for id, text in zip(all_item_ids, entries)])
 
-    assert not execute_in_page('returnval(ets()[0].old_ver);').is_displayed()
-    execute_in_page('returnval(ets()[0].details_but);').click()
-    assert root_mapping_id in containers['mapping_preview_container'].text
+    assert not execute_in_page(f'returnval(ets()[{mapping_idx}].old_ver);')\
+        .is_displayed()
+    execute_in_page(f'returnval(ets()[{mapping_idx}].details_but);').click()
+    assert variant['root_mapping_id'] in \
+        containers['mapping_preview_container'].text
     assert_container_displayed('mapping_preview_container')
 
     execute_in_page('returnval(install_view.mapping_back_but);').click()
@@ -145,16 +165,20 @@ def test_install_normal_usage(driver, execute_in_page, complex_variant):
 
     # Verify the install
     db_contents = get_db_contents(execute_in_page)
-    for item_type, ids in \
-        [('mapping', {root_mapping_id}), ('resource', set(resource_ids))]:
+    all_map_ids = {id for id in all_item_ids if id.startswith('mapping')}
+    all_res_ids = {id for id in all_item_ids if id.startswith('resource')}
+    for item_type, ids in [
+            ('mapping', all_map_ids),
+            ('resource', all_res_ids)
+    ]:
         assert set([it['identifier'] for it in db_contents[item_type]]) == ids
 
-    assert all([len(db_contents[store]) == files_count
+    assert all([len(db_contents[store]) == variant['files_count']
                 for store in ('file', 'file_uses')])
 
     # Update the installed mapping to a newer version.
     execute_in_page('returnval(install_view.show(...arguments));',
-                    'https://hydril.la/', 'mapping', root_mapping_id)
+                    'https://hydril.la/', 'mapping', variant['root_mapping_id'])
     assert execute_in_page('returnval(shw(4));') == [['show'], True]
     # resources are already in the newest versions, hence they should not appear
     # in the install preview list.
@@ -171,12 +195,19 @@ def test_install_normal_usage(driver, execute_in_page, complex_variant):
 
     # Verify the newer version install.
     old_db_contents, db_contents = db_contents, get_db_contents(execute_in_page)
-    old_db_contents['mapping'][0]['version'][-1] += 1
-    assert db_contents['mapping'] == old_db_contents['mapping']
+
+    old_root_mapping = [m for m in old_db_contents['mapping']
+                        if m['identifier'] == variant['root_mapping_id']][0]
+    old_root_mapping['version'][-1] += 1
+
+    new_root_mapping = [m for m in db_contents['mapping']
+                        if m['identifier'] == variant['root_mapping_id']][0]
+
+    assert old_root_mapping == new_root_mapping
 
     # All items are up to date - verify dialog is instead shown in this case.
     execute_in_page('install_view.show(...arguments);',
-                    'https://hydril.la/', 'mapping', root_mapping_id)
+                    'https://hydril.la/', 'mapping', variant['root_mapping_id'])
 
     fetched = lambda d: 'Fetching ' not in containers['dialog_container'].text
     WebDriverWait(driver, 10).until(fetched)