[devscripts] 11/14: Fix various (potential) test failures.

James McCoy jamessan at debian.org
Thu Nov 21 03:53:46 UTC 2013


This is an automated email from the git hooks/post-receive script.

jamessan pushed a commit to branch master
in repository devscripts.

commit b8eae5b8c457786179672cebdec7d8c6551844c6
Author: James McCoy <jamessan at debian.org>
Date:   Tue Nov 19 22:35:19 2013 -0500

    Fix various (potential) test failures.
    
    Some devscripts commands can be configured by a config file.  If that is
    present in the test environment, it could cause a false negative test
    failure, as I noticed with the new uscan test.  Use --no-conf to ignore
    the config files.
    
    The online uscan test was also failing due to a change in the format of
    the version numbers used by the upstream authors, which is fixed by
    broadening the allowable version characters.
    
    Signed-off-by: James McCoy <jamessan at debian.org>
---
 debian/changelog       |    3 +++
 test/test_debchange    |    4 ++--
 test/test_debdiff      |    4 ++--
 test/test_licensecheck |    4 ++--
 test/test_uscan        |    2 +-
 test/test_uscan_online |    4 ++--
 6 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index b2c1faa..90eec65 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -7,6 +7,9 @@ devscripts (2.13.5) UNRELEASED; urgency=low
     automatically repack upstream archives which contain non-DFSG content.
     Thanks to Andreas Tille, gregor herrmann, and Rafael Laboissiere for the
     patches.  (Closes: #685787)
+  * test/*:
+    + Pass --no-conf to commands which may be influenced by ~/.devscripts
+    + Fix a test failure in test_uscan_online due to different version format.
 
   [ Evgeni Golov ]
   * debcheckout: allow setting the user for auth mode in the config.  (Closes:
diff --git a/test/test_debchange b/test/test_debchange
index d7d7095..041c0ae 100755
--- a/test/test_debchange
+++ b/test/test_debchange
@@ -15,10 +15,10 @@
 # OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 if test "$1" = --installed; then
-    COMMAND=dch
+    COMMAND="dch --no-conf"
     shift
 else
-    COMMAND="perl -I ${0%/*}/.. ${0%/*}/../scripts/debchange.pl"
+    COMMAND="perl -I ${0%/*}/.. ${0%/*}/../scripts/debchange.pl --no-conf"
 fi
 
 . "${0%/*}/shunit2-helper-functions.sh"
diff --git a/test/test_debdiff b/test/test_debdiff
index 926d218..2fb0cfe 100755
--- a/test/test_debdiff
+++ b/test/test_debdiff
@@ -16,10 +16,10 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 
 if test "$1" = --installed; then
-    COMMAND=debdiff
+    COMMAND="debdiff --no-conf"
     shift
 else
-    COMMAND="perl -I ${0%/*}/.. ${0%/*}/../scripts/debdiff.pl"
+    COMMAND="perl -I ${0%/*}/.. ${0%/*}/../scripts/debdiff.pl --no-conf"
 fi
 
 WORKDIR="$(readlink -f "${0%/*}")"
diff --git a/test/test_licensecheck b/test/test_licensecheck
index f057467..ac22e84 100755
--- a/test/test_licensecheck
+++ b/test/test_licensecheck
@@ -15,10 +15,10 @@
 # OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 if test "$1" = --installed; then
-    COMMAND=licensecheck
+    COMMAND="licensecheck --no-conf"
     shift
 else
-    COMMAND="../scripts/licensecheck.pl"
+    COMMAND="../scripts/licensecheck.pl --no-conf"
 fi
 
 WORKDIR="$(readlink -f "${0%/*}")"
diff --git a/test/test_uscan b/test/test_uscan
index fd213bc..e836f37 100755
--- a/test/test_uscan
+++ b/test/test_uscan
@@ -15,7 +15,7 @@
 # On Debian systems, the complete text of the GNU General Public License
 # version 3 can be found in the /usr/share/common-licenses/GPL-3 file.
 
-COMMAND="$(pwd)/../scripts/uscan.pl"
+COMMAND="$(pwd)/../scripts/uscan.pl --no-conf"
 
 cleanup(){
     kill -9 $(cat $TMPDIR/repo/pid)
diff --git a/test/test_uscan_online b/test/test_uscan_online
index 47da566..18732e1 100755
--- a/test/test_uscan_online
+++ b/test/test_uscan_online
@@ -14,7 +14,7 @@
 # ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 # OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
-COMMAND="${0%/*}/../scripts/uscan.pl"
+COMMAND="${0%/*}/../scripts/uscan.pl --no-conf"
 
 . "${0%/*}/shunit2-helper-functions.sh"
 
@@ -25,7 +25,7 @@ found() {
 testS3Bucket() {
     local latest=$(curl -s http://s3.amazonaws.com/rds-downloads | \
                    sed 's@\(</[A-Za-z]*>\)@\1\n at g' | grep '<Key>RDSCli-' | \
-                   sed 's at .*<Key>RDSCli-\([0-9.]*\)\.zip</Key>.*@\1 at g' | \
+                   sed 's at .*<Key>RDSCli-\([-0-9.]*\)\.zip</Key>.*@\1 at g' | \
                    sort -n | tail -n 1)
     local result="Processing watchfile line for package rdscli...
 Newest version on remote site is ${latest}, local version is 1.4.007

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/collab-maint/devscripts.git



More information about the devscripts-devel mailing list