Bug 22836: (follow-up) Bug 22478: (QA follow-up) Make test consistent with variable...
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 3 May 2019 01:04:54 +0000 (21:04 -0400)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 30 May 2019 07:05:07 +0000 (08:05 +0100)
One occurrence was missing.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 7799f4ddd1f1fe111081dc3695c9038fa500e961)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

t/db_dependent/selenium/regressions.t

index e8d355f..64ea406 100644 (file)
@@ -226,7 +226,7 @@ subtest 'XSS vulnerabilities in pagination' => sub {
     is( $alert_text, undef, 'No alert box displayed, even if evil intent' );
 
     my $second_page = $driver->find_element('//div[@class="pages"]/span[@class="currentPage"]/following-sibling::a');
-    like( $second_page->get_attribute('href'), qr{category=1%22%3E%3Cscript%3Ealert%28%27booh%21%27%29%3C%2Fscript%3E}, 'The second patch should displayed the variables and attributes correctly URI escaped' );
+    like( $second_page->get_attribute('href'), qr{category=2%22%3E%3Cscript%3Ealert%28%27booh%21%27%29%3C%2Fscript%3E}, 'The second patch should displayed the variables and attributes correctly URI escaped' );
 
     push @cleanup, $patron, $patron->category, $patron->library;
 };