Passed
Pull Request — master (#11)
by Pavel
06:35
created
DependencyInjection/BankiruJsonRpcServerExtension.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 
3 3
 namespace Bankiru\Api\JsonRpc\DependencyInjection;
4 4
 
5
-use Bankiru\Api\JsonRpc\Adapters\JMS\Compiler\JmsDriverPass;
6 5
 use Bankiru\Api\JsonRpc\Adapters\JMS\Compiler\RelationHandlerHelper;
7 6
 use Bankiru\Api\Rpc\RpcEvents;
8 7
 use JMS\SerializerBundle\JMSSerializerBundle;
Please login to merge, or discard this patch.
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -28,15 +28,15 @@
 block discarded – undo
28 28
 
29 29
         if (!empty($config['view_listener'])) {
30 30
             $container->getDefinition($config['view_listener'])
31
-                      ->setPublic(true)
32
-                      ->addTag(
33
-                          'kernel.event_listener',
34
-                          [
35
-                              'event'    => RpcEvents::VIEW,
36
-                              'method'   => 'onPlainResponse',
37
-                              'priority' => -254,
38
-                          ]
39
-                      );
31
+                        ->setPublic(true)
32
+                        ->addTag(
33
+                            'kernel.event_listener',
34
+                            [
35
+                                'event'    => RpcEvents::VIEW,
36
+                                'method'   => 'onPlainResponse',
37
+                                'priority' => -254,
38
+                            ]
39
+                        );
40 40
         }
41 41
     }
42 42
 
Please login to merge, or discard this patch.
Tests/JsonRpcControllerTest.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -165,6 +165,9 @@
 block discarded – undo
165 165
         );
166 166
     }
167 167
 
168
+    /**
169
+     * @param string $uri
170
+     */
168 171
     private function createJsonRequest($uri, $content)
169 172
     {
170 173
         return Request::create($uri, 'POST', [], [], [], [], json_encode($content));
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
                 'method'  => 'test',
94 94
             ]
95 95
         );
96
-        $response   = $controller->jsonRpcAction($request);
96
+        $response = $controller->jsonRpcAction($request);
97 97
 
98 98
         self::assertTrue($response->isSuccessful());
99 99
         self::assertEquals('{"jsonrpc":"2.0","id":"test","result":{"success":true}}', $response->getContent());
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
                 ],
113 113
             ]
114 114
         );
115
-        $response   = $controller->jsonRpcAction($request);
115
+        $response = $controller->jsonRpcAction($request);
116 116
 
117 117
         self::assertTrue($response->isSuccessful());
118 118
         self::assertEquals('[{"jsonrpc":"2.0","id":"test","result":{"success":true}}]', $response->getContent());
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
                 'method'  => 'exception',
130 130
             ]
131 131
         );
132
-        $response   = $controller->jsonRpcAction($request);
132
+        $response = $controller->jsonRpcAction($request);
133 133
 
134 134
         self::assertTrue($response->isSuccessful());
135 135
         self::assertEquals(
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
                 ],
157 157
             ]
158 158
         );
159
-        $response   = $controller->jsonRpcAction($request);
159
+        $response = $controller->jsonRpcAction($request);
160 160
 
161 161
         self::assertTrue($response->isSuccessful());
162 162
         self::assertEquals(
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
         $kernel   = $this->prophesize(KernelInterface::class);
177 177
         $resolver = $this->prophesize(ControllerResolverInterface::class);
178 178
         $resolver->getController(Argument::type(RpcRequestInterface::class))->willReturn(
179
-            function (JsonRpcRequestInterface $request) {
179
+            function(JsonRpcRequestInterface $request) {
180 180
                 if ($request->getMethod() === 'exception') {
181 181
                     throw new \LogicException('Failure!');
182 182
                 }
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
             }
186 186
         );
187 187
         $resolver->getArguments(Argument::type(RpcRequestInterface::class), Argument::any())->will(
188
-            function (array $args) {
188
+            function(array $args) {
189 189
                 return [
190 190
                     $args[0],
191 191
                 ];
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
         $evm = $this->prophesize(EventDispatcherInterface::class);
196 196
         $evm->dispatch(Argument::exact(RpcEvents::EXCEPTION), Argument::type(GetExceptionResponseEvent::class))
197 197
             ->will(
198
-                function ($args) {
198
+                function($args) {
199 199
                     /** @var GetExceptionResponseEvent $event */
200 200
                     $event = $args[1];
201 201
 
Please login to merge, or discard this patch.
DependencyInjection/Configuration.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -15,9 +15,9 @@
 block discarded – undo
15 15
         $root    = $builder->root('jsonrpc_server');
16 16
 
17 17
         $root->children()
18
-             ->scalarNode('view_listener')
19
-             ->info('View listener service ID')
20
-             ->defaultValue('jsonrpc_server.builtin_adapter.view_listener');
18
+                ->scalarNode('view_listener')
19
+                ->info('View listener service ID')
20
+                ->defaultValue('jsonrpc_server.builtin_adapter.view_listener');
21 21
 
22 22
         $adapters = $root->children()->arrayNode('adapters');
23 23
         $this->configureJms($adapters);
Please login to merge, or discard this patch.
Tests/JsonRpcHttpResponseTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
             'empty array'    => [[]],
25 25
             'single success' => [clone $success],
26 26
             'single failure' => [clone $error],
27
-            'mixed array'    => [[clone $success, clone $error,]],
27
+            'mixed array'    => [[clone $success, clone $error, ]],
28 28
         ];
29 29
     }
30 30
 
Please login to merge, or discard this patch.
Test/Tests/RouterTest.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,9 +23,11 @@
 block discarded – undo
23 23
     {
24 24
         $client = self::createClient();
25 25
 
26
-        foreach (['test', 'test_private'] as $endpoint) /** @var MethodCollection $collection */ {
26
+        foreach (['test', 'test_private'] as $endpoint) {
27
+            /** @var MethodCollection $collection */ {
27 28
             /** @var Router $router */
28 29
             $router = $client->getContainer()->get('rpc_server.endpoint_router.' . $endpoint);
30
+        }
29 31
             self::assertNotNull($router);
30 32
             $collection = $router->getMethodCollection();
31 33
             self::assertNotNull($router);
Please login to merge, or discard this patch.