MINOR Using Config system instead of Director::addRules() in tests

silverstripe · May 22, 2012 · f446547 · f446547
1 parent ae1a106
commit f446547
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/tests/control/DirectorTest.php b/tests/control/DirectorTest.php
@@ -17,7 +17,7 @@ function setUp() {
 			self::$originalRequestURI = $_SERVER['REQUEST_URI'];
 		}
 
-		Director::addRules(99, array(
+		Config::inst()->update('Director', 'rules', array(
 			'DirectorTestRule/$Action/$ID/$OtherID' => 'DirectorTestRequest_Controller'
 		));
 	}

diff --git a/tests/control/RequestHandlingTest.php b/tests/control/RequestHandlingTest.php
@@ -250,7 +250,7 @@ function testActionHandlingOnField() {
 /**
  * Director rules for the test
  */
-Director::addRules(50, array(
+Config::inst()->update('Director', 'rules', array(
 	// If we don't request any variables, then the whole URL will get shifted off.  This is fine, but it means that the
 	// controller will have to parse the Action from the URL itself.
 	'testGoodBase1' => "RequestHandlingTest_Controller",

diff --git a/tests/forms/FormTest.php b/tests/forms/FormTest.php
@@ -508,6 +508,7 @@ function getViewer($action = null) {
 	}
 }
 
-Director::addRules(50, array(
-	'FormTest_Controller' => "FormTest_Controller",
+Config::inst()->update('Director', 'rules', array(
+	'FormTest_Controller' => 'FormTest_Controller'
 ));
+