Skip to content

Commit

Permalink
fixed more portability issues in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
kraih committed Oct 2, 2011
1 parent e06d667 commit 2894096
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Changes
Expand Up @@ -7,7 +7,7 @@ This file documents the revision history for Perl extension Mojolicious.
- Improved message parser performance slightly.
- Improved resolver tests.
- Fixed small formatting bug in Mojo::Headers.
- Fixed Windows and FreeBSD portability issues in tests.
- Fixed FreeBSD and Perl 5.12.3 portability issues in tests.

1.99 2011-09-29 00:00:00
- Deprecated direct hash access to the flash in
Expand Down
5 changes: 2 additions & 3 deletions t/mojo/websocket.t
Expand Up @@ -7,12 +7,11 @@ BEGIN {
$ENV{MOJO_IOWATCHER} = 'Mojo::IOWatcher';
}

# FreeBSD 8.0 and 8.1 are known to cause problems
# FreeBSD 8.0/8.1 and Perl 5.12.3 are known to cause problems
use Test::More;
plan skip_all => 'This test does not work on some older versions of FreeBSD!'
if $^O =~ /freebsd/;
plan skip_all => 'Perl 5.14 required for this test on Windows!'
if ($^O eq 'MSWin32' || $^O =~ /cygwin/) && $] < 5.014;
plan skip_all => 'Perl 5.14 required for this test!' if $] < 5.014;
plan tests => 42;

# "I can't believe it! Reading and writing actually paid off!"
Expand Down
5 changes: 2 additions & 3 deletions t/mojolicious/websocket_lite_app.t
Expand Up @@ -11,12 +11,11 @@ BEGIN {
$ENV{MOJO_IOWATCHER} = 'Mojo::IOWatcher';
}

# FreeBSD 8.0 and 8.1 are known to cause problems
# FreeBSD 8.0/8.1 and Perl 5.12.3 are known to cause problems
use Test::More;
plan skip_all => 'This test does not work on some older versions of FreeBSD!'
if $^O =~ /freebsd/;
plan skip_all => 'Perl 5.14 required for this test on Windows!'
if ($^O eq 'MSWin32' || $^O =~ /cygwin/) && $] < 5.014;
plan skip_all => 'Perl 5.14 required for this test!' if $] < 5.014;
plan tests => 78;

# "Your mistletoe is no match for my *tow* missile."
Expand Down

0 comments on commit 2894096

Please sign in to comment.