@@ -, +, @@ --- .../prog/js/modals/add_catalog_concern.js | 1 + .../intranet-tmpl/prog/js/modals/display_ticket.js | 1 + .../intranet-tmpl/prog/js/modals/place_booking.js | 1 + xt/js_tidy.t | 11 ++++++----- 4 files changed, 9 insertions(+), 5 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/js/modals/add_catalog_concern.js +++ a/koha-tmpl/intranet-tmpl/prog/js/modals/add_catalog_concern.js @@ -1,3 +1,4 @@ +/* keep tidy */ $(document).ready(function() { // Pre-populate empty message with template --- a/koha-tmpl/intranet-tmpl/prog/js/modals/display_ticket.js +++ a/koha-tmpl/intranet-tmpl/prog/js/modals/display_ticket.js @@ -1,3 +1,4 @@ +/* keep tidy */ $(document).ready(function() { $('#ticketDetailsModal').on('show.bs.modal', function(event) { let modal = $(this); --- a/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js +++ a/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js @@ -1,3 +1,4 @@ +/* keep tidy */ let dataFetched = false; let bookable_items, bookings, --- a/xt/js_tidy.t +++ a/xt/js_tidy.t @@ -1,4 +1,4 @@ -#!/usr/bin/perl +#/usr/bin/perl # This file is part of Koha. # @@ -22,17 +22,18 @@ use FindBin(); use Data::Dumper qw( Dumper ); use Test::More tests => 1; -my @js_files; -push @js_files, `git ls-files 'koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js'`; +my $cmd = q{git grep -l '/\* keep tidy \*/' -- '*.js'}; +my @js_files = qx{$cmd}; my @not_tidy; foreach my $filepath (@js_files) { chomp $filepath; - my $tidy = qx{yarn --silent run prettier --trailing-comma es5 --arrow-parens avoid $filepath}; + my $tidy = qx{yarn --silent run prettier --trailing-comma es5 --arrow-parens avoid $filepath}; my $content = read_file $filepath; if ( $content ne $tidy ) { push @not_tidy, $filepath; } } -is(scalar(@not_tidy), 0, 'No .js file should be messy') or diag Dumper \@not_tidy; +is( scalar(@not_tidy), 0, sprintf( 'No .js file should be messy %s/%s', scalar(@not_tidy), scalar(@js_files) ) ) + or diag Dumper \@not_tidy; --