Test for cookie expiration and extension behavior, add cookie_expires, disable condit...

[catagits/Catalyst-Plugin-Session-State-Cookie.git] / t / basic.t

diff --git a/t/basic.t b/t/basic.t
index aa78b68..526aeba 100644 (file)
--- a/t/basic.t
+++ b/t/basic.t
@@ -24,20 +24,20 @@ $res->set_always( cookies => \%res_cookies );
 my $cxt =
   Test::MockObject::Extends->new("Catalyst::Plugin::Session::State::Cookie");
 
-$cxt->set_always( config => { } );
+$cxt->set_always( config   => {} );
 $cxt->set_always( request  => $req );
 $cxt->set_always( response => $res );
+$cxt->set_always( session  => { __expires => 123 } );
 $cxt->set_false("debug");
 my $sessionid;
 $cxt->mock( sessionid => sub { shift; $sessionid = shift if @_; $sessionid } );
 
-
 can_ok( $m, "setup_session" );
 
 $cxt->setup_session;
 
-is( $cxt->config->{session}{cookie_name}, "session", "default cookie name is set" );
-
+is( $cxt->config->{session}{cookie_name},
+    "session", "default cookie name is set" );
 
 can_ok( $m, "prepare_cookies" );
 
@@ -56,16 +56,18 @@ is( $cxt->sessionid, "the session id", "session ID was restored from cookie" );
 $cxt->clear;
 $res->clear;
 
-can_ok( $m, "finalize" );
-$cxt->finalize;
+can_ok( $m, "finalize_cookies" );
+$cxt->finalize_cookies;
+{ local $TODO = "This check is a pain to write, should be done by catalyst itself";
 ok( !$res->called("cookies"),
     "response cookie was not set since res cookie is already there" );
+}
 
 $cxt->clear;
 $sessionid = undef;
 $res->clear;
 
-$cxt->finalize;
+$cxt->finalize_cookies;
 ok( !$res->called("cookies"),
 "response cookie was not set when sessionid was deleted, even if req cookie is still there"
 );
@@ -74,11 +76,11 @@ $sessionid = "some other ID";
 $cxt->clear;
 $res->clear;
 
-$cxt->finalize;
+$cxt->finalize_cookies;
 $res->called_ok( "cookies", "response cookie was set when sessionid changed" );
 is_deeply(
     \%res_cookies,
-    { session => { value => $sessionid } },
+    { session => { value => $sessionid, expires => 123 } },
     "cookie was set correctly"
 );
 
@@ -88,7 +90,7 @@ $res->clear;
 %res_cookies = ();
 $sessionid   = undef;
 
-$cxt->finalize;
+$cxt->finalize_cookies;
 ok( !$res->called("cookies"),
     "response cookie was not set when there is no sessionid or request cookie"
 );
@@ -98,13 +100,13 @@ $sessionid   = "123";
 %res_cookies = ();
 $res->clear;
 
-$cxt->finalize;
+$cxt->finalize_cookies;
 
 $res->called_ok( "cookies",
     "response cookie was set when session was created" );
 is_deeply(
     \%res_cookies,
-    { session => { value => $sessionid } },
+    { session => { value => $sessionid, expires => 123 } },
     "cookie was set correctly"
 );