Moves abstract classes to helper folder for better orgnisation and au…

…toloading. [skip ci]
Delpfine · Oct 25, 2015 · 0e4ad25 · 0e4ad25
1 parent 897e5ba
commit 0e4ad25
Show file tree

Hide file tree

Showing 7 changed files with 108 additions and 90 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -1,5 +1,7 @@
 language: php
 
+sudo: false
+
 php:
   - 5.4
   - 5.5

diff --git a/tests/_helpers/AbstractRequiredTest.php b/tests/_helpers/AbstractRequiredTest.php
@@ -0,0 +1,102 @@
+<?php
+/**
+ * @package   Fuel\Validation
+ * @version   2.0
+ * @author    Fuel Development Team
+ * @license   MIT License
+ * @copyright 2010 - 2013 Fuel Development Team
+ * @link      http://fuelphp.com
+ */
+
+namespace Fuel\Validation\Rule;
+
+class AbstractRequiredTest extends AbstractRuleTest
+{
+
+	/**
+	 * {@inheritdocs}
+	 */
+	protected $message = 'The field is required and has not been specified.';
+
+	protected function _before()
+	{
+		$this->object = new Required;
+	}
+
+	/**
+	 * @dataProvider validateProvider
+	 */
+	public function testValidate()
+	{
+		list($value, $field, $expected, $data) = func_get_args();
+
+		$this->assertEquals(
+			$expected,
+			$this->object->validate($value, $field, $data)
+		);
+	}
+
+	/**
+	 * {@inheritdocs}
+	 */
+	public function validateProvider()
+	{
+		return array(
+			0 => array('[email protected]', null, false, null),
+			1 => array('', null, false, null),
+			2 => array(array(), null, false, null),
+			3 => array(null, null, false, null),
+			4 => array(false, null, false, null),
+
+			5 => array('test string 5', 'test', false,
+				array()
+			),
+
+			6 => array('test string 6', 'test', false,
+				array(
+					'foo' => 'bar',
+					'baz' => 'bat',
+				)
+			),
+
+			7 => array('test string 7', 'test', true,
+				array(
+					'foo' => 'bar',
+					'test' => 'value',
+					'baz' => 'bat',
+				)
+			),
+
+			8 => array('bla', 'test', true, null),
+			9 => array('', 'test', false, null),
+			10 => array('bla', null, false, array()),
+			11 => array('', null, false, array()),
+
+			12 => array(false, 'foo', true,
+				array(
+					'foo' => false,
+				)
+			),
+			13 => array(true, 'foo', true,
+				array(
+					'foo' => true,
+				)
+			),
+
+			14 => array(false, 'bar', true,
+				array(
+					'bar' => false,
+				)
+			),
+			15 => array(0, 'test', true,
+				array(
+					'foo' => 'bar',
+					'test' => 0,
+					'baz' => 'bat',
+				)
+			),
+			16 => array(0, 'test', true, array('test' => 0)),
+		);
+	}
+
+}
diff --git a/tests/unit/Rule/AbstractRuleTest.php → tests/_helpers/AbstractRuleTest.php b/tests/unit/Rule/AbstractRuleTest.php → tests/_helpers/AbstractRuleTest.php
diff --git a/tests/_helpers/_generated/CodeGuyActions.php b/tests/_helpers/_generated/CodeGuyActions.php
@@ -1,4 +1,4 @@
-<?php  //[STAMP] 8cfe98b3e6ddaa718333872e2edd6e01
+<?php  //[STAMP] a8cf3c7461a512a056a510cd9f14df23
 namespace _generated;
 
 // This class was automatically generated by build task

diff --git a/tests/unit/Rule/RequiredIfTest.php b/tests/unit/Rule/RequiredIfTest.php
@@ -10,7 +10,7 @@
 
 namespace Fuel\Validation\Rule;
 
-class RequiredIfTest extends RequiredTest
+class RequiredIfTest extends AbstractRequiredTest
 {
 
 	protected function _before()

diff --git a/tests/unit/Rule/RequiredTest.php b/tests/unit/Rule/RequiredTest.php
@@ -10,93 +10,7 @@
 
 namespace Fuel\Validation\Rule;
 
-class RequiredTest extends AbstractRuleTest
+class RequiredTest extends AbstractRequiredTest
 {
 
-	/**
-	 * {@inheritdocs}
-	 */
-	protected $message = 'The field is required and has not been specified.';
-
-	protected function _before()
-	{
-		$this->object = new Required;
-	}
-
-	/**
-	 * @dataProvider validateProvider
-	 */
-	public function testValidate()
-	{
-		list($value, $field, $expected, $data) = func_get_args();
-
-		$this->assertEquals(
-			$expected,
-			$this->object->validate($value, $field, $data)
-		);
-	}
-
-	/**
-	 * {@inheritdocs}
-	 */
-	public function validateProvider()
-	{
-		return array(
-			0 => array('[email protected]', null, false, null),
-			1 => array('', null, false, null),
-			2 => array(array(), null, false, null),
-			3 => array(null, null, false, null),
-			4 => array(false, null, false, null),
-
-			5 => array('test string 5', 'test', false,
-				array()
-			),
-
-			6 => array('test string 6', 'test', false,
-				array(
-					'foo' => 'bar',
-					'baz' => 'bat',
-				)
-			),
-
-			7 => array('test string 7', 'test', true,
-				array(
-					'foo' => 'bar',
-					'test' => 'value',
-					'baz' => 'bat',
-				)
-			),
-
-			8 => array('bla', 'test', true, null),
-			9 => array('', 'test', false, null),
-			10 => array('bla', null, false, array()),
-			11 => array('', null, false, array()),
-
-			12 => array(false, 'foo', true,
-				array(
-					'foo' => false,
-				)
-			),
-			13 => array(true, 'foo', true,
-				array(
-					'foo' => true,
-				)
-			),
-
-			14 => array(false, 'bar', true,
-				array(
-					'bar' => false,
-				)
-			),
-			15 => array(0, 'test', true,
-				array(
-					'foo' => 'bar',
-					'test' => 0,
-					'baz' => 'bat',
-				)
-			),
-			16 => array(0, 'test', true, array('test' => 0)),
-		);
-	}
-
 }
diff --git a/tests/unit/Rule/RequiredWithTest.php b/tests/unit/Rule/RequiredWithTest.php
@@ -10,7 +10,7 @@
 
 namespace Fuel\Validation\Rule;
 
-class RequiredWithTest extends RequiredTest
+class RequiredWithTest extends AbstractRequiredTest
 {
 
 	protected function _before()