GlobalTradingTechnologies / workflow-extensions-bundle

Tests/Functional/Kernel/BaseTestKernel.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     /**
      * Returns base bundle list
      *
-     * @return array
+     * @return \Symfony\Bundle\FrameworkBundle\FrameworkBundle[]
      */
     protected function getBaseBundles()
     {

Spacing +3 added lines, -3 removed lines

@@ -65,14 +65,14 @@ 
         $this->setRootDir($rootDir);
 
         $this->configDir = realpath($configDir);
-        if (!is_dir($this->configDir . '/' . $testCase)) {
+        if (!is_dir($this->configDir.'/'.$testCase)) {
             throw new \InvalidArgumentException(sprintf('The test case "%s" does not exist.', $testCase));
         }
         $this->testCase = $testCase;
 
         $fs = new Filesystem();
         if (!$fs->isAbsolutePath($rootConfig) &&
-            !file_exists($rootConfig = $this->configDir . '/' . $testCase . '/' . $rootConfig)) {
+            !file_exists($rootConfig = $this->configDir.'/'.$testCase.'/'.$rootConfig)) {
             throw new \InvalidArgumentException(sprintf('The root config "%s" does not exist.', $rootConfig));
         }
         $this->rootConfig = $rootConfig;
@@ -127,7 +127,7 @@ 
     {
         $bundles     = array();
         $baseBundles = $this->getBaseBundles();
-        if (file_exists($filename = $this->configDir . '/' . $this->testCase . '/bundles.php')) {
+        if (file_exists($filename = $this->configDir.'/'.$this->testCase.'/bundles.php')) {
             $bundles = include $filename;
         }
         return array_merge($baseBundles, $bundles);

Tests/Functional/ScheduleCaseTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 
     /**
      * @param ContainerInterface $container
-     * @param array|string|false $fixturesPath
+     * @param string $fixturesPath
      * @param string $em
      * @param bool|true $append
      * @return array

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
 
         // load fixtures
         $fixturesBundle = new ClientBundle();
-        $fixturesPath = $fixturesBundle->getPath() . '/DataFixtures/ORM';
+        $fixturesPath = $fixturesBundle->getPath().'/DataFixtures/ORM';
         $this->loadFixtures($this->client->getContainer(), $fixturesPath);
     }
 
@@ -67,8 +67,8 @@ 
                 "app_name"    => "ScheduleCaseTest",
                 "test_case"   => "ScheduleCase",
                 "root_config" => "config.yml",
-                "config_dir"  => __DIR__ . "/Configuration",
-                "root_dir"    => __DIR__ . "/Configuration/ScheduleCase",
+                "config_dir"  => __DIR__."/Configuration",
+                "root_dir"    => __DIR__."/Configuration/ScheduleCase",
                 "environment" => "test",
                 "debug"       => false
             )

Tests/Guard/ExpressionGuardTest.php

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
         $event->expects(self::never())->method("setBlocked");
 
         $guard->registerGuardExpression("eventName", "workflow", "expression");
-        $guard->guardTransition($event ,"eventName");
+        $guard->guardTransition($event, "eventName");
     }
 
     /**
@@ -113,7 +113,7 @@ 
         $logger->expects($this->exactly($loggerInvocationCount))->method("debug");
 
         $guard->registerGuardExpression("eventName", "workflow", $expression);
-        $guard->guardTransition($event ,"eventName");
+        $guard->guardTransition($event, "eventName");
     }
 
     public function expressionProvider()

Doc Comments +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|Registry
+     * @return Registry
      */
     private function prepareValidWorkflowRegistryMock()
     {

Tests/Functional/EventTriggerWithGuardsCaseTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
                 "app_name" => "EventTriggerWithGuardsCaseTest",
                 "test_case" => "EventTriggerWithGuardsCase",
                 "root_config" => "config.yml",
-                "config_dir" => __DIR__ . "/Configuration",
+                "config_dir" => __DIR__."/Configuration",
                 "environment" => "test",
                 "debug" => false
             )

DependencyInjection/Compiler/CheckSubjectManipulatorConfigPass.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     /**
      * Workflow registry definition id
      */
-    const WORKFLOW_REGISTRY_ID= 'workflow.registry';
+    const WORKFLOW_REGISTRY_ID = 'workflow.registry';
 
     /**
      * Workflow id prefix used in main workflow bundle
@@ -71,7 +71,7 @@ 
                 $workflowReference      = $callArgs[0];
                 $workflowSupportedClass = $callArgs[1];
 
-                $workflowIdWithPrefix =  (string) $workflowReference;
+                $workflowIdWithPrefix = (string) $workflowReference;
                 if (substr($workflowIdWithPrefix, 0, strlen(self::WORKFLOW_ID_PREFIX)) != self::WORKFLOW_ID_PREFIX) {
                     throw new RuntimeException(
                         sprintf(

DependencyInjection/WorkflowExtensionsExtension.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     {
         $configuration = $this->getConfiguration($config, $container);
         $config = $this->processConfiguration($configuration, $config);
-        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('applier.xml');
 
         $this->registerSubjectManipulatorConfiguration($loader, $container, $config['subject_manipulator']);

WorkflowExtensionsBundle.php

Indentation +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
  */
 class WorkflowExtensionsBundle extends Bundle
 {    /**
-     * {@inheritdoc}
-     */
+ * {@inheritdoc}
+ */
     public function build(ContainerBuilder $container)
     {
         parent::build($container);

Entity/Repository/ScheduledJobRepository.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Doctrine\ORM\EntityRepository;
 use Gtt\Bundle\WorkflowExtensionsBundle\Entity\ScheduledJob;
 use Gtt\Bundle\WorkflowExtensionsBundle\Exception\NonUniqueReschedulabeJobFoundException;
-use Gtt\Bundle\WorkflowExtensionsBundle\Schedule\Action\ScheduledAction;
 use JMS\JobQueueBundle\Entity\Job;
 
 /**

Tests/Action/ExecutorTest.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Gtt\Bundle\WorkflowExtensionsBundle\Action\Registry;
 use Gtt\Bundle\WorkflowExtensionsBundle\Tests\Action\Reference\ContainerAwareActionReferenceInterface;
 use Gtt\Bundle\WorkflowExtensionsBundle\WorkflowContext;
-use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\Workflow\Workflow;