Author: jkeenan
Date: Sat Feb 16 10:48:31 2008
New Revision: 25770

Modified:
   branches/tcif/lib/Parrot/Configure/Parallel/Trace.pm
   branches/tcif/t/configure/062-parallel.t
   branches/tcif/t/configure/063-parallel.t
   branches/tcif/t/configure/064-parallel.t

Log:
Simplify name of subroutine from 'store_this_step_pure' to 'store_this_step'.

Modified: branches/tcif/lib/Parrot/Configure/Parallel/Trace.pm
==============================================================================
--- branches/tcif/lib/Parrot/Configure/Parallel/Trace.pm        (original)
+++ branches/tcif/lib/Parrot/Configure/Parallel/Trace.pm        Sat Feb 16 
10:48:31 2008
@@ -32,7 +32,7 @@
         conf        => $conf,
     } );
 
-    $trace->store_this_step_pure();
+    $trace->store_this_step();
 
 Used only in configuration step tests found in F<t/steps/>.
 
@@ -331,7 +331,7 @@
     return 1;
 }
 
-=head2 C<store_this_step_pure()>
+=head2 C<store_this_step()>
 
 B<Purpose:>  As needed:  Retrieves the configuration state from the
 Storable file on disk.  Constructs a Parrot::Configure::Parallel object,
@@ -356,7 +356,7 @@
 
 =cut 
 
-sub store_this_step_pure {
+sub store_this_step {
     my $self = shift;
     my $pkg = $self->{step};
     my $step_position = $self->get_step_position($pkg);

Modified: branches/tcif/t/configure/062-parallel.t
==============================================================================
--- branches/tcif/t/configure/062-parallel.t    (original)
+++ branches/tcif/t/configure/062-parallel.t    Sat Feb 16 10:48:31 2008
@@ -31,8 +31,8 @@
             't/steps/init_defaults-01.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 1,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 1,
+        "Got expected return value from store_this_step()");
 
     my $sto = $trace->get_storable_file();
     ok( -e $sto,
@@ -41,8 +41,8 @@
             't/steps/init_defaults-02.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 2,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 2,
+        "Got expected return value from store_this_step()");
 
     ok( (chdir $cwd), "Able to change back to starting directory");
 }

Modified: branches/tcif/t/configure/063-parallel.t
==============================================================================
--- branches/tcif/t/configure/063-parallel.t    (original)
+++ branches/tcif/t/configure/063-parallel.t    Sat Feb 16 10:48:31 2008
@@ -32,8 +32,8 @@
             't/steps/init_defaults-01.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 1,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 1,
+        "Got expected return value from store_this_step()");
 
     my $sto = $trace->get_storable_file();
     ok( -e $sto,
@@ -42,8 +42,8 @@
             't/steps/init_install-01.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 1,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 1,
+        "Got expected return value from store_this_step()");
 
     ok( (chdir $cwd), "Able to change back to starting directory");
 }

Modified: branches/tcif/t/configure/064-parallel.t
==============================================================================
--- branches/tcif/t/configure/064-parallel.t    (original)
+++ branches/tcif/t/configure/064-parallel.t    Sat Feb 16 10:48:31 2008
@@ -32,8 +32,8 @@
             't/steps/init_defaults-01.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 1,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 1,
+        "Got expected return value from store_this_step()");
 
     my $state;
     $state = $trace->get_previous_state();
@@ -44,8 +44,8 @@
             't/steps/init_install-01.t'
     );
     ok(defined $trace, "Constructor returned defined value");
-    is($trace->store_this_step_pure(), 1,
-        "Got expected return value from store_this_step_pure()");
+    is($trace->store_this_step(), 1,
+        "Got expected return value from store_this_step()");
     $state = $trace->get_previous_state();
     ok(defined $state,
         "There was a previous config step, hence state is now defined");

Reply via email to