Skip to content

Commit

Permalink
Rewrite id() usage
Browse files Browse the repository at this point in the history
`select="id('a1-stmt')"` works in Oxygen but not xspec.bat or CI/CD. Expressing that select attribute a different way.

Also, I fixed a select attribute where the namespace prefix
was missing. The omission caused the test to pass for the wrong
reason (empty context).
  • Loading branch information
galtm authored and iMichaela committed Feb 26, 2024
1 parent fe22cb1 commit c8b7b54
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/utils/resolver-pipeline/testing/4_modified/modify.xspec
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,7 @@

<x:scenario label="Tests for match=set-parameter template">
<x:scenario label="set-param child does not match anything">
<x:context select="//set-parameter">
<x:context select="//o:set-parameter">
<catalog id="abc">
<param id="p1">
<label>Parameter #1</label>
Expand Down Expand Up @@ -1158,7 +1158,7 @@
</x:scenario>
<x:scenario label="Edge case: Empty $modifications param">
<x:call function="oscal:patches-to-id-targeting-ancestor">
<x:param name="here" select="id('a1-stmt')">
<x:param name="here" select="//*[@id='a1-stmt']">
<catalog id="abc">
<control id="a1">
<title>Control A</title>
Expand Down

0 comments on commit c8b7b54

Please sign in to comment.