Remove redundant paragonie/sodium_compat dependency

.phpunit.result.cache

@@ -0,0 +1 @@
+{"version":1,"defects":{"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testPersistEntity":4,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testNoUpdateOnReadEncrypted":4,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testStoredDataIsEncrypted":4,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testPersistEntity":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testNoUpdateOnReadEncrypted":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testStoredDataIsEncrypted":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\DependencyInjection\\DoctrineEncryptExtensionTest::testConfigLoadCustom":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testEncryptWithoutExtensionThrowsException":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\DefuseEncryptorTest::testEncrypt":4,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\DefuseEncryptorTest::testGenerateKey":4,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testEncryptExtension":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testGenerateKey":1,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsNoEncryptor":4},"times":{"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testPersistEntity":0.456,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testNoUpdateOnReadEncrypted":1.046,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryDefuseTest::testStoredDataIsEncrypted":0.654,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testPersistEntity":0.106,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testNoUpdateOnReadEncrypted":0.142,"Ambta\\DoctrineEncryptBundle\\Tests\\Functional\\BasicQueryTest\\BasicQueryHaliteTest::testStoredDataIsEncrypted":0.126,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\DependencyInjection\\DoctrineEncryptExtensionTest::testConfigLoadHalite":0.031,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\DependencyInjection\\DoctrineEncryptExtensionTest::testConfigLoadDefuse":0.012,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\DependencyInjection\\DoctrineEncryptExtensionTest::testConfigLoadCustom":0.013,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\DefuseEncryptorTest::testEncrypt":0.252,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\DefuseEncryptorTest::testGenerateKey":0.132,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testEncryptExtension":0.005,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testGenerateKey":0.008,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Encryptors\\HaliteEncryptorTest::testEncryptWithoutExtensionThrowsException":0.001,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testSetRestorEncryptor":0.01,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsEncrypt":0.003,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsEncryptExtend":0.003,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsEncryptEmbedded":0.004,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsEncryptNull":0.002,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsNoEncryptor":0.001,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsDecrypt":0.002,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsDecryptExtended":0.003,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsDecryptEmbedded":0.003,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsDecryptNull":0.002,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testProcessFieldsDecryptNonEncrypted":0.002,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testOnFlush":0.052,"Ambta\\DoctrineEncryptBundle\\Tests\\Unit\\Subscribers\\DoctrineEncryptSubscriberTest::testPostFlush":0.006}}

.travis.yml

@@ -26,6 +26,12 @@ matrix:
     - php: 7.3
       env: COVERAGE=true PHPUNIT_FLAGS="-v --coverage-text"
 
+    - php: 7.4
+      env: COVERAGE=true PHPUNIT_FLAGS="-v --coverage-text"
+
+    - php: 8.0
+      env: COVERAGE=true PHPUNIT_FLAGS="-v --coverage-text"
+
       # Latest commit to master
     - php: 7.3
       env: STABILITY="dev"

Tests/Functional/AbstractFunctionalTestCase.php

@@ -32,7 +32,7 @@ abstract class AbstractFunctionalTestCase extends TestCase
 
     abstract protected function getEncryptor(): EncryptorInterface;
 
-    public function setUp()
+    public function setUp(): void
     {
         // Create a simple "default" Doctrine ORM configuration for Annotations
         $isDevMode                 = true;
@@ -73,8 +73,9 @@ public function setUp()
         error_reporting(E_ALL);
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
+        $this->entityManager->getConnection()->close();
         unlink($this->dbFile);
     }
 
@@ -132,4 +133,4 @@ public function assertStringDoesNotContain($needle, $string, $ignoreCase = false
 
         static::assertThat($string, $constraint, $message);
     }
-}
+}

Tests/Functional/BasicQueryTest/AbstractBasicQueryTestCase.php

@@ -8,7 +8,7 @@
 
 abstract class AbstractBasicQueryTestCase extends AbstractFunctionalTestCase
 {
-    public function testPersistEntity()
+    public function testPersistEntity(): void
     {
         $user = new CascadeTarget();
         $user->setNotSecret('My public information');
@@ -21,7 +21,7 @@ public function testPersistEntity()
         $this->assertEquals(3,$this->getCurrentQueryCount());
     }
 
-    public function testNoUpdateOnReadEncrypted()
+    public function testNoUpdateOnReadEncrypted(): void
     {
         $this->entityManager->beginTransaction();
         $this->assertEquals(1,$this->getCurrentQueryCount());
@@ -57,7 +57,7 @@ public function testNoUpdateOnReadEncrypted()
         $this->assertEquals(4,$this->getCurrentQueryCount());
     }
 
-    public function testStoredDataIsEncrypted()
+    public function testStoredDataIsEncrypted(): void
     {
         $user = new CascadeTarget();
         $user->setNotSecret('My public information');

Tests/Functional/DoctrineEncryptSubscriber/AbstractDoctrineEncryptSubscriberTestCase.php

@@ -11,7 +11,7 @@
 
 abstract class AbstractDoctrineEncryptSubscriberTestCase extends AbstractFunctionalTestCase
 {
-    public function testEncryptionHappensOnOnlyAnnotatedFields()
+    public function testEncryptionHappensOnOnlyAnnotatedFields(): void
     {
         $secret    = "It's a secret";
         $notSecret = "You're all welcome to know this.";
@@ -44,7 +44,7 @@ public function testEncryptionHappensOnOnlyAnnotatedFields()
         $this->assertEquals($secret, $decrypted);
     }
 
-    public function testEncryptionCascades()
+    public function testEncryptionCascades(): void
     {
         $secret        = "It's a secret";
         $notSecret     = "You're all welcome to know this.";
@@ -87,7 +87,7 @@ public function testEncryptionCascades()
      * @throws \Doctrine\DBAL\DBALException
      * @throws \Doctrine\ORM\OptimisticLockException
      */
-    public function testEncryptionDoesNotHappenWhenThereIsNoChange()
+    public function testEncryptionDoesNotHappenWhenThereIsNoChange(): void
     {
         $secret    = "It's a secret";
         $notSecret = "You're all welcome to know this.";
@@ -155,7 +155,7 @@ public function testEncryptionDoesNotHappenWhenThereIsNoChange()
 
     }
 
-    public function testEncryptionDoesHappenWhenASecretIsChanged()
+    public function testEncryptionDoesHappenWhenASecretIsChanged(): void
     {
         $secret    = "It's a secret";
         $notSecret = "You're all welcome to know this.";
@@ -198,4 +198,4 @@ public function testEncryptionDoesHappenWhenASecretIsChanged()
         $this->assertStringEndsWith('<ENC>', $shouldBeDifferentFromBefore); // is encrypted
         $this->assertNotEquals($originalEncryption, $shouldBeDifferentFromBefore);
     }
-}
+}

Tests/Unit/DependencyInjection/DoctrineEncryptExtensionTest.php

@@ -16,20 +16,20 @@ class DoctrineEncryptExtensionTest extends TestCase
      */
     private $extension;
 
-    protected function setUp()
+    protected function setUp(): void
     {
         $this->extension = new DoctrineEncryptExtension();
     }
 
-    public function testConfigLoadHalite()
+    public function testConfigLoadHalite(): void
     {
         $container = $this->createContainer();
         $this->extension->load([[]], $container);
 
         $this->assertSame(HaliteEncryptor::class, $container->getParameter('ambta_doctrine_encrypt.encryptor_class_name'));
     }
 
-    public function testConfigLoadDefuse()
+    public function testConfigLoadDefuse(): void
     {
         $container = $this->createContainer();
 
@@ -41,7 +41,7 @@ public function testConfigLoadDefuse()
         $this->assertSame(DefuseEncryptor::class, $container->getParameter('ambta_doctrine_encrypt.encryptor_class_name'));
     }
 
-    public function testConfigLoadCustom()
+    public function testConfigLoadCustom(): void
     {
         $container = $this->createContainer();
         $config = [
@@ -54,7 +54,7 @@ public function testConfigLoadCustom()
         $this->assertSame(self::class, $container->getParameter('ambta_doctrine_encrypt.encryptor_class_name'));
     }
 
-    private function createContainer()
+    private function createContainer(): ContainerBuilder
     {
         $container = new ContainerBuilder(
             new ParameterBag(['kernel.debug' => false])

Tests/Unit/Encryptors/DefuseEncryptorTest.php

@@ -9,7 +9,7 @@ class DefuseEncryptorTest extends TestCase
 {
     private const DATA = 'foobar';
 
-    public function testEncrypt()
+    public function testEncrypt(): void
     {
         $keyfile = __DIR__.'/fixtures/defuse.key';
         $key = file_get_contents($keyfile);
@@ -24,7 +24,7 @@ public function testEncrypt()
         $this->assertSame($key, $newkey, 'The key must not be modified');
     }
 
-    public function testGenerateKey()
+    public function testGenerateKey(): void
     {
         $keyfile = sys_get_temp_dir().'/defuse-'.md5(time());
         if (file_exists($keyfile)) {

Tests/Unit/Encryptors/HaliteEncryptorTest.php

@@ -9,7 +9,7 @@ class HaliteEncryptorTest extends TestCase
 {
     private const DATA = 'foobar';
 
-    public function testEncryptExtension()
+    public function testEncryptExtension(): void
     {
         if (! extension_loaded('sodium')) {
             $this->markTestSkipped('This test only runs when the sodium extension is enabled.');
@@ -27,7 +27,7 @@ public function testEncryptExtension()
         $this->assertSame($key, $newkey, 'The key must not be modified');
     }
 
-    public function testGenerateKey()
+    public function testGenerateKey(): void
     {
         if (! extension_loaded('sodium')) {
             $this->markTestSkipped('This test only runs when the sodium extension is enabled.');
@@ -46,7 +46,7 @@ public function testGenerateKey()
     }
 
 
-    public function testEncryptWithoutExtensionThrowsException()
+    public function testEncryptWithoutExtensionThrowsException(): void
     {
         if (extension_loaded('sodium')) {
             $this->markTestSkipped('This only runs when the sodium extension is disabled.');

Tests/Unit/Subscribers/DoctrineEncryptSubscriberTest.php

@@ -35,7 +35,7 @@ class DoctrineEncryptSubscriberTest extends TestCase
      */
     private $reader;
 
-    protected function setUp()
+    protected function setUp(): void
     {
         $this->encryptor = $this->createMock(EncryptorInterface::class);
         $this->encryptor
@@ -71,7 +71,7 @@ protected function setUp()
         $this->subscriber = new DoctrineEncryptSubscriber($this->reader, $this->encryptor);
     }
 
-    public function testSetRestorEncryptor()
+    public function testSetRestorEncryptor(): void
     {
         $replaceEncryptor = $this->createMock(EncryptorInterface::class);
 
@@ -82,7 +82,7 @@ public function testSetRestorEncryptor()
         $this->assertSame($this->encryptor, $this->subscriber->getEncryptor());
     }
 
-    public function testProcessFieldsEncrypt()
+    public function testProcessFieldsEncrypt(): void
     {
         $user = new User('David', 'Switzerland');
 
@@ -92,7 +92,7 @@ public function testProcessFieldsEncrypt()
         $this->assertStringStartsWith('encrypted-', $user->getAddress());
     }
 
-    public function testProcessFieldsEncryptExtend()
+    public function testProcessFieldsEncryptExtend(): void
     {
         $user = new ExtendedUser('David', 'Switzerland', 'extra');
 
@@ -103,7 +103,7 @@ public function testProcessFieldsEncryptExtend()
         $this->assertStringStartsWith('encrypted-', $user->extra);
     }
 
-    public function testProcessFieldsEncryptEmbedded()
+    public function testProcessFieldsEncryptEmbedded(): void
     {
         $withUser = new WithUser('Thing', 'foo', new User('David', 'Switzerland'));
 
@@ -115,7 +115,7 @@ public function testProcessFieldsEncryptEmbedded()
         $this->assertStringStartsWith('encrypted-', $withUser->user->getAddress());
     }
 
-    public function testProcessFieldsEncryptNull()
+    public function testProcessFieldsEncryptNull(): void
     {
         $user = new User('David', null);
 
@@ -125,7 +125,7 @@ public function testProcessFieldsEncryptNull()
         $this->assertNull($user->getAddress());
     }
 
-    public function testProcessFieldsNoEncryptor()
+    public function testProcessFieldsNoEncryptor(): void
     {
         $user = new User('David', 'Switzerland');
 
@@ -136,7 +136,7 @@ public function testProcessFieldsNoEncryptor()
         $this->assertSame('Switzerland', $user->getAddress());
     }
 
-    public function testProcessFieldsDecrypt()
+    public function testProcessFieldsDecrypt(): void
     {
         $user = new User('encrypted-David<ENC>', 'encrypted-Switzerland<ENC>');
 
@@ -146,7 +146,7 @@ public function testProcessFieldsDecrypt()
         $this->assertSame('Switzerland', $user->getAddress());
     }
 
-    public function testProcessFieldsDecryptExtended()
+    public function testProcessFieldsDecryptExtended(): void
     {
         $user = new ExtendedUser('encrypted-David<ENC>', 'encrypted-Switzerland<ENC>', 'encrypted-extra<ENC>');
 
@@ -157,7 +157,7 @@ public function testProcessFieldsDecryptExtended()
         $this->assertSame('extra', $user->extra);
     }
 
-    public function testProcessFieldsDecryptEmbedded()
+    public function testProcessFieldsDecryptEmbedded(): void
     {
         $withUser = new WithUser('encrypted-Thing<ENC>', 'foo', new User('encrypted-David<ENC>', 'encrypted-Switzerland<ENC>'));
 
@@ -169,7 +169,7 @@ public function testProcessFieldsDecryptEmbedded()
         $this->assertSame('Switzerland', $withUser->user->getAddress());
     }
 
-    public function testProcessFieldsDecryptNull()
+    public function testProcessFieldsDecryptNull(): void
     {
         $user = new User('encrypted-David<ENC>', null);
 
@@ -179,7 +179,7 @@ public function testProcessFieldsDecryptNull()
         $this->assertNull($user->getAddress());
     }
 
-    public function testProcessFieldsDecryptNonEncrypted()
+    public function testProcessFieldsDecryptNonEncrypted(): void
     {
         // no trailing <ENC> but somethint that our mock decrypt would change if called
         $user = new User('encrypted-David', 'encrypted-Switzerland');
@@ -193,7 +193,7 @@ public function testProcessFieldsDecryptNonEncrypted()
     /**
      * Test that fields are encrypted before flushing.
      */
-    public function testOnFlush()
+    public function testOnFlush(): void
     {
         $user = new User('David', 'Switzerland');
 
@@ -222,7 +222,7 @@ public function testOnFlush()
     /**
      * Test that fields are decrypted again after flushing
      */
-    public function testPostFlush()
+    public function testPostFlush(): void
     {
         $user = new User('encrypted-David<ENC>', 'encrypted-Switzerland<ENC>');