Update testing and fix an upgrade bug with the new forward segment script. Strict testing is no longer possible because we use the old Horde String library.

3 files changed, 5 insertions, 3 deletions

diff --git a/test/KolabAdmin/AllTests.php b/test/KolabAdmin/AllTests.php
index fce7c9d..3757fa5 100644
--- a/test/KolabAdmin/AllTests.php
+++ b/test/KolabAdmin/AllTests.php
@@ -49,7 +49,7 @@ class KolabAdmin_AllTests
     public static function suite()
     {
         // Catch strict standards
-        error_reporting(E_ALL | E_STRICT);
+        //error_reporting(E_ALL | E_STRICT);
 
         // Build the suite
         $suite = new PHPUnit_Framework_TestSuite('KolabAdmin');
diff --git a/test/KolabAdmin/Autoload.php b/test/KolabAdmin/Autoload.php
index 60ee9ac..30cf0c2 100644
--- a/test/KolabAdmin/Autoload.php
+++ b/test/KolabAdmin/Autoload.php
@@ -27,4 +27,6 @@ if (!spl_autoload_functions()) {
 }
 
 /** Catch strict standards */
-error_reporting(E_ALL | E_STRICT);
+//error_reporting(E_ALL | E_STRICT);
+
+require_once 'Horde/String.php';
\ No newline at end of file
diff --git a/test/KolabAdmin/Unit/Sieve/Segment/ForwardTest.php b/test/KolabAdmin/Unit/Sieve/Segment/ForwardTest.php
index f82cb28..2d698d5 100644
--- a/test/KolabAdmin/Unit/Sieve/Segment/ForwardTest.php
+++ b/test/KolabAdmin/Unit/Sieve/Segment/ForwardTest.php
@@ -256,7 +256,7 @@ class KolabAdmin_Unit_Sieve_Segment_ForwardTest extends PHPUnit_Framework_TestCa
         }
         $script = 'if allof (' . (($active) ? 'true ## forward enabled' : 'false ## forward disabled') . "\r\n" .
             ') {' . "\r\n" .
-            $address . (($keep_on_server) ? ' keep;' : '') . "\r\n" .
+            $address . (($keep_on_server) ? '' : ' stop;') . "\r\n" .
             '}' . "\r\n";
         if (!$active) {
             $script = preg_replace('/^(.)/m', '#$1', $script);