Add sorting to GridFieldDetailFormTest fixtures

Broke SQLite3 tests in some cases due to random ordering of results.
silverstripe · Jul 6, 2012 · bc1c7f7 · bc1c7f7
1 parent 6466993
commit bc1c7f7
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/tests/forms/gridfield/GridFieldDetailFormTest.php b/tests/forms/gridfield/GridFieldDetailFormTest.php
@@ -63,8 +63,8 @@ public function testViewForm() {
 		$surname   = $parser->getBySelector('#Form_ItemEditForm_Surname');
 
 		$this->assertFalse($response->isError());
-		$this->assertEquals('Joe', (string) $firstName[0]);
-		$this->assertEquals('Bloggs', (string) $surname[0]);
+		$this->assertEquals('Jane', (string) $firstName[0]);
+		$this->assertEquals('Doe', (string) $surname[0]);
 	}
 
 	function testEditForm() {
@@ -188,6 +188,8 @@ class GridFieldDetailFormTest_Person extends DataObject implements TestOnly {
 		'Categories' => 'GridFieldDetailFormTest_Category'
 	);
 
+	static $default_sort = 'FirstName';
+
 	function getCMSFields() {
 		$fields = parent::getCMSFields();
 		// TODO No longer necessary once FormScaffolder uses GridField
@@ -209,7 +211,9 @@ class GridFieldDetailFormTest_PeopleGroup extends DataObject implements TestOnly
 	static $has_many = array(
 		'People' => 'GridFieldDetailFormTest_Person'
 	);
-
+
+	static $default_sort = 'Name';
+
 	function getCMSFields() {
 		$fields = parent::getCMSFields();
 		// TODO No longer necessary once FormScaffolder uses GridField
@@ -232,6 +236,8 @@ class GridFieldDetailFormTest_Category extends DataObject implements TestOnly {
 		'People' => 'GridFieldDetailFormTest_Person'
 	);
 
+	static $default_sort = 'Name';
+
 	function getCMSFields() {
 		$fields = parent::getCMSFields();
 		// TODO No longer necessary once FormScaffolder uses GridField