add unit test to trigger crash + feature resolve problem with leafref in augments

Author: frimpler

tests/schema/test_augment.c

@@ -145,6 +145,38 @@ test_leafref(void **state)
     }
 }
 
+static void
+test_leafref_w_feature1(void **state)
+{
+    int length;
+    char *path = *state;
+    const struct lys_module *module;
+
+    ly_ctx_set_searchdir(ctx, path);
+    length = strlen(path);
+    strcpy(path + length, "/leafref_w_feature1-mod3.yang");
+    if (!(module = lys_parse_path(ctx, path, LYS_IN_YANG))) {
+        fail();
+    }
+    lys_print_mem(&yang_modules[YANG_MOD_IDX(0)], module, LYS_OUT_YANG, NULL);
+}
+
+static void
+test_leafref_w_feature2(void **state)
+{
+    int length;
+    char *path = *state;
+    const struct lys_module *module;
+
+    ly_ctx_set_searchdir(ctx, path);
+    length = strlen(path);
+    strcpy(path + length, "/leafref_w_feature2-mod1.yang");
+    if (!(module = lys_parse_path(ctx, path, LYS_IN_YANG))) {
+        fail();
+    }
+    lys_print_mem(&yang_modules[YANG_MOD_IDX(0)], module, LYS_OUT_YANG, NULL);
+}
+
 static void
 test_target_augment(void **state)
 {
@@ -291,6 +323,8 @@ main(void)
         cmocka_unit_test_setup_teardown(test_target_include_submodule, setup_ctx_yin, teardown_ctx),
         cmocka_unit_test_setup_teardown(test_leafref, setup_ctx_yin, teardown_ctx),
         cmocka_unit_test_setup_teardown(test_target_augment, setup_ctx_yin, teardown_ctx),
+        cmocka_unit_test_setup_teardown(test_leafref_w_feature1, setup_ctx_yang, teardown_ctx),
+        cmocka_unit_test_setup_teardown(test_leafref_w_feature2, setup_ctx_yang, teardown_ctx),
         cmocka_unit_test_setup_teardown(test_unres_augment, setup_ctx_yin, teardown_ctx),
         cmocka_unit_test_setup_teardown(test_import_augment_target, setup_ctx_yin, teardown_ctx),
         cmocka_unit_test_setup_teardown(test_target_include_submodule, setup_ctx_yang, teardown_ctx),

tests/schema/yang/files/leafref_w_feature1-mod1.yang

@@ -0,0 +1,13 @@
+module leafref_w_feature1-mod1 {
+
+  namespace "urn:fri:params:xml:ns:yang:leafref_w_feature1-mod1";
+  prefix lr-w-ftr1-m1;
+
+  feature feature1;
+
+  container cont1 {
+    if-feature feature1;
+  }
+}
+
+

tests/schema/yang/files/leafref_w_feature1-mod2.yang

@@ -0,0 +1,34 @@
+module leafref_w_feature1-mod2 {
+
+  namespace "urn:fri:params:xml:ns:yang:leafref_w_feature1-mod2";
+  prefix lr-w-ftr1-m2;
+
+  import leafref_w_feature1-mod1 {
+    prefix lr-w-ftr1-m1;
+  }
+
+  typedef list2-ref {
+    type leafref {
+      path "/lr-w-ftr1-m1:cont1"
+         + "/lr-w-ftr1-m2:cont2"
+         + "/lr-w-ftr1-m2:list2"
+         + "/lr-w-ftr1-m2:name";
+    }
+  }
+
+  augment "/lr-w-ftr1-m1:cont1" {
+
+    description "mod2's cont1 augment";
+
+    container cont2 {
+      list list2 {
+        key name;
+        leaf name {
+          type string;
+        }
+      }
+    }
+  }
+}
+
+

tests/schema/yang/files/leafref_w_feature1-mod3.yang

@@ -0,0 +1,33 @@
+module leafref_w_feature1-mod3 {
+
+  namespace "urn:fri:params:xml:ns:yang:leafref_w_feature1-mod3";
+  prefix lr-w-ftr1-m3;
+
+  import leafref_w_feature1-mod1 {
+    prefix lr-w-ftr1-m1;
+  }
+  import leafref_w_feature1-mod2 {
+    prefix lr-w-ftr1-m2;
+  }
+
+  augment "/lr-w-ftr1-m1:cont1" {
+
+    description "mod3's cont1 augment";
+
+    container cont3 {
+      list list3 {
+        key name;
+        leaf name {
+          type string;
+        }
+        choice choice3 {
+          case case3 {
+            leaf-list llist3 {
+              type lr-w-ftr1-m2:list2-ref;
+            }
+          }
+        }
+      }
+    }
+  }
+}

tests/schema/yang/files/leafref_w_feature2-mod1.yang

@@ -0,0 +1,45 @@
+module leafref_w_feature2-mod1 {
+
+  namespace "urn:fri:params:xml:ns:yang:leafref_w_feature2-mod1";
+  prefix lr-w-ftr2-m1;
+
+  feature feature2;
+
+  container cont1 {
+    if-feature feature2;
+  }
+
+  augment "/cont1" {
+
+    container cont11 {
+
+      list list11 {
+        key name;
+
+        leaf name {
+          type string;
+        }
+      }
+    }
+  }
+
+  augment "/cont1" {
+
+    container cont12 {
+
+      list list12 {
+        key name;
+
+        leaf name {
+          type string;
+        }
+
+        leaf-list llist12 {
+          type leafref {
+            path "/cont1/cont11/list11/name";
+          }
+        }
+      }
+    }
+  }
+}