Attachment 97825 Details for Bug 24494 – Bug 24494: tests: 00-valid-xml.t shouldn't check node_modules

[patch] Bug 24494: tests: 00-valid-xml.t shouldn't check node_modules

Bug-24494-tests-00-valid-xmlt-shouldnt-check-nodem.patch (text/plain), 1.15 KB, created by Victor Grousset/tuxayo on 2020-01-23 15:09:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Victor Grousset/tuxayo

Created: 2020-01-23 15:09:43 UTC

Size: 1.15 KB

patch

obsolete

>From 675abada5e9dff0b9a6defeba068a860f1a40225 Mon Sep 17 00:00:00 2001
>From: Victor Grousset/tuxayo <victor@tuxayo.net>
>Date: Thu, 23 Jan 2020 15:55:18 +0100
>Subject: [PATCH] Bug 24494: tests: 00-valid-xml.t shouldn't check node_modules
>
>== Test plan ==
>- work with SCSS (just recompile it if you never did)
>  - https://wiki.koha-community.org/wiki/Working_with_staff_client_SCSS
>- now you have a node_modules directory
>- prove -r t/00-valid-xml.t
>- it should fail on files in node_modules
>- apply this patch
>- prove -r t/00-valid-xml.t
>- it should work
>- sabotage a legitimate XML file, like etc/SIPconfig.xml
>- prove -r t/00-valid-xml.t
>- it should fail, good the test is still useful
>- undo the sabotage
>---
> t/00-valid-xml.t | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/t/00-valid-xml.t b/t/00-valid-xml.t
>index 2c5393abd7..d19b4e68d1 100644
>--- a/t/00-valid-xml.t
>+++ b/t/00-valid-xml.t
>@@ -30,6 +30,7 @@ find({
>     no_chdir => 1,
>     wanted => sub {
>         my $file = $_;
>+        return if $file =~ /node_modules/i;
>         return unless $file =~ /(\.xml|\.xsl|\.xslt|\.xsd)$/i;
>         my $dom;
>         eval { $dom = $parser->parse_file($file); };
>-- 
>2.25.0

Actions: View | Diff | Splinter Review

Attachments on bug 24494: 97825 | 98861 | 98862 | 99221 | 99222