Adding extra tests for resolving relative file paths

Author: ialarmedalien

tests/test_utils/input/imports_relative/L0_0/L1_0_0/L2_0_0_2/four.yaml

@@ -0,0 +1,13 @@
+id: four
+name: import_four
+title: Import Four
+description: |
+  Import loaded by the StepChild class.
+imports:
+  - linkml:types
+classes:
+  Four:
+    attributes:
+      value:
+        range: string
+        ifabsent: "Four"

tests/test_utils/input/imports_relative/L0_0/L1_0_0/L2_0_0_2/one.yaml

@@ -0,0 +1,14 @@
+id: one
+name: import_one
+title: Import One
+description: |
+  Import loaded by the StepChild class.
+imports:
+  - linkml:types
+  - two
+classes:
+  One:
+    attributes:
+      value:
+        range: string
+        ifabsent: "One"

tests/test_utils/input/imports_relative/L0_0/L1_0_0/L2_0_0_2/stepchild.yaml

@@ -0,0 +1,16 @@
+id: stepchild
+name: stepchild
+title: stepchild
+description: |
+  Child class that imports files in the same directory as itself without consistently using `./` in the link notation.
+imports:
+  - linkml:types
+  - one
+  - two
+  - ./three
+classes:
+  StepChild:
+    attributes:
+      value:
+        range: string
+        ifabsent: "StepChild"

tests/test_utils/input/imports_relative/L0_0/L1_0_0/L2_0_0_2/three.yaml

@@ -0,0 +1,14 @@
+id: three
+name: import_three
+title: Import Three
+description: |
+  Import loaded by the StepChild class.
+imports:
+  - linkml:types
+  - ./four
+classes:
+  Three:
+    attributes:
+      value:
+        range: string
+        ifabsent: "Three"

tests/test_utils/input/imports_relative/L0_0/L1_0_0/L2_0_0_2/two.yaml

@@ -0,0 +1,13 @@
+id: two
+name: import_two
+title: Import Two
+description: |
+  Import loaded by the StepChild class.
+imports:
+  - linkml:types
+classes:
+  Two:
+    attributes:
+      value:
+        range: string
+        ifabsent: "Two"

tests/test_utils/input/imports_relative/L0_0/L1_0_0/main.yaml

@@ -8,10 +8,11 @@ imports:
   - ../../L0_1/cousin
   - ./L2_0_0_0/child
   - ./L2_0_0_1/child
+  - L2_0_0_2/stepchild
 classes:
   Main:
     description: "Our intrepid main class!"
     attributes:
       value:
         range: string
-        ifabsent: "Main"
+        ifabsent: "Main"

tests/test_utils/test_schemaview.py

@@ -19,7 +19,7 @@
 
 SCHEMA_NO_IMPORTS = Path(INPUT_DIR) / 'kitchen_sink_noimports.yaml'
 SCHEMA_WITH_IMPORTS = Path(INPUT_DIR) / 'kitchen_sink.yaml'
-SCHEMA_WITH_STRUCTURED_PATTERNS = Path(INPUT_DIR) / "pattern-example.yaml"
+SCHEMA_WITH_STRUCTURED_PATTERNS = Path(INPUT_DIR) / 'pattern-example.yaml'
 SCHEMA_IMPORT_TREE = Path(INPUT_DIR) / 'imports' / 'main.yaml'
 SCHEMA_RELATIVE_IMPORT_TREE = Path(INPUT_DIR) / 'imports_relative' / 'L0_0' / 'L1_0_0' / 'main.yaml'
 SCHEMA_RELATIVE_IMPORT_TREE2 = Path(INPUT_DIR) / 'imports_relative' / 'L0_2' / 'main.yaml'
@@ -357,7 +357,7 @@ def test_caching():
     view.add_class(ClassDefinition('X'))
     assert len(['X']) == len(view.all_classes())
     view.add_class(ClassDefinition('Y'))
-    assert len(['X', 'Y']) ==  len(view.all_classes())
+    assert len(['X', 'Y']) == len(view.all_classes())
     # bypass view method and add directly to schema;
     # in general this is not recommended as the cache will
     # not be updated
@@ -544,8 +544,13 @@ def test_imports_relative():
         '../../L0_1/L1_1_0/index',
         '../../L0_1/cousin',
         '../L1_0_1/dupe',
-        './L2_0_0_0/child',
-        './L2_0_0_1/child',
+        'L2_0_0_0/child',
+        'L2_0_0_1/child',
+        'L2_0_0_2/two',
+        'L2_0_0_2/one',
+        'L2_0_0_2/four',
+        'L2_0_0_2/three',
+        'L2_0_0_2/stepchild',
         'main'
     ]
 
@@ -716,6 +721,7 @@ def test_slot_inheritance():
     with pytest.raises(ValueError):
         view.slot_ancestors('s5')
 
+
 def test_attribute_inheritance():
     """
     Tests attribute inheritance edge cases.