Carteni / crypto-bundle

Command/AbstractCommand.php

Doc Comments +5 added lines, -5 removed lines

@@ -74,8 +74,8 @@ 
     }
 
     /**
-     * @param $question
-     * @param $default
+     * @param string $question
+     * @param string $default
      * @param string $sep
      *
      * @return string
@@ -90,7 +90,7 @@ 
 
     /**
      * @param \Symfony\Component\Console\Output\OutputInterface $output
-     * @param $text
+     * @param string $text
      * @param string $style
      */
     protected function writeSection(OutputInterface $output, $text, $style = 'bg=blue;fg=white')
@@ -109,8 +109,8 @@ 
     }
 
     /**
-     * @param $text
-     * @param $style
+     * @param string $text
+     * @param string $style
      *
      * @return string
      */

Indentation +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
          * @var FormatterHelper
          */
         $formatter = $this->getHelperSet()
-                          ->get('formatter');
+                            ->get('formatter');
 
         $output->writeln(array(
             '',

Encryption.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * @param Key|KeyInterface $key
      *
-     * @return \Defuse\Crypto\Key|\Defuse\Crypto\KeyProtectedByPassword
+     * @return \Defuse\Crypto\Key
      */
     private function unlockKey(Key $key)
     {

Indentation +1 added lines, -1 removed lines

@@ -49,6 +49,6 @@
     private function unlockKey(Key $key)
     {
         return $key->unlock()
-                   ->getRawKey();
+                    ->getRawKey();
     }
 }

Tests/Command/KeyGeneratorCommandTest.php

Doc Comments +3 added lines

@@ -135,6 +135,9 @@
         $this->helper = null;
     }
 
+    /**
+     * @param string[] $input
+     */
     private function getInputStream($input)
     {
         $stream = fopen('php://memory', 'r+', false);

Model/Key.php

Indentation +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 
         // rawKey is locked and become instance of \Defuse\Crypto\Key.
         $this->rawKey = $this->getRawKey()
-                             ->unlockKey($this->getSecret());
+                                ->unlockKey($this->getSecret());
 
         return $this;
     }

Command/SecretGeneratorCommand.php

Indentation +3 added lines, -3 removed lines

@@ -26,13 +26,13 @@
     protected function configure()
     {
         $this->setName('mes:crypto:generate-secret')
-             ->setDescription('Generates an authentication secret')
-             ->setHelp(<<<'EOF'
+                ->setDescription('Generates an authentication secret')
+                ->setHelp(<<<'EOF'
 The <info>%command.name%</info> generates an authentication secret.
 
 <info>%command.full_name%</info>
 EOF
-             );
+                );
     }
 
     /**

DependencyInjection/MesCryptoExtension.php

Indentation +15 added lines, -15 removed lines

@@ -125,27 +125,27 @@ 
 
             // Reads encoded key from configuration file if key is not external.
             $rawKeyDefinition->setClass($defuseKey)
-                             ->setFactory(array(
-                                 $defuseKey,
-                                 'loadFromAsciiSafeString',
-                             ));
+                                ->setFactory(array(
+                                    $defuseKey,
+                                    'loadFromAsciiSafeString',
+                                ));
             $rawKeyDefinition->setArguments(array(
                 $keyIsExternal ? $loadKey : $key,
             ));
         } else {
             $rawKeyDefinition->setClass($defuseKey)
-                             ->setArguments($secretExists ? array($secretIsExternal ? $loadSecret : $secret) : array())
-                             ->setFactory($secretExists ? array(
-                                 'Defuse\Crypto\KeyProtectedByPassword',
-                                 'createRandomPasswordProtectedKey',
-                             ) : array(
-                                 'Defuse\Crypto\Key',
-                                 'createNewRandomKey',
-                             ));
+                                ->setArguments($secretExists ? array($secretIsExternal ? $loadSecret : $secret) : array())
+                                ->setFactory($secretExists ? array(
+                                    'Defuse\Crypto\KeyProtectedByPassword',
+                                    'createRandomPasswordProtectedKey',
+                                ) : array(
+                                    'Defuse\Crypto\Key',
+                                    'createNewRandomKey',
+                                ));
         }
 
         $container->setDefinition('mes_crypto.raw_key', $rawKeyDefinition)
-                  ->setPublic(false);
+                    ->setPublic(false);
 
         // Key
         $keyDefinition = new Definition('Mes\Security\CryptoBundle\Model\Key', array(
@@ -157,7 +157,7 @@ 
             'create',
         ));
         $container->setDefinition('mes_crypto.key', $keyDefinition)
-                  ->setPublic(false);
+                    ->setPublic(false);
 
         $keyManagerDefinition = $container->getDefinition('mes_crypto.key_manager_wrapper');
 
@@ -192,7 +192,7 @@ 
     private function setCryptoLoaderResource(ContainerBuilder $container, $resource)
     {
         $container->getDefinition('mes_crypto.crypto_loader')
-                  ->replaceArgument(0, $resource);
+                    ->replaceArgument(0, $resource);
     }
 
     /**

DependencyInjection/Configuration.php

Indentation +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
         $rootNode = $treeBuilder->root('mes_crypto');
 
         $rootNode
-          ->children()
+            ->children()
             ->arrayNode('key')
                 ->addDefaultsIfNotSet()
                 ->children()
@@ -98,7 +98,7 @@ 
                 ->info('your_encryption_service_id. Default: Mes Encryption.')
                 ->defaultNull()
             ->end()
-          ->end()
+            ->end()
         ;
 
         return $treeBuilder;

Tests/KeyManagerTest.php

Indentation +20 added lines, -20 removed lines

@@ -36,9 +36,9 @@ 
     public function testGenerateCreatesKey()
     {
         $this->keyGenerator->expects($this->once())
-                           ->method('generate')
-                           ->with(null)
-                           ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
+                            ->method('generate')
+                            ->with(null)
+                            ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
 
         $key = $this->keyManager->generate();
 
@@ -51,9 +51,9 @@ 
         $secret = 'ThisIsASecretPassword';
 
         $this->keyGenerator->expects($this->once())
-                           ->method('generateFromAscii')
-                           ->with($key_encoded, $secret)
-                           ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
+                            ->method('generateFromAscii')
+                            ->with($key_encoded, $secret)
+                            ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
 
         $key = $this->keyManager->generateFromAscii($key_encoded, $secret);
 
@@ -63,16 +63,16 @@ 
     public function testGetKeyReadsNotExistingKey()
     {
         $this->keyStorage->expects($this->once())
-                         ->method('getKey')
-                         ->will($this->returnValue(null));
+                            ->method('getKey')
+                            ->will($this->returnValue(null));
 
         $this->keyGenerator->expects($this->once())
-                           ->method('generate')
-                           ->with(null)
-                           ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
+                            ->method('generate')
+                            ->with(null)
+                            ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
 
         $this->keyStorage->expects($this->once())
-                         ->method('setKey');
+                            ->method('setKey');
 
         $key = $this->keyManager->getKey();
 
@@ -82,14 +82,14 @@ 
     public function testGetKeyReadsExistingKey()
     {
         $this->keyStorage->expects($this->never())
-                         ->method('setKey');
+                            ->method('setKey');
 
         $this->keyGenerator->expects($this->never())
-                           ->method('generate');
+                            ->method('generate');
 
         $this->keyStorage->expects($this->once())
-                         ->method('getKey')
-                         ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
+                            ->method('getKey')
+                            ->will($this->returnValue($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface')));
 
         $key = $this->keyManager->getKey();
 
@@ -99,8 +99,8 @@ 
     public function testSetKeyStoresKey()
     {
         $this->keyStorage->expects($this->once())
-                         ->method('setKey')
-                         ->with($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'));
+                            ->method('setKey')
+                            ->with($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'));
 
         $this->keyManager->setKey($this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'));
     }
@@ -115,10 +115,10 @@ 
     protected function setUp()
     {
         $this->keyStorage = $this->getMockBuilder('Mes\Security\CryptoBundle\KeyStorage\KeyStorageInterface')
-                                 ->getMock();
+                                    ->getMock();
 
         $this->keyGenerator = $this->getMockBuilder('Mes\Security\CryptoBundle\KeyGenerator\KeyGeneratorInterface')
-                                   ->getMock();
+                                    ->getMock();
 
         $this->keyManager = new KeyManager($this->keyStorage, $this->keyGenerator);
     }

Tests/EncryptionWrapperTest.php

Indentation +13 added lines, -13 removed lines

@@ -34,9 +34,9 @@ 
     public function testEncryptEncryptsPlaintext()
     {
         $this->encryption->expects($this->once())
-                         ->method('encrypt')
-                         ->with('The quick brown fox jumps over the lazy dog', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
-                         ->will($this->returnValue('ThisIsACipherText'));
+                            ->method('encrypt')
+                            ->with('The quick brown fox jumps over the lazy dog', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
+                            ->will($this->returnValue('ThisIsACipherText'));
 
         $ciphertext = $this->wrapper->encrypt('The quick brown fox jumps over the lazy dog', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'));
 
@@ -50,9 +50,9 @@ 
     {
         try {
             $this->encryption->expects($this->once())
-                             ->method('encrypt')
-                             ->with('The quick brown fox jumps over the lazy dog', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
-                             ->will($this->throwException(new EnvironmentIsBrokenException()));
+                                ->method('encrypt')
+                                ->with('The quick brown fox jumps over the lazy dog', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
+                                ->will($this->throwException(new EnvironmentIsBrokenException()));
         } catch (EnvironmentIsBrokenException $e) {
             $this->throwException(new CryptoException());
         }
@@ -63,9 +63,9 @@ 
     public function testDecryptDecryptsCiphertext()
     {
         $this->encryption->expects($this->once())
-                         ->method('decrypt')
-                         ->with('ThisIsACipherText', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
-                         ->will($this->returnValue('The quick brown fox jumps over the lazy dog'));
+                            ->method('decrypt')
+                            ->with('ThisIsACipherText', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
+                            ->will($this->returnValue('The quick brown fox jumps over the lazy dog'));
 
         $decryptedText = $this->wrapper->decrypt('ThisIsACipherText', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'));
 
@@ -79,9 +79,9 @@ 
     {
         try {
             $this->encryption->expects($this->once())
-                             ->method('decrypt')
-                             ->with('ThisIsACipherText', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
-                             ->will($this->throwException(new BaseCryptoException()));
+                                ->method('decrypt')
+                                ->with('ThisIsACipherText', $this->getMock('Mes\Security\CryptoBundle\Model\KeyInterface'))
+                                ->will($this->throwException(new BaseCryptoException()));
         } catch (BaseCryptoException $e) {
             $this->throwException(new CryptoException());
         }
@@ -92,7 +92,7 @@ 
     protected function setUp()
     {
         $this->encryption = $this->getMockBuilder('Mes\Security\CryptoBundle\EncryptionInterface')
-                                 ->getMock();
+                                    ->getMock();
         $this->wrapper = new EncryptionWrapper($this->encryption);
     }