Changeset 8587

Timestamp:

08/19/10 18:55:12 (18 months ago)

Author:

MichaelTempest

Message:

Item2286: It is better to read from the session object *before* calling finish(), so that the test still works if finish actually does clear out all of the fields (including this field, which is only used for testing).

File:

• trunk/UnitTestContrib/test/unit/RegisterTests.pm (modified) (1 diff)

trunk/UnitTestContrib/test/unit/RegisterTests.pm

@@ -923 +923 @@
         $this->assert( 0, "expected an oops redirect" );
     };
+    # Read the verification code before finish()'ing the session
+    my $debugVerificationCode = $this->{session}->{DebugVerificationCode};
     $this->{session}->finish();
 
     # For verification process everything including finish(), so don't just
     # call verifyEmails
-    my $code = shift || $this->{session}->{DebugVerificationCode};
+    my $code = shift || $debugVerificationCode;
     $query = new Unit::Request(
         {