chiark / gitweb /
Testing: Break out srcdump-save-check
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Tue, 3 Nov 2015 18:44:33 +0000 (18:44 +0000)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Tue, 3 Nov 2015 18:44:33 +0000 (18:44 +0000)
Originally I thought I would keep only one of the srcdump tests, but I
have changed my mind.  So we need to get rid of this clone-and-hack.

srcdump-save-check was identical in the two scripts.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
tests/autotest
tests/srcdump-login.at
tests/srcdump-loginback.at

index b07913f..4737b1e 100755 (executable)
@@ -98,6 +98,65 @@ proc loginas {user {password sesame} {wanttxt ACCESSGRANTED}} {
      submitform-expect $wanttxt
 }
 
      submitform-expect $wanttxt
 }
 
+proc srcdump-save-check {} {
+    global tt
+    global outertar sd files
+
+    set outertar $tt/t-srcdump.tgz
+    set sd $tt/t-srcdump.d
+
+    send "\x15$outertar\r"
+    etxt "Download complete"
+
+    exec mkdir $sd
+
+    set files [exec tar -C $sd -zvxf - < $outertar]
+
+    foreach f {
+       manifest.txt
+       licence.data       
+       cgi-auth-flexible.pm
+       .gitignore
+    } {
+       set got($f) 0
+    }
+
+    if {[file exists .git]} {
+       set got(.git/objects/) 0
+    }
+
+    foreach f $files {
+       switch -glob -- $f {
+           licence.data - manifest.txt {
+               set got($f) 1
+           }
+           *.tar {
+               foreach g [exec tar -C $sd -tf - < $sd/$f] {
+                   switch -glob -- $g {
+                       cgi-auth-flexible.pm - .gitignore - .git/objects/ {
+                           set got($g) 1
+                       }
+                       *~ {
+                           error "file $g inside $f !"
+                       }
+                       .git/* {
+                       }
+                       *.db {
+                           error "file $g inside $f outside .git !"
+                       }
+                   }
+               }
+           }
+       }
+    }
+
+    foreach g [array names got] {
+       if {!$got($g)} {
+           error "file $g missing"
+       }
+    }
+}
+
 source $testfile
 
 puts "$id: ok"
 source $testfile
 
 puts "$id: ok"
index 8fbfb8c..895fa36 100644 (file)
@@ -29,56 +29,4 @@ submitform-expect "Viewing"
 
 loginas alice sesame "Save file to"
 
 
 loginas alice sesame "Save file to"
 
-set outertar $tt/t-srcdump.tgz
-set sd $tt/t-srcdump.d
-
-send "\x15$outertar\r"
-etxt "Download complete"
-
-exec mkdir $sd
-
-set files [exec tar -C $sd -zvxf - < $outertar]
-
-foreach f {
-    manifest.txt
-    licence.data       
-    cgi-auth-flexible.pm
-    .gitignore
-} {
-    set got($f) 0
-}
-
-if {[file exists .git]} {
-    set got(.git/objects/) 0
-}
-
-foreach f $files {
-    switch -glob -- $f {
-       licence.data - manifest.txt {
-           set got($f) 1
-       }
-       *.tar {
-           foreach g [exec tar -C $sd -tf - < $sd/$f] {
-               switch -glob -- $g {
-                   cgi-auth-flexible.pm - .gitignore - .git/objects/ {
-                       set got($g) 1
-                   }
-                   *~ {
-                       error "file $g inside $f !"
-                   }
-                   .git/* {
-                   }
-                   *.db {
-                       error "file $g inside $f outside .git !"
-                   }
-               }
-           }
-       }
-    }
-}
-
-foreach g [array names got] {
-    if {!$got($g)} {
-       error "file $g missing"
-    }
-}
+srcdump-save-check
index 76554ab..4a2e402 100644 (file)
@@ -29,56 +29,4 @@ send "B/source available\r\r"
 
 etxt "Save file to"
 
 
 etxt "Save file to"
 
-set outertar $tt/t-srcdump.tgz
-set sd $tt/t-srcdump.d
-
-send "\x15$outertar\r"
-etxt "Download complete"
-
-exec mkdir $sd
-
-set files [exec tar -C $sd -zvxf - < $outertar]
-
-foreach f {
-    manifest.txt
-    licence.data       
-    cgi-auth-flexible.pm
-    .gitignore
-} {
-    set got($f) 0
-}
-
-if {[file exists .git]} {
-    set got(.git/objects/) 0
-}
-
-foreach f $files {
-    switch -glob -- $f {
-       licence.data - manifest.txt {
-           set got($f) 1
-       }
-       *.tar {
-           foreach g [exec tar -C $sd -tf - < $sd/$f] {
-               switch -glob -- $g {
-                   cgi-auth-flexible.pm - .gitignore - .git/objects/ {
-                       set got($g) 1
-                   }
-                   *~ {
-                       error "file $g inside $f !"
-                   }
-                   .git/* {
-                   }
-                   *.db {
-                       error "file $g inside $f outside .git !"
-                   }
-               }
-           }
-       }
-    }
-}
-
-foreach g [array names got] {
-    if {!$got($g)} {
-       error "file $g missing"
-    }
-}
+srcdump-save-check