Skip to content

Commit

Permalink
Fixed: [Choice and Leafref](#469)
Browse files Browse the repository at this point in the history
  • Loading branch information
olofhagsand committed Jan 17, 2024
1 parent bb603e0 commit 9e6557a
Show file tree
Hide file tree
Showing 4 changed files with 172 additions and 5 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ Developers may need to change their code

### Corrected Bugs

* Fixed: [Choice and Leafref](https://github.com/clicon/clixon/issues/469)
* Fixed: [Problem deleting non-last list element if ordered-by user](https://github.com/clicon/clixon/issues/475)
* Fixed: [Tab completion mounted devices with lists](https://github.com/clicon/clixon-controller/issues/72)
* Fixed: kill-session cleanup when client none existant, and for all db:s
Expand Down
17 changes: 13 additions & 4 deletions lib/src/clixon_path.c
Original file line number Diff line number Diff line change
Expand Up @@ -255,8 +255,9 @@ clixon_path_print(FILE *f,
/*! Given an XML node, return root node
*
* A root node is an ancestor xr of x with one or both of the following properties
* - its XML parent is NULL parent,
* - its associated yang specification's parent is a yang module.
* (1) its XML parent is NULL parent,
* (2) its associated yang specification's parent is a yang module.
* Note that if there are no yang-specs, only (1) applies
* @param[in] x XML node
* @param[out] xr XML root
* @retval 0 OK
Expand All @@ -270,13 +271,21 @@ xml_yang_root(cxobj *x,
yang_stmt *yp;

while ((xp = xml_parent(x)) != NULL){
if ((y = xml_spec(x)) != NULL &&
(yp = yang_parent_get(y)) != NULL)
if ((y = xml_spec(x)) != NULL){
while ((yp = yang_parent_get(y)) != NULL){
if (yang_datanode(yp))
break;
if (yang_keyword_get(yp) == Y_MODULE ||
yang_keyword_get(yp) == Y_SUBMODULE)
break;
y = yp;
}
/* Actually, maybe only the Y_MODULE clause is relevant */
if (yp==NULL ||
yang_keyword_get(yp) == Y_MODULE ||
yang_keyword_get(yp) == Y_SUBMODULE)
break; /* x is the root */
}
x = xp;
}
*xr = x;
Expand Down
18 changes: 17 additions & 1 deletion lib/src/clixon_xpath_yang.c
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,23 @@ xp_yang_eval_step(xp_yang_ctx *xy0,
} /* nodetest xs_type */
break;
case A_PARENT:
xy->xy_node = yang_parent_get(ys);
{
yang_stmt *yp;
ys1 = ys;
while ((yp = yang_parent_get(ys1)) != NULL){
if (yang_datanode(yp)){
ys1 = yp;
break;
}
if (yang_keyword_get(yp) == Y_MODULE ||
yang_keyword_get(yp) == Y_SUBMODULE){
ys1 = yp;
break;
}
ys1 = yp;
}
xy->xy_node = ys1;
}
break;
default:
clixon_err(OE_YANG, 0, "Invalid path-arg step: %s",
Expand Down
141 changes: 141 additions & 0 deletions test/test_leafref_choice.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,141 @@
#!/usr/bin/env bash
# Yang leafref + choice test
# See https://github.com/clicon/clixon/issues/469

# Magic line must be first in script (see README.md)
s="$_" ; . ./lib.sh || if [ "$s" = $0 ]; then exit 0; else return 0; fi

APPNAME=example

cfg=$dir/conf_yang.xml
fyang=$dir/leafref.yang

cat <<EOF > $cfg
<clixon-config xmlns="http://clicon.org/config">
<CLICON_CONFIGFILE>$cfg</CLICON_CONFIGFILE>
<CLICON_YANG_DIR>$dir</CLICON_YANG_DIR>
<CLICON_YANG_DIR>${YANG_INSTALLDIR}</CLICON_YANG_DIR>
<CLICON_YANG_DIR>$IETFRFC</CLICON_YANG_DIR>
<CLICON_YANG_MAIN_FILE>$fyang</CLICON_YANG_MAIN_FILE>
<CLICON_CLISPEC_DIR>/usr/local/lib/$APPNAME/clispec</CLICON_CLISPEC_DIR>
<CLICON_CLI_DIR>/usr/local/lib/$APPNAME/cli</CLICON_CLI_DIR>
<CLICON_CLI_MODE>$APPNAME</CLICON_CLI_MODE>
<CLICON_SOCK>/usr/local/var/run/$APPNAME.sock</CLICON_SOCK>
<CLICON_BACKEND_PIDFILE>/usr/local/var/run/$APPNAME.pidfile</CLICON_BACKEND_PIDFILE>
<CLICON_XMLDB_DIR>$dir</CLICON_XMLDB_DIR>
</clixon-config>
EOF

cat <<EOF > $fyang
module example{
yang-version 1.1;
namespace "urn:example:clixon";
prefix ex;
grouping fruits-and-flowers{
list fruit {
key "name";
leaf name {
type string;
}
}
list flower {
key "name";
leaf name {
type string;
}
}
choice myChoice {
description "Fruit or Flowers";
case fruit {
leaf fruit-name {
description "Fruit name";
type leafref {
path "../fruit/name";
}
}
}
case flower {
leaf flower-name {
description "Flower name";
type leafref {
path "../flower/name";
}
}
}
}
}
container c {
uses fruits-and-flowers;
}
uses fruits-and-flowers;
}
EOF

# Leafref and choice test
# Args:
# 1: prefix
function testrun()
{
prefix=$1

new "add fruit"
expectpart "$($clixon_cli -1f $cfg set $prefix fruit apple)" 0 "^$"

new "add fruit"
expectpart "$($clixon_cli -1f $cfg set $prefix fruit orange)" 0 "^$"

new "add flower"
expectpart "$($clixon_cli -1f $cfg set $prefix flower daisy)" 0 "^$"

new "add flower"
expectpart "$($clixon_cli -1f $cfg set $prefix flower rose)" 0 "^$"

new "commit"
expectpart "$($clixon_cli -1f $cfg commit)" 0 "^$"

new "expand fruit leafref"
expectpart "$(echo "set $prefix fruit-name ?" | $clixon_cli -f $cfg -o CLICON_CLI_EXPAND_LEAFREF=true 2> /dev/null)" 0 apple orange

new "add fruit-name"
expectpart "$($clixon_cli -1f $cfg set $prefix fruit-name apple)" 0 "^$"

new "validate"
expectpart "$($clixon_cli -1f $cfg validate)" 0 "^$"
}

new "test params: -f $cfg"

if [ $BE -ne 0 ]; then
new "kill old backend"
sudo clixon_backend -zf $cfg
if [ $? -ne 0 ]; then
err
fi
new "start backend -s init -f $cfg"
start_backend -s init -f $cfg
fi

new "wait backend"
wait_backend

new "Test top-level"
testrun ""

new "Test in container"
testrun "c "

if [ $BE -ne 0 ]; then
new "Kill backend"
# Check if premature kill
pid=$(pgrep -u root -f clixon_backend)
if [ -z "$pid" ]; then
err "backend already dead"
fi
# kill backend
stop_backend -f $cfg
fi

rm -rf $dir

new "endtest"
endtest

0 comments on commit 9e6557a

Please sign in to comment.