Display the file list in alphabetic order

Bug 16003: Add exception pattern list for codespell checks

The 'isnt' Test::More subroutine should not be considered as a failure
by the codespell check.
To avoid that this patch introduces the concept of "exception patterns"
for this check.

NOTE: This is better than introducing a '-x {exception file}', because
      it dynamically handles the isnt( issue. If other similar function
      name issues arise in the future, modifications to the
      i_dont_fail_spelling.pl file and the exception list can be made.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Revert "The html filter is not needed anymore (see bug 13618)"

This reverts commit a9ddabb936a0a1539d01992908b3e6484b6d6466.

Bug 15753: String update in the README for qa-test-tools

Trivial string patch. Adds "sudo" to "$ apt-get install codespell" on
line number 25 of the README file.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

The html filter is not needed anymore (see bug 13618)

(follow-up) type subroutine should be _type

Koha::Borrower has been moved by bug 15548

Koha::Branches has been removed by bug 15294

type subroutine should be _type

See bug 15446

Add ProhibitUnusedVariables to perlcritic rules

See bug 15258.

(follow-up) Check if opac.css and opac.less are in sync

Check if opac.css and opac.less are in sync

If opac.css and/or opac.less is changed, they must have been kept in
sync.

To sync them:
 $ cd koha-tmpl/opac-tmpl
 $ lessc --compress bootstrap/less/opac.less > bootstrap/css/opac.css

This test checks if
 lessc --compress bootstrap/less/opac.less > /tmp/file.css
 diff bootstrap/css/opac.css /tmp/file.css | wc -l
returns 0

pass attribute should be defined at File level

spelling: Don't die if codespell is not installed

codespell is not available on wheezy, the script should continue if the
bin does not exist.

The tests will be executed, but the report will mark it as skipped.

spelling follow-up: Add the test file

Add a spelling check for pl and tt files (using codespell)

You will need to install codespell to make it works.

updatedatabase.pl: "INSERT IGNORE INTO systempreferences" should be use

http://lists.koha-community.org/pipermail/koha-devel/2015-April/041310.html

Check the charset collate for kohastructure.sql

See bug 11944.

Make the tests pass

For an odd reason, the perlcriticrc was not found:

An error occured : Could not parse profile
"QohA/File/../../perlcriticrc": Failed to open file
'QohA/File/../../perlcriticrc' for reading: No such file or directory at
t/Perl.t line 210.

The prog theme for the OPAC is deprecated, now use bootstrap

See bug 12162 comment 27.

Fix unit tests - should not be based on hash keys order

template name should not contain .tmpl anymore

Add a progress bar

For big patches, the process can be very long (20 minutes).
Now you know how long you will wait :)

The --no-progress param removes this progress bar.

Remove smartmatch/given/when to work with Perl 5.18

NOTE: The change from ~~ to == on line 65 is experimental, and may cause
issues.

TT plugins have been merged (see bug 10626)

Datatables files are in an include file (see bug 10868)

wthdrawn is a typo (see bug 10550).

Exit if there are unstaged changes.

In order to avoid a lost of data, the qa tests cannot be launched if
changes are unstaged.

Specific test for sysprefs.sql

Since bug 10610 has been pushed, the installer/data/mysql/sysprefs.sql
file is sorted by syspref names.

The failure is not a blocker one. The QAer is called to propose a
followup in order to fix the order rather than to fail QA the patch.

/!\ The file was generated using the mysql order by clause. Mysql sorts
underscore last.

Koha is now under the GPLv3 license

Update README: PERL5LIB has to contain the qa-tools directory

Fix: the perlcriticrc path should be build from the package path.

On this way, the perlcriticrc file is used on launching tests and using
the koha-qa.pl script.

Fix unit tests: the koha-qa dir name should not be hardcoded

Add a bug in the todo list (commits with only deleted files)

fixing compile typo in t/data/4/perl/i_fail_patterns.pl

update POD and README

added perl-deps file

Followup Add new pattern for theme

The line containing the modified file name should not be checked.
Side effect: +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/[...] raises
an false alert.

Add new pattern for theme (see bug 2774)

just some small display tweaks...

Bug 10080 renames IndependantBranches to IndependentBranches

Followup Fix unit tests for i_fail_patterns.tt

Add missing file.

Fix display: if v>1 errors description should be below the filename

Fix unit tests for i_fail_patterns.tt

fixed tests, and made a little display format tweak

update README

add LICENSE file

always use Koha's ./t/perlcriticrc file

perlcritic check ~/.perlcritirc, not ~/perlcriticrc

FIX line number cannot be decrement if it is a deletion

Merge marker, tab and whitespace are forbidden for template files too

FIX false positive on perl critic

Sometimes, a commit is ok while it is false!

Dump of $full_buf QohA::File::Perl::check_critic for the same commit
successively executed:

$VAR1 = [
          '1..1
',
          '# Subroutines::ProhibitExplicitReturnUndef: Got 1 violation(s).  Expected no more than 0.
',
          '# Got a total of 1. Expected no more than 0.
',
          'not ok 1 - Test::Perl::Critic::Progressive
',
          '#   Failed test \'Test::Perl::Critic::Progressive\'
',
          '# Looks like you failed 1 test of 1.
'
        ];
 * C4/Reserves.pm                                                           OK
    pod                         OK
    forbidden patterns          OK
    valid                       OK
    critic                      OK

$VAR1 = [
          '1..1
',
          '# Subroutines::ProhibitExplicitReturnUndef: Got 1 violation(s).  Expected no more than 0.
',
          '# Too many Perl::Critic violations...
',
          '# Got a total of 1. Expected no more than 0.
',
          'not ok 1 - Test::Perl::Critic::Progressive
',
          '#   Failed test \'Test::Perl::Critic::Progressive\'
',
          '#   at -e line 3.
',
          '# Looks like you failed 1 test of 1.
'
        ];
 * C4/Reserves.pm                                                           FAIL
    pod                         OK
    forbidden patterns          OK
    valid                       OK
    critic                      FAIL
        # Subroutines::ProhibitExplicitReturnUndef: Got 1 violation(s).

A serialisation failed with the run command.

This patch removes all \n characters in order to remove the "Too many Perl::Critic violations"
string with the regexp $line =~ s/Expected no more than.*$//g;

Followup improved display output (FIX UT)

We don't want any simple-quote string in js code

Bug 8942 : http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8942

Remove the perlcritic error on require a module between quotes

improved display output

added 1 line, to skip 'Subroutine $FOO redefined' type warnings

Koha currently has too many of these warnings, causing false-positive fails with the koha-qa.pl testing

when we work out a way of fixing these warnings, we can revert this commit

Typo occured/occurred

add a new flag "new_file" to mark a file if it is a newly created

added ~/perlcriticrc file test

added .t files, for testing

added perlcriticrc file, updated readme

fixed perlcritic test

Build diff reports improvement

This patch test if the new patches introduce new errors even if an error
is dupplicated.
So in the case where a single error occurred and now it occurs twice, it
is a regression.

Refactoring: new File::add_to_report routine

new check for perl files: POD

Forbiden patterns: Add line number(experimental) + refactoring

UT: add unit tests for tmpl

FIX Tests order

FIX: If the error is empty, it's not really an error

e.g. if
tt_valid' => ['lines 656, 666, 687', '']
There were 3 errors for tt_valid before patches, but now there is
no error so the report must be return OK

FIX: we want the diff (between HEAD and HEAD~X) and not a detail for each commit

FIX datatype, report diff uses array and scalar

FIX: Perl valid, We can't know the line number. And forbidden pattern is only for new line (^\+)

Followup FIX the perl valid check

We want to remove the "syntax ok" line

FIX the perl valid check

If the syntax is OK, a patch could introduces some perl warnings. We
want to highlight these regressions.

[OOP] Add forbidden patterns (tabulation and widespace characters)

[OOP] FIX template checks on new files

[OOP] FIX perl_critic test We have to delete the .pc file in first

Add some POD

Add a --nocolor option to not display with color the status

By default, we display colors

Set the verbosity = 0 by default

Simplify the filter method

[OOP] Add yaml check (+ ut)

[OOP]Add unit test for perl files

[OOP] Simplify the Report->add method

[OOP] Add some pod (File and File::Perl)

[OOP] Add perl test file

(OOP] FIX perl critic on new files

[OOP] Prevent noexisting file for perl check_valid

[OOP] FIX Compatibility with perl 5.14 (push arrayref)

Introduces OOP in QohA

- Adds inheritance (File::XML, File::Perl, File::Template)
- Now each file is processed on the previous branch and on the current
  branch. Before this patch the idea was to change branch batween each
  tests.
  It decreases the execution time:
  before:
    koha-qa.pl -c 5 -v 2  4,95s user 0,34s system 98% cpu 5,392 total
  with this patch:
    koha-qa.pl -c 5 -v 2  7,99s user 0,77s system 155% cpu 5,641 total
- Introduces a forbidden patterns check (ie. IFNULL, <<<<<<<, etc.).
  This can increase the execution time. We want to think about this
  utilisation. Is it really useful ?

It is a wip and cannot be executed safely !
To FIX: the check_critic does not work on new file (Is it already the
case on master ?)
TODO: POD and unit tests !

now skips testing deleted files

added -d arg for smart-comments

updated readme

fix regex typo

a fix to ignore deleted or removed files from testing

fix var typo, and remove debug

- misc tweaks
- added a new xml_valid test
modified:   QohA/Errors.pm
modified:   QohA/FileFind.pm
modified:   QohA/Git.pm
modified:   QohA/Perl.pm
modified:   QohA/Template.pm
modified:   koha-qa.pl

add a pre-commit git hook, useful for Koha

add a todo: If a file is deleted, we don't want to launch tests on it