[Rt-commit] rt branch, 4.2/inline-server-failed-tests, repushed

? sunnavy sunnavy at bestpractical.com
Tue Jul 1 09:43:04 EDT 2014


The branch 4.2/inline-server-failed-tests was deleted and repushed:
       was f2534682a5400db1cf27b760819940b7a2be461b
       now 37ca0d2fae96332ab17c07bb2783769005080dc7

1:  44fbf0a = 1:  7b37672 require actual_server since mailgate is called there
2:  f253468 ! 2:  37ca0d2 missed warnings check, which causes tests to fail in inline server mode
    @@ -1,12 +1,18 @@
     Author: sunnavy <sunnavy at bestpractical.com>
     
    -    missed one warning check, which causes tests to fail in inline server mode
    -    
    -    the error is like:
    -    
    -    Unhandled warning: Couldn't auto-create user 'conflicting' when attempting WebRemoteUser: Email address in use
    -    DESTROY created new reference to dead object 'Plack::Middleware::Test::StashWarnings' during global destruction.
    -    DESTROY created new reference to dead object 'Plack::Middleware::Test::StashWarnings' during global destruction.
    +    missed warnings check, which causes tests to fail in inline server mode
    +
    +diff --git a/t/web/path-traversal.t b/t/web/path-traversal.t
    +--- a/t/web/path-traversal.t
    ++++ b/t/web/path-traversal.t
    +@@
    + # dotfiles, for example.
    + $agent->get("$baseurl/Ticket/Attachment/28/9/.bashrc");
    + is($agent->status, 200); # Even for a file not found, we return 200
    ++$agent->next_warning_like(qr/could not be loaded/, "couldn't loaded warning");
    + $agent->content_like(qr/Attachment \S+ could not be loaded/);
    + 
    + # do not reject these URLs, even though they contain /. outside the path
     
     diff --git a/t/web/remote_user.t b/t/web/remote_user.t
     --- a/t/web/remote_user.t



More information about the rt-commit mailing list