From 01e525106b076a4084bee26758e02b11381495da Mon Sep 17 00:00:00 2001 From: Axel Beckert Date: Tue, 5 Mar 2013 23:07:04 +0100 Subject: [PATCH] Use Test::NoTabs in t/no-tabs.t --- debian/changelog | 1 + debian/control | 1 + t/no-tabs.t | 47 +++++++---------------------------------------- 3 files changed, 9 insertions(+), 40 deletions(-) diff --git a/debian/changelog b/debian/changelog index bb7db7b..f85df23 100644 --- a/debian/changelog +++ b/debian/changelog @@ -38,6 +38,7 @@ xen-tools (4.4~dev-1) UNRELEASED; urgency=low - Fix filesystem tools installation in 91-install-fs-tools (which was broken since 4.3~rc1-1) by merging 91-install-fs-tools back into 90-make-fstab. Also supports RPM-based distributions now. + - Uses Test::NoTabs; add according build-dependency. * Add debian/gbp.conf to be able to to build xen-tools with git-buildpackage. * Clean up debian/rules: diff --git a/debian/control b/debian/control index 7c2f1ce..8b8e9a1 100644 --- a/debian/control +++ b/debian/control @@ -8,6 +8,7 @@ Build-Depends: debhelper (>= 7.0.0), libfile-slurp-perl, libfile-which-perl, libmoose-perl, + libtest-notabs-perl, libtest-pod-coverage-perl, libtest-pod-perl, libtext-template-perl diff --git a/t/no-tabs.t b/t/no-tabs.t index 4511e35..d5efbe4 100755 --- a/t/no-tabs.t +++ b/t/no-tabs.t @@ -9,8 +9,10 @@ use strict; use File::Find; -use Test::More; +use Test::NoTabs; +# Check all Perl files +all_perl_files_ok(); # # Find all the files beneath the current directory, @@ -18,8 +20,6 @@ use Test::More; # find( { wanted => \&checkFile, no_chdir => 1 }, '.' ); -done_testing(); - # # Check a file. # @@ -62,50 +62,17 @@ sub checkFile ( $line =~ /\/bin\/bash/ ) ) { $isShell = 1; + last; } if ( $line =~ /\/usr\/bin\/perl/ ) { - $isPerl = 1; + last; } } close( INPUT ); # - # Return if it wasn't a perl file. + # Run check if it is a shell file. # - if ( $isShell || $isPerl ) - { - # - # Count TAB characters - # - my $count = countTabCharacters( $file ); - - is( $count, 0, "Script has no tab characters: $file" ); - } -} - - -=head2 countTabCharacters - -=cut - -sub countTabCharacters -{ - my ( $file ) = (@_); - - my $count = 0; - - open( FILE, "<", $file ) - or die "Cannot open $file - $!"; - foreach my $line ( ) - { - while( $line =~ /(.*)\t(.*)/ ) - { - $count += 1; - $line = $1 . $2; - } - } - close( FILE ); - - return( $count ); + notabs_ok( $file ) if $isShell; }