Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

Commit 4ccf485

Browse files
committed
Merge pull request #39 from grogy/code-style
Refactor - four space for identation
2 parents d94caff + 3c6a526 commit 4ccf485

File tree

2 files changed

+56
-56
lines changed

2 files changed

+56
-56
lines changed

src/Manager.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -66,15 +66,15 @@ public function run(Settings $settings = null)
6666
$parallelLint->setShortTagEnabled($settings->shortTag);
6767

6868
$parallelLint->setProcessCallback(function ($status, $file) use ($output) {
69-
if ($status === ParallelLint::STATUS_OK) {
70-
$output->ok();
71-
} elseif ($status === ParallelLint::STATUS_SKIP) {
72-
$output->skip();
73-
} elseif ($status === ParallelLint::STATUS_ERROR) {
74-
$output->error();
75-
} else {
76-
$output->fail();
77-
}
69+
if ($status === ParallelLint::STATUS_OK) {
70+
$output->ok();
71+
} elseif ($status === ParallelLint::STATUS_SKIP) {
72+
$output->skip();
73+
} elseif ($status === ParallelLint::STATUS_ERROR) {
74+
$output->error();
75+
} else {
76+
$output->fail();
77+
}
7878
});
7979

8080
$result = $parallelLint->lint($files);

tests/Settings.parseArguments.phpt

Lines changed: 47 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -11,61 +11,61 @@ use Tester\Assert;
1111

1212
class SettingsParseArgumentsTest extends Tester\TestCase
1313
{
14-
public function testNoneArguments()
15-
{
16-
$commandLine = "./parallel-lint .";
17-
$argv = explode(" ", $commandLine);
18-
$settings = Settings::parseArguments($argv);
14+
public function testNoneArguments()
15+
{
16+
$commandLine = "./parallel-lint .";
17+
$argv = explode(" ", $commandLine);
18+
$settings = Settings::parseArguments($argv);
1919

20-
$expectedSettings = new Settings();
21-
$expectedSettings->phpExecutable = 'php';
22-
$expectedSettings->shortTag = false;
23-
$expectedSettings->aspTags = false;
24-
$expectedSettings->parallelJobs = 10;
25-
$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
26-
$expectedSettings->paths = array('.');
27-
$expectedSettings->excluded = array();
28-
$expectedSettings->colors = true;
29-
$expectedSettings->json = false;
20+
$expectedSettings = new Settings();
21+
$expectedSettings->phpExecutable = 'php';
22+
$expectedSettings->shortTag = false;
23+
$expectedSettings->aspTags = false;
24+
$expectedSettings->parallelJobs = 10;
25+
$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
26+
$expectedSettings->paths = array('.');
27+
$expectedSettings->excluded = array();
28+
$expectedSettings->colors = true;
29+
$expectedSettings->json = false;
3030

31-
Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
32-
Assert::equal($expectedSettings->shortTag, $settings->shortTag);
33-
Assert::equal($expectedSettings->aspTags, $settings->aspTags);
34-
Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
35-
Assert::equal($expectedSettings->extensions, $settings->extensions);
36-
Assert::equal($expectedSettings->paths, $settings->paths);
37-
Assert::equal($expectedSettings->excluded, $settings->excluded);
38-
Assert::equal($expectedSettings->colors, $settings->colors);
31+
Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
32+
Assert::equal($expectedSettings->shortTag, $settings->shortTag);
33+
Assert::equal($expectedSettings->aspTags, $settings->aspTags);
34+
Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
35+
Assert::equal($expectedSettings->extensions, $settings->extensions);
36+
Assert::equal($expectedSettings->paths, $settings->paths);
37+
Assert::equal($expectedSettings->excluded, $settings->excluded);
38+
Assert::equal($expectedSettings->colors, $settings->colors);
3939
Assert::equal($expectedSettings->json, $settings->json);
40-
}
40+
}
4141

42-
public function testMoreArguments()
43-
{
44-
$commandLine = "./parallel-lint --exclude vendor --no-colors .";
45-
$argv = explode(" ", $commandLine);
46-
$settings = Settings::parseArguments($argv);
42+
public function testMoreArguments()
43+
{
44+
$commandLine = "./parallel-lint --exclude vendor --no-colors .";
45+
$argv = explode(" ", $commandLine);
46+
$settings = Settings::parseArguments($argv);
4747

48-
$expectedSettings = new Settings();
49-
$expectedSettings->phpExecutable = 'php';
50-
$expectedSettings->shortTag = false;
51-
$expectedSettings->aspTags = false;
52-
$expectedSettings->parallelJobs = 10;
53-
$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
54-
$expectedSettings->paths = array('.');
55-
$expectedSettings->excluded = array('vendor');
56-
$expectedSettings->colors = false;
48+
$expectedSettings = new Settings();
49+
$expectedSettings->phpExecutable = 'php';
50+
$expectedSettings->shortTag = false;
51+
$expectedSettings->aspTags = false;
52+
$expectedSettings->parallelJobs = 10;
53+
$expectedSettings->extensions = array('php', 'phtml', 'php3', 'php4', 'php5');
54+
$expectedSettings->paths = array('.');
55+
$expectedSettings->excluded = array('vendor');
56+
$expectedSettings->colors = false;
5757
$expectedSettings->json = false;
5858

59-
Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
60-
Assert::equal($expectedSettings->shortTag, $settings->shortTag);
61-
Assert::equal($expectedSettings->aspTags, $settings->aspTags);
62-
Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
63-
Assert::equal($expectedSettings->extensions, $settings->extensions);
64-
Assert::equal($expectedSettings->paths, $settings->paths);
65-
Assert::equal($expectedSettings->excluded, $settings->excluded);
66-
Assert::equal($expectedSettings->colors, $settings->colors);
59+
Assert::equal($expectedSettings->phpExecutable, $settings->phpExecutable);
60+
Assert::equal($expectedSettings->shortTag, $settings->shortTag);
61+
Assert::equal($expectedSettings->aspTags, $settings->aspTags);
62+
Assert::equal($expectedSettings->parallelJobs, $settings->parallelJobs);
63+
Assert::equal($expectedSettings->extensions, $settings->extensions);
64+
Assert::equal($expectedSettings->paths, $settings->paths);
65+
Assert::equal($expectedSettings->excluded, $settings->excluded);
66+
Assert::equal($expectedSettings->colors, $settings->colors);
6767
Assert::equal($expectedSettings->json, $settings->json);
68-
}
68+
}
6969
}
7070

7171
$testCase = new SettingsParseArgumentsTest;

0 commit comments

Comments
 (0)