Changed logic, where the after_test was not executed if before_test has failed.

jgebal · jgebal · commit 84d6dd425030 · 2017-02-22T21:35:27.000Z
Now the after_test gets executed regardless of before_test status
diff --git a/source/core/types/ut_test.tpb b/source/core/types/ut_test.tpb
@@ -65,10 +65,10 @@ create or replace type body ut_test as
         if l_completed_without_errors then
           -- execute the test
           self.item.do_execute(self, a_listener);
-          
-          -- perform cleanup regardless of the test failure
-          self.after_test.do_execute(self, a_listener);
+
         end if;
+        -- perform cleanup regardless of the test or setup failure
+        self.after_test.do_execute(self, a_listener);
 
         self.rollback_to_savepoint(l_savepoint);
 
@@ -87,7 +87,7 @@ create or replace type body ut_test as
     self.results := ut_assert_processor.get_asserts_results();
     self.results_count := ut_results_counter(self.result);
   end;
-  
+
   overriding member procedure fail(self in out nocopy ut_test, a_listener in out nocopy ut_event_listener_base, a_failure_msg varchar2) is
   begin
     ut_utils.debug_log('ut_test.fail');
@@ -98,7 +98,7 @@ create or replace type body ut_test as
     self.end_time := self.start_time;
     a_listener.fire_after_event(ut_utils.gc_test, self);
   end;
- 
+
 
 end;
 /
