From b2d87c531837c04df5501d65ad67a56a689f3eb5 Mon Sep 17 00:00:00 2001
From: Mike Gerwitz <gerwitm@lovullo.com>
Date: Tue, 5 Jul 2016 23:57:59 -0400
Subject: Re-order graph:dep-lookup arguments to favor partial application

* src/graph.xsl (graph:dep-lookup): Re-order arguments
* test/graph.xspec: Updated accordingly
---
 test/graph.xspec | 39 ++++++++++++++++++++++-----------------
 1 file changed, 22 insertions(+), 17 deletions(-)

(limited to 'test/graph.xspec')

diff --git a/test/graph.xspec b/test/graph.xspec
index 209f965..2b44b44 100644
--- a/test/graph.xspec
+++ b/test/graph.xspec
@@ -192,13 +192,14 @@
       <scenario label="in the same package">
         <scenario label="where dependencies exist">
           <call function="graph:dep-lookup">
-            <param name="symbol"
-                   select="$foo:document/preproc:symtable
-                             /preproc:sym[ @name='local' ]" />
+            <param name="lookup" select="()" />
+
             <param name="graph"
                    select="$foo:document/preproc:sym-deps" />
 
-            <param name="lookup" select="()" />
+            <param name="symbol"
+                   select="$foo:document/preproc:symtable
+                             /preproc:sym[ @name='local' ]" />
           </call>
 
           <expect label="returns local dependencies"
@@ -210,13 +211,14 @@
 
         <scenario label="with external @src but local deps">
           <call function="graph:dep-lookup">
-            <param name="symbol"
-                   select="$foo:document/preproc:symtable
-                             /preproc:sym[ @name='external-but-not' ]" />
+            <param name="lookup" select="()" />
+
             <param name="graph"
                    select="$foo:document/preproc:sym-deps" />
 
-            <param name="lookup" select="()" />
+            <param name="symbol"
+                   select="$foo:document/preproc:symtable
+                             /preproc:sym[ @name='external-but-not' ]" />
           </call>
 
           <expect label="returns local dependencies"
@@ -228,13 +230,15 @@
 
         <scenario label="where dependencies are missing">
           <call function="graph:dep-lookup">
-            <param name="symbol"
-                   select="$foo:document/preproc:symtable
-                             /preproc:sym[ @name='missing-deps' ]" />
+            <param name="lookup" select="()" />
+
             <param name="graph"
                    select="$foo:document/preproc:sym-deps" />
 
-            <param name="lookup" select="()" />
+
+            <param name="symbol"
+                   select="$foo:document/preproc:symtable
+                             /preproc:sym[ @name='missing-deps' ]" />
           </call>
 
           <expect label="return empty sequence"
@@ -250,14 +254,15 @@
 
         <scenario label="where dependencies exist">
           <call function="graph:dep-lookup">
-            <param name="symbol"
-                   select="$foo:document/preproc:symtable
-                             /preproc:sym[ @name='external' ]" />
+            <param name="lookup"
+                   select="foo:lookup( $foo:expected-lookup )" />
+
             <param name="graph"
                    select="$foo:document/preproc:sym-deps" />
 
-            <param name="lookup"
-                   select="foo:lookup( $foo:expected-lookup )" />
+            <param name="symbol"
+                   select="$foo:document/preproc:symtable
+                             /preproc:sym[ @name='external' ]" />
           </call>
 
           <expect label="returns external dependenices"
-- 
cgit v1.2.1