Skip to content

Commit

Permalink
Item14237: Tests are now passing completely.
Browse files Browse the repository at this point in the history
No expected failures needed.
  • Loading branch information
vrurg committed May 12, 2018
1 parent 1a21f75 commit 83f7976
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
5 changes: 3 additions & 2 deletions UnitTestContrib/test/unit/AccessControlTests.pm
Original file line number Diff line number Diff line change
Expand Up @@ -918,8 +918,9 @@ THIS
# ShortURLs may involve a {ScriptUrlPaths}{view} of '' or something
# like '/foswiki' (where {ScriptUrlPath} looks like '/foswiki/bin').
# In any case, the test is hard-wired to ignore {ScriptSuffix}
$this->expect_failure( 'Test does\'t cater to ShortURL configurations',
using => 'ShortURLs' );
# XXX Test passes ok on v3.0
#$this->expect_failure( 'Test does\'t cater to ShortURL configurations',
# using => 'ShortURLs' );

if ( $this->check_dependency('Foswiki,<,1.2') ) {

Expand Down
5 changes: 3 additions & 2 deletions UnitTestContrib/test/unit/AdminOnlyAccessControlTests.pm
Original file line number Diff line number Diff line change
Expand Up @@ -778,8 +778,9 @@ THIS
# ShortURLs may involve a {ScriptUrlPaths}{view} of '' or something
# like '/foswiki' (where {ScriptUrlPath} looks like '/foswiki/bin').
# In any case, the test is hard-wired to ignore {ScriptSuffix}
$this->expect_failure( 'Test does\'t cater to ShortURL configurations',
using => 'ShortURLs' );
# XXX Test passes ok on v3.0
#$this->expect_failure( 'Test does\'t cater to ShortURL configurations',
# using => 'ShortURLs' );

# Check we got a 401
my ($status) = $text =~ m/^Status: (\d+)\r?$/m;
Expand Down

0 comments on commit 83f7976

Please sign in to comment.