Merge pull request #27 from grogy/test-refactor

refactor - moved tests case to class and  method

Author: JakubOnderka

tests/Settings.parseArguments.phpt

@@ -6,54 +6,60 @@ require_once __DIR__ . '/../src/Settings.php';
 use JakubOnderka\PhpParallelLint\Settings;
 use Tester\Assert;
 
+class SettingsParseArgumentsTest extends Tester\TestCase
+{
+	public function testNoneArguments()
+	{
+		$commandLine = "./parallel-lint .";
+		$argv = explode(" ", $commandLine);
+		$settings = Settings::parseArguments($argv);
 
+		$expectedSettings = new Settings();
+		$expectedSettings->phpExecutable = 'php';
+		$expectedSettings->shortTag = false;
+		$expectedSettings->aspTags = false;
+		$expectedSettings->parallelJobs = 10;
+		$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
+		$expectedSettings->paths = array('.');
+		$expectedSettings->excluded = array();
+		$expectedSettings->colors = true;
 
-// test command line string "./parallel-lint ."
-$commandLine = "./parallel-lint .";
-$argv = explode(" ", $commandLine);
-$settings = Settings::parseArguments($argv);
-
-$expectedSettings = new Settings();
-$expectedSettings->phpExecutable = 'php';
-$expectedSettings->shortTag = false;
-$expectedSettings->aspTags = false;
-$expectedSettings->parallelJobs = 10;
-$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
-$expectedSettings->paths = array('.');
-$expectedSettings->excluded = array();
-$expectedSettings->colors = true;
-
-Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
-Assert::equal($expectedSettings->shortTag, $settings->shortTag);
-Assert::equal($expectedSettings->aspTags, $settings->aspTags);
-Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
-Assert::equal($expectedSettings->extensions, $settings->extensions);
-Assert::equal($expectedSettings->paths, $settings->paths);
-Assert::equal($expectedSettings->excluded, $settings->excluded);
-Assert::equal($expectedSettings->colors, $settings->colors);
-
-
-
-// test command line string "./parallel-lint --exclude vendor --no-colors ."
-$commandLine = "./parallel-lint --exclude vendor --no-colors .";
-$argv = explode(" ", $commandLine);
-$settings = Settings::parseArguments($argv);
-
-$expectedSettings = new Settings();
-$expectedSettings->phpExecutable = 'php';
-$expectedSettings->shortTag = false;
-$expectedSettings->aspTags = false;
-$expectedSettings->parallelJobs = 10;
-$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
-$expectedSettings->paths = array('.');
-$expectedSettings->excluded = array('vendor');
-$expectedSettings->colors = false;
-
-Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
-Assert::equal($expectedSettings->shortTag, $settings->shortTag);
-Assert::equal($expectedSettings->aspTags, $settings->aspTags);
-Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
-Assert::equal($expectedSettings->extensions, $settings->extensions);
-Assert::equal($expectedSettings->paths, $settings->paths);
-Assert::equal($expectedSettings->excluded, $settings->excluded);
-Assert::equal($expectedSettings->colors, $settings->colors);
+		Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
+		Assert::equal($expectedSettings->shortTag, $settings->shortTag);
+		Assert::equal($expectedSettings->aspTags, $settings->aspTags);
+		Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
+		Assert::equal($expectedSettings->extensions, $settings->extensions);
+		Assert::equal($expectedSettings->paths, $settings->paths);
+		Assert::equal($expectedSettings->excluded, $settings->excluded);
+		Assert::equal($expectedSettings->colors, $settings->colors);
+	}
+
+	public function testMoreArguments()
+	{
+		$commandLine = "./parallel-lint --exclude vendor --no-colors .";
+		$argv = explode(" ", $commandLine);
+		$settings = Settings::parseArguments($argv);
+
+		$expectedSettings = new Settings();
+		$expectedSettings->phpExecutable = 'php';
+		$expectedSettings->shortTag = false;
+		$expectedSettings->aspTags = false;
+		$expectedSettings->parallelJobs = 10;
+		$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
+		$expectedSettings->paths = array('.');
+		$expectedSettings->excluded = array('vendor');
+		$expectedSettings->colors = false;
+
+		Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
+		Assert::equal($expectedSettings->shortTag, $settings->shortTag);
+		Assert::equal($expectedSettings->aspTags, $settings->aspTags);
+		Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
+		Assert::equal($expectedSettings->extensions, $settings->extensions);
+		Assert::equal($expectedSettings->paths, $settings->paths);
+		Assert::equal($expectedSettings->excluded, $settings->excluded);
+		Assert::equal($expectedSettings->colors, $settings->colors);
+	}
+}
+
+$testCase = new SettingsParseArgumentsTest;
+$testCase->run();