Failed Conditions
Pull Request — master (#33)
by Matthias
02:31
created
example/test.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -10,16 +10,16 @@
 block discarded – undo
10 10
 
11 11
 // Please run "run-test.sh", and not this file directly.
12 12
 
13
-(function () {
14
-    require_once  __DIR__ . '/../vendor/autoload.php';
13
+(function() {
14
+    require_once  __DIR__.'/../vendor/autoload.php';
15 15
 
16 16
     $whitelistSymbols = ['null', 'true', 'false', 'static', 'self', 'parent'];
17
-    $defaultExtensions = ["Core","standard"]; // "bcmath","calendar","ctype","date","filter","hash","iconv","json","mcrypt","SPL","pcre","Reflection","session","standard","mysqlnd","tokenizer","zip","zlib","libxml","dom","PDO","Phar","SimpleXML","xml","wddx","xmlreader","xmlwriter"];
17
+    $defaultExtensions = ["Core", "standard"]; // "bcmath","calendar","ctype","date","filter","hash","iconv","json","mcrypt","SPL","pcre","Reflection","session","standard","mysqlnd","tokenizer","zip","zlib","libxml","dom","PDO","Phar","SimpleXML","xml","wddx","xmlreader","xmlwriter"];
18 18
 
19 19
     $getPackageSourceFiles = new LocateComposerPackageSourceFiles();
20 20
 
21 21
     $sourcesASTs  = new LocateASTFromFiles((new ParserFactory())->create(ParserFactory::PREFER_PHP7));
22
-    $composerJson = __DIR__ . '/test-data/zend-feed/composer.json';
22
+    $composerJson = __DIR__.'/test-data/zend-feed/composer.json';
23 23
 
24 24
     $definedVendorSymbols = (new LocateDefinedSymbolsFromASTRoots())->__invoke($sourcesASTs(
25 25
         (new ComposeGenerators())->__invoke(
Please login to merge, or discard this patch.
src/ComposerRequireChecker/DependencyGuesser/GuessFromLoadedExtensions.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
         foreach ($this->loadedExtensions as $extensionName) {
22 22
             $extensionSymbols = $definedSymbolsFromExtensions([$extensionName]);
23 23
             if (in_array($symbolName, $extensionSymbols)) {
24
-                yield 'ext-' . $extensionName;
24
+                yield 'ext-'.$extensionName;
25 25
             }
26 26
         }
27 27
     }
Please login to merge, or discard this patch.
NodeVisitor/DefinedSymbolCollectorFunctionalTest.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace ComposerRequireCheckerTest\NodeVisitor;
4 4
 
5 5
 use ComposerRequireChecker\NodeVisitor\DefinedSymbolCollector;
6
-use PhpParser\Node;
7 6
 use PhpParser\NodeTraverser;
8 7
 use PhpParser\NodeTraverserInterface;
9 8
 use PhpParser\NodeVisitor\NameResolver;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
 
98 98
     private function traverseStringAST(string $phpSource) : array
99 99
     {
100
-        return $this->traverser->traverse($this->parser->parse('<?php ' . $phpSource));
100
+        return $this->traverser->traverse($this->parser->parse('<?php '.$phpSource));
101 101
     }
102 102
 
103 103
     private function traverseClassAST(string $className) : array
Please login to merge, or discard this patch.
NodeVisitor/UsedSymbolCollectorFunctionalTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace ComposerRequireCheckerTest\NodeVisitor;
4 4
 
5 5
 use ComposerRequireChecker\NodeVisitor\UsedSymbolCollector;
6
-use PhpParser\Node;
7 6
 use PhpParser\NodeTraverser;
8 7
 use PhpParser\NodeTraverserInterface;
9 8
 use PhpParser\NodeVisitor\NameResolver;
Please login to merge, or discard this patch.
test/ComposerRequireCheckerTest/DependencyGuesser/DependencyGuesserTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
 
21 21
     public function testGuessExtJson()
22 22
     {
23
-        if(!extension_loaded('json')) {
23
+        if (!extension_loaded('json')) {
24 24
             $this->markTestSkipped('extension json is not available');
25 25
         }
26 26
         $result = $this->guesser->__invoke('json_decode');
Please login to merge, or discard this patch.
test/ComposerRequireCheckerTest/Cli/OptionsTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
 
40 40
         $optionsFromFile = new Options(
41 41
             json_decode(file_get_contents(
42
-                __DIR__ . '/../../../data/config.dist.json'
42
+                __DIR__.'/../../../data/config.dist.json'
43 43
             ), true)
44 44
         );
45 45
 
Please login to merge, or discard this patch.
src/ComposerRequireChecker/FileLocator/LocateComposerPackageSourceFiles.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -29,14 +29,14 @@
 block discarded – undo
29 29
     {
30 30
         $flattened = array_reduce(
31 31
             $sourceDirs,
32
-            function (array $sourceDirs, $sourceDir) {
33
-                return array_merge($sourceDirs, (array)$sourceDir);
32
+            function(array $sourceDirs, $sourceDir) {
33
+                return array_merge($sourceDirs, (array) $sourceDir);
34 34
             },
35 35
             []
36 36
         );
37 37
         return array_values(array_map(
38
-            function (string $sourceDir) use ($packageDir) {
39
-                return $this->normalizePath($packageDir . '/' . ltrim($sourceDir, '/'));
38
+            function(string $sourceDir) use ($packageDir) {
39
+                return $this->normalizePath($packageDir.'/'.ltrim($sourceDir, '/'));
40 40
             },
41 41
             $flattened
42 42
         ));
Please login to merge, or discard this patch.
src/ComposerRequireChecker/JsonLoader.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,11 +20,11 @@
 block discarded – undo
20 20
     public function __construct($path)
21 21
     {
22 22
         if (!is_readable($path) || ($content = file_get_contents($path)) === false) {
23
-            throw new NotReadableException('unable to read ' . $path);
23
+            throw new NotReadableException('unable to read '.$path);
24 24
         }
25 25
         $this->data = json_decode($content, true);
26 26
         if ($this->data === null && JSON_ERROR_NONE !== json_last_error()) {
27
-            throw new InvalidJsonException('error parsing ' . $path . ': ' . json_last_error_msg());
27
+            throw new InvalidJsonException('error parsing '.$path.': '.json_last_error_msg());
28 28
         }
29 29
     }
30 30
 
Please login to merge, or discard this patch.
test/ComposerRequireCheckerTest/JsonLoaderTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -15,21 +15,21 @@
 block discarded – undo
15 15
 
16 16
     public function testHasErrorWithWrongPath()
17 17
     {
18
-        $path = __DIR__ . '/wrong/path/non-existing-file.json';
18
+        $path = __DIR__.'/wrong/path/non-existing-file.json';
19 19
         $this->expectException(NotReadableException::class);
20 20
         new JsonLoader($path);
21 21
     }
22 22
 
23 23
     public function testHasErrorWithInvalidFile()
24 24
     {
25
-        $path = __DIR__ . '/../fixtures/invalidJson';
25
+        $path = __DIR__.'/../fixtures/invalidJson';
26 26
         $this->expectException(InvalidJsonException::class);
27 27
         new JsonLoader($path);
28 28
     }
29 29
 
30 30
     public function testHasDataWithValidFile()
31 31
     {
32
-        $path = __DIR__ . '/../fixtures/validJson.json';
32
+        $path = __DIR__.'/../fixtures/validJson.json';
33 33
         $loader = new JsonLoader($path);
34 34
         $this->assertEquals($loader->getData(), ['foo' => 'bar']);
35 35
     }
Please login to merge, or discard this patch.