Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sometime failed t/mojolicious/websocket_lite_app.t test on 078fc07b550344bfe35d(v1.22) Windows7 #127

Closed
yuki-kimoto opened this issue Apr 28, 2011 · 5 comments

Comments

@yuki-kimoto
Copy link
Contributor

The following test is sometime failed.

t/mojolicious/websocket_lite_app.t ............ 1/41
#   Failed test 'right result'
#   at t/mojolicious/websocket_lite_app.t line 475.
#          got: undef
#     expected: 'hihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihi
ihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihi
ihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihi
ihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihi
ihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihihi


# WebSocket /echo (64bit length)
$result = undef;
$ua->websocket(
  '/echo' => sub {
    my $tx = pop;
    $tx->on_finish(sub { $loop->stop });
    $tx->on_message(
      sub {
        my ($tx, $message) = @_;
        $result = $message;
        $tx->finish;
      }
    );
    $tx->send_message('hi' x 100000);
  }
);
$loop->start;
is $result, 'hi' x 100000, 'right result';

This test is sometime failed( about 1 / 10 )

On windows7, "16 bit lenght" test is always success, but "64bit length" test is sometime failed.

On Linux(CentOS), If chunk size is changed (100000 -> 200000), the test is failed many times.

I think if the byte size is more than some size, this test is failed many times.

@kraih
Copy link
Member

kraih commented Apr 28, 2011

I'm aware of this, but i'm not sure if it can be fixed.

@kraih kraih closed this as completed Apr 28, 2011
@kraih
Copy link
Member

kraih commented Apr 28, 2011

Actually it might be fixable, i just can't replicate the failing test result, so someone else would have to fix it.

@kraih kraih reopened this Apr 28, 2011
@kraih
Copy link
Member

kraih commented Apr 30, 2011

There is a small chance this might be fixed now. f7901c4
I still can't replicate the failing test though, so no confirmation yet.

@kraih
Copy link
Member

kraih commented May 3, 2011

And fixed. :) 301294b

@kraih kraih closed this as completed May 3, 2011
@yuki-kimoto
Copy link
Contributor Author

thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants