Use ::class notation where possible

src/Wallabag/ApiBundle/Controller/UserRestController.php

@@ -15,6 +15,7 @@ use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\Translation\TranslatorInterface;
 use Wallabag\ApiBundle\Entity\Client;
 use Wallabag\UserBundle\Entity\User;
+use Wallabag\UserBundle\Form\NewUserType;
 
 class UserRestController extends WallabagRestController
 {
@@ -63,7 +64,7 @@ class UserRestController extends WallabagRestController
         // user will be disabled BY DEFAULT to avoid spamming account to be enabled
         $user->setEnabled(false);
 
-        $form = $this->createForm('Wallabag\UserBundle\Form\NewUserType', $user, [
+        $form = $this->createForm(NewUserType::class, $user, [
             'csrf_protection' => false,
         ]);
 

src/Wallabag/ApiBundle/Form/Type/ClientType.php

@@ -9,6 +9,7 @@ use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\Extension\Core\Type\UrlType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\ApiBundle\Entity\Client;
 
 class ClientType extends AbstractType
 {
@@ -40,7 +41,7 @@ class ClientType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\ApiBundle\Entity\Client',
+            'data_class' => Client::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Controller/IgnoreOriginInstanceRuleController.php

@@ -11,6 +11,7 @@ use Symfony\Component\HttpFoundation\Session\SessionInterface;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Translation\TranslatorInterface;
 use Wallabag\CoreBundle\Entity\IgnoreOriginInstanceRule;
+use Wallabag\CoreBundle\Form\Type\IgnoreOriginInstanceRuleType;
 use Wallabag\CoreBundle\Repository\IgnoreOriginInstanceRuleRepository;
 
 /**
@@ -45,7 +46,7 @@ class IgnoreOriginInstanceRuleController extends Controller
     {
         $ignoreOriginInstanceRule = new IgnoreOriginInstanceRule();
 
-        $form = $this->createForm('Wallabag\CoreBundle\Form\Type\IgnoreOriginInstanceRuleType', $ignoreOriginInstanceRule);
+        $form = $this->createForm(IgnoreOriginInstanceRuleType::class, $ignoreOriginInstanceRule);
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {
@@ -77,7 +78,7 @@ class IgnoreOriginInstanceRuleController extends Controller
     public function editAction(Request $request, IgnoreOriginInstanceRule $ignoreOriginInstanceRule)
     {
         $deleteForm = $this->createDeleteForm($ignoreOriginInstanceRule);
-        $editForm = $this->createForm('Wallabag\CoreBundle\Form\Type\IgnoreOriginInstanceRuleType', $ignoreOriginInstanceRule);
+        $editForm = $this->createForm(IgnoreOriginInstanceRuleType::class, $ignoreOriginInstanceRule);
         $editForm->handleRequest($request);
 
         if ($editForm->isSubmitted() && $editForm->isValid()) {

src/Wallabag/CoreBundle/Controller/SiteCredentialController.php

@@ -12,6 +12,7 @@ use Symfony\Component\HttpFoundation\Session\SessionInterface;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Translation\TranslatorInterface;
 use Wallabag\CoreBundle\Entity\SiteCredential;
+use Wallabag\CoreBundle\Form\Type\SiteCredentialType;
 use Wallabag\CoreBundle\Helper\CryptoProxy;
 use Wallabag\CoreBundle\Repository\SiteCredentialRepository;
 use Wallabag\UserBundle\Entity\User;
@@ -52,7 +53,7 @@ class SiteCredentialController extends Controller
 
         $credential = new SiteCredential($this->getUser());
 
-        $form = $this->createForm('Wallabag\CoreBundle\Form\Type\SiteCredentialType', $credential);
+        $form = $this->createForm(SiteCredentialType::class, $credential);
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {
@@ -91,7 +92,7 @@ class SiteCredentialController extends Controller
         $this->checkUserAction($siteCredential);
 
         $deleteForm = $this->createDeleteForm($siteCredential);
-        $editForm = $this->createForm('Wallabag\CoreBundle\Form\Type\SiteCredentialType', $siteCredential);
+        $editForm = $this->createForm(SiteCredentialType::class, $siteCredential);
         $editForm->handleRequest($request);
 
         if ($editForm->isSubmitted() && $editForm->isValid()) {

src/Wallabag/CoreBundle/Form/Type/ConfigType.php

@@ -77,7 +77,7 @@ class ConfigType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Config',
+            'data_class' => Config::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/EditEntryType.php

@@ -8,6 +8,7 @@ use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\Extension\Core\Type\UrlType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\Entry;
 
 class EditEntryType extends AbstractType
 {
@@ -39,7 +40,7 @@ class EditEntryType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Entry',
+            'data_class' => Entry::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/FeedType.php

@@ -6,6 +6,7 @@ use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\Config;
 
 class FeedType extends AbstractType
 {
@@ -25,7 +26,7 @@ class FeedType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Config',
+            'data_class' => Config::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/IgnoreOriginInstanceRuleType.php

@@ -7,6 +7,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\IgnoreOriginInstanceRule;
 
 class IgnoreOriginInstanceRuleType extends AbstractType
 {
@@ -26,7 +27,7 @@ class IgnoreOriginInstanceRuleType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\IgnoreOriginInstanceRule',
+            'data_class' => IgnoreOriginInstanceRule::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/IgnoreOriginUserRuleType.php

@@ -7,6 +7,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\IgnoreOriginUserRule;
 
 class IgnoreOriginUserRuleType extends AbstractType
 {
@@ -26,7 +27,7 @@ class IgnoreOriginUserRuleType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\IgnoreOriginUserRule',
+            'data_class' => IgnoreOriginUserRule::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/NewEntryType.php

@@ -6,6 +6,7 @@ use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\UrlType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\Entry;
 
 class NewEntryType extends AbstractType
 {
@@ -23,7 +24,7 @@ class NewEntryType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Entry',
+            'data_class' => Entry::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/NewTagType.php

@@ -7,6 +7,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\Tag;
 
 class NewTagType extends AbstractType
 {
@@ -28,7 +29,7 @@ class NewTagType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Tag',
+            'data_class' => Tag::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/RenameTagType.php

@@ -6,6 +6,7 @@ use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\Tag;
 
 class RenameTagType extends AbstractType
 {
@@ -24,7 +25,7 @@ class RenameTagType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\Tag',
+            'data_class' => Tag::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/SiteCredentialType.php

@@ -8,6 +8,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\SiteCredential;
 
 class SiteCredentialType extends AbstractType
 {
@@ -33,7 +34,7 @@ class SiteCredentialType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\SiteCredential',
+            'data_class' => SiteCredential::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/TaggingRuleType.php

@@ -7,6 +7,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\CoreBundle\Entity\TaggingRule;
 use Wallabag\CoreBundle\Form\DataTransformer\StringToListTransformer;
 
 class TaggingRuleType extends AbstractType
@@ -35,7 +36,7 @@ class TaggingRuleType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\CoreBundle\Entity\TaggingRule',
+            'data_class' => TaggingRule::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Form/Type/UserInformationType.php

@@ -2,6 +2,7 @@
 
 namespace Wallabag\CoreBundle\Form\Type;
 
+use FOS\UserBundle\Form\Type\RegistrationFormType;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
 use Symfony\Component\Form\Extension\Core\Type\EmailType;
@@ -9,6 +10,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\UserBundle\Entity\User;
 
 class UserInformationType extends AbstractType
 {
@@ -40,13 +42,13 @@ class UserInformationType extends AbstractType
 
     public function getParent()
     {
-        return 'FOS\UserBundle\Form\Type\RegistrationFormType';
+        return RegistrationFormType::class;
     }
 
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\UserBundle\Entity\User',
+            'data_class' => User::class,
         ]);
     }
 

src/Wallabag/CoreBundle/Helper/DownloadImages.php

@@ -162,7 +162,7 @@ class DownloadImages
         switch ($ext) {
             case 'gif':
                 // use Imagick if available to keep GIF animation
-                if (class_exists('\\Imagick')) {
+                if (class_exists(\Imagick::class)) {
                     try {
                         $imagick = new \Imagick();
                         $imagick->readImageBlob($res->getBody());

src/Wallabag/UserBundle/Controller/ManageController.php

@@ -18,7 +18,9 @@ use Symfony\Component\HttpFoundation\Session\SessionInterface;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Translation\TranslatorInterface;
 use Wallabag\UserBundle\Entity\User;
+use Wallabag\UserBundle\Form\NewUserType;
 use Wallabag\UserBundle\Form\SearchUserType;
+use Wallabag\UserBundle\Form\UserType;
 
 /**
  * User controller.
@@ -38,7 +40,7 @@ class ManageController extends Controller
         // enable created user by default
         $user->setEnabled(true);
 
-        $form = $this->createForm('Wallabag\UserBundle\Form\NewUserType', $user);
+        $form = $this->createForm(NewUserType::class, $user);
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {
@@ -72,7 +74,7 @@ class ManageController extends Controller
         $userManager = $this->container->get(UserManagerInterface::class);
 
         $deleteForm = $this->createDeleteForm($user);
-        $form = $this->createForm('Wallabag\UserBundle\Form\UserType', $user);
+        $form = $this->createForm(UserType::class, $user);
         $form->handleRequest($request);
 
         // `googleTwoFactor` isn't a field within the User entity, we need to define it's value in a different way

src/Wallabag/UserBundle/Form/NewUserType.php

@@ -12,6 +12,7 @@ use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Validator\Constraints\Length;
 use Symfony\Component\Validator\Constraints\NotBlank;
+use Wallabag\UserBundle\Entity\User;
 
 class NewUserType extends AbstractType
 {
@@ -48,7 +49,7 @@ class NewUserType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\UserBundle\Entity\User',
+            'data_class' => User::class,
         ]);
     }
 

src/Wallabag/UserBundle/Form/UserType.php

@@ -9,6 +9,7 @@ use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Wallabag\UserBundle\Entity\User;
 
 class UserType extends AbstractType
 {
@@ -49,7 +50,7 @@ class UserType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'data_class' => 'Wallabag\UserBundle\Entity\User',
+            'data_class' => User::class,
         ]);
     }
 }

tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php

@@ -174,7 +174,7 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $author = $content->getPublishedBy();
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertSame($this->url, $content->getUrl());
         $this->assertStringContainsString('la cryptomonnaie de Facebook', $content->getTitle());
         $this->assertSame('fr', $content->getLanguage());
@@ -246,7 +246,7 @@ class EntryControllerTest extends WallabagCoreTestCase
             ->getRepository(Entry::class)
             ->findByUrlAndUserId($url, $this->getLoggedInUserId());
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $authors = $content->getPublishedBy();
         $this->assertSame('2017-04-05', $content->getPublishedAt()->format('Y-m-d'));
         $this->assertSame('fr', $content->getLanguage());
@@ -1217,7 +1217,7 @@ class EntryControllerTest extends WallabagCoreTestCase
             ->getRepository(Entry::class)
             ->findByUrlAndUserId($url, $this->getLoggedInUserId());
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $entry);
+        $this->assertInstanceOf(Entry::class, $entry);
         $this->assertSame($url, $entry->getUrl());
         $this->assertStringContainsString('Judo', $entry->getTitle());
         // instead of checking for the filename (which might change) check that the image is now local
@@ -1536,7 +1536,7 @@ class EntryControllerTest extends WallabagCoreTestCase
             ->getRepository(Entry::class)
             ->findByUrlAndUserId($url, $this->getLoggedInUserId());
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertSame($url, $content->getUrl());
         $this->assertSame($expectedLanguage, $content->getLanguage());
     }
@@ -1587,7 +1587,7 @@ class EntryControllerTest extends WallabagCoreTestCase
             ->getRepository(Entry::class)
             ->findByUrlAndUserId($url, $this->getLoggedInUserId());
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertSame('Quand Manille manœuvre', $content->getTitle());
 
         $client->getContainer()->get(Config::class)->set('restricted_access', 0);

tests/Wallabag/CoreBundle/Event/Listener/LocaleListenerTest.php

@@ -78,7 +78,7 @@ class LocaleListenerTest extends TestCase
 
     private function getEvent(Request $request)
     {
-        $kernel = $this->getMockBuilder('Symfony\Component\HttpKernel\HttpKernelInterface')
+        $kernel = $this->getMockBuilder(HttpKernelInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/CoreBundle/Event/Subscriber/TablePrefixSubscriberTest.php

@@ -6,31 +6,33 @@ use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Platforms\MySqlPlatform;
 use Doctrine\DBAL\Platforms\PostgreSqlPlatform;
 use Doctrine\DBAL\Platforms\SqlitePlatform;
+use Doctrine\ORM\EntityManager;
 use Doctrine\ORM\Event\LoadClassMetadataEventArgs;
 use Doctrine\ORM\Mapping\ClassMetadata;
 use PHPUnit\Framework\TestCase;
 use Wallabag\CoreBundle\Event\Subscriber\TablePrefixSubscriber;
+use Wallabag\UserBundle\Entity\User;
 
 class TablePrefixSubscriberTest extends TestCase
 {
     public function dataForPrefix()
     {
         return [
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'wallabag_user', '"wallabag_user"', new PostgreSqlPlatform()],
+            ['wallabag_', User::class, '`user`', 'user', 'wallabag_user', '"wallabag_user"', new PostgreSqlPlatform()],
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'wallabag_user', '`wallabag_user`', new MySqlPlatform()],
+            ['wallabag_', User::class, '`user`', 'user', 'wallabag_user', '`wallabag_user`', new MySqlPlatform()],
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'wallabag_user', '"wallabag_user"', new SqlitePlatform()],
+            ['wallabag_', User::class, '`user`', 'user', 'wallabag_user', '"wallabag_user"', new SqlitePlatform()],
 
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'wallabag_user', 'wallabag_user', new PostgreSqlPlatform()],
+            ['wallabag_', User::class, 'user', 'user', 'wallabag_user', 'wallabag_user', new PostgreSqlPlatform()],
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'wallabag_user', 'wallabag_user', new MySqlPlatform()],
+            ['wallabag_', User::class, 'user', 'user', 'wallabag_user', 'wallabag_user', new MySqlPlatform()],
-            ['wallabag_', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'wallabag_user', 'wallabag_user', new SqlitePlatform()],
+            ['wallabag_', User::class, 'user', 'user', 'wallabag_user', 'wallabag_user', new SqlitePlatform()],
 
-            ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '"user"', new PostgreSqlPlatform()],
+            ['', User::class, '`user`', 'user', 'user', '"user"', new PostgreSqlPlatform()],
-            ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '`user`', new MySqlPlatform()],
+            ['', User::class, '`user`', 'user', 'user', '`user`', new MySqlPlatform()],
-            ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '"user"', new SqlitePlatform()],
+            ['', User::class, '`user`', 'user', 'user', '"user"', new SqlitePlatform()],
 
-            ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new PostgreSqlPlatform()],
+            ['', User::class, 'user', 'user', 'user', 'user', new PostgreSqlPlatform()],
-            ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new MySqlPlatform()],
+            ['', User::class, 'user', 'user', 'user', 'user', new MySqlPlatform()],
-            ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new SqlitePlatform()],
+            ['', User::class, 'user', 'user', 'user', 'user', new SqlitePlatform()],
         ];
     }
 
@@ -39,7 +41,7 @@ class TablePrefixSubscriberTest extends TestCase
      */
     public function testPrefix($prefix, $entityName, $tableName, $tableNameExpected, $finalTableName, $finalTableNameQuoted, $platform)
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -63,7 +65,7 @@ class TablePrefixSubscriberTest extends TestCase
      */
     public function testSubscribedEvents($prefix, $entityName, $tableName, $tableNameExpected, $finalTableName, $finalTableNameQuoted, $platform)
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -85,7 +87,7 @@ class TablePrefixSubscriberTest extends TestCase
 
     public function testPrefixManyToMany()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilderTest.php

@@ -2,6 +2,7 @@
 
 namespace Tests\Wallabag\CoreBundle\GuzzleSiteAuthenticator;
 
+use Graby\SiteConfig\ConfigBuilder;
 use Graby\SiteConfig\SiteConfig as GrabySiteConfig;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
@@ -10,6 +11,7 @@ use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
 use Tests\Wallabag\CoreBundle\WallabagCoreTestCase;
 use Wallabag\CoreBundle\GuzzleSiteAuthenticator\GrabySiteConfigBuilder;
 use Wallabag\CoreBundle\Repository\SiteCredentialRepository;
+use Wallabag\UserBundle\Entity\User;
 
 class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
 {
@@ -17,7 +19,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
 
     public function testBuildConfigExists()
     {
-        $grabyConfigBuilderMock = $this->getMockBuilder('Graby\SiteConfig\ConfigBuilder')
+        $grabyConfigBuilderMock = $this->getMockBuilder(ConfigBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -38,7 +40,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
         $handler = new TestHandler();
         $logger->pushHandler($handler);
 
-        $siteCrentialRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\SiteCredentialRepository')
+        $siteCrentialRepo = $this->getMockBuilder(SiteCredentialRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
         $siteCrentialRepo->expects($this->once())
@@ -46,7 +48,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
             ->with(['api.example.com', '.example.com'], 1)
             ->willReturn(['username' => 'foo', 'password' => 'bar']);
 
-        $user = $this->getMockBuilder('Wallabag\UserBundle\Entity\User')
+        $user = $this->getMockBuilder(User::class)
             ->disableOriginalConstructor()
             ->getMock();
         $user->expects($this->once())
@@ -84,7 +86,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
 
     public function testBuildConfigDoesntExist()
     {
-        $grabyConfigBuilderMock = $this->getMockBuilder('\Graby\SiteConfig\ConfigBuilder')
+        $grabyConfigBuilderMock = $this->getMockBuilder(ConfigBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -97,7 +99,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
         $handler = new TestHandler();
         $logger->pushHandler($handler);
 
-        $siteCrentialRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\SiteCredentialRepository')
+        $siteCrentialRepo = $this->getMockBuilder(SiteCredentialRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
         $siteCrentialRepo->expects($this->once())
@@ -105,7 +107,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
             ->with(['unknown.com', '.com'], 1)
             ->willReturn(null);
 
-        $user = $this->getMockBuilder('Wallabag\UserBundle\Entity\User')
+        $user = $this->getMockBuilder(User::class)
             ->disableOriginalConstructor()
             ->getMock();
         $user->expects($this->once())
@@ -135,7 +137,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
 
     public function testBuildConfigWithBadExtraFields()
     {
-        $grabyConfigBuilderMock = $this->getMockBuilder('Graby\SiteConfig\ConfigBuilder')
+        $grabyConfigBuilderMock = $this->getMockBuilder(ConfigBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -156,7 +158,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
         $handler = new TestHandler();
         $logger->pushHandler($handler);
 
-        $siteCrentialRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\SiteCredentialRepository')
+        $siteCrentialRepo = $this->getMockBuilder(SiteCredentialRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
         $siteCrentialRepo->expects($this->once())
@@ -164,7 +166,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
             ->with(['example.com', '.com'], 1)
             ->willReturn(['username' => 'foo', 'password' => 'bar']);
 
-        $user = $this->getMockBuilder('Wallabag\UserBundle\Entity\User')
+        $user = $this->getMockBuilder(User::class)
             ->disableOriginalConstructor()
             ->getMock();
         $user->expects($this->once())
@@ -202,7 +204,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
 
     public function testBuildConfigUserNotDefined()
     {
-        $grabyConfigBuilderMock = $this->getMockBuilder('\Graby\SiteConfig\ConfigBuilder')
+        $grabyConfigBuilderMock = $this->getMockBuilder(ConfigBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -215,7 +217,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
         $handler = new TestHandler();
         $logger->pushHandler($handler);
 
-        $siteCrentialRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\SiteCredentialRepository')
+        $siteCrentialRepo = $this->getMockBuilder(SiteCredentialRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -265,7 +267,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
      */
     public function testBuildConfigWithDbAccess($host, $expectedUsername = null, $expectedPassword = null)
     {
-        $grabyConfigBuilderMock = $this->getMockBuilder('Graby\SiteConfig\ConfigBuilder')
+        $grabyConfigBuilderMock = $this->getMockBuilder(ConfigBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -282,7 +284,7 @@ class GrabySiteConfigBuilderTest extends WallabagCoreTestCase
             ->with($host)
             ->willReturn($grabySiteConfig);
 
-        $user = $this->getMockBuilder('Wallabag\UserBundle\Entity\User')
+        $user = $this->getMockBuilder(User::class)
             ->disableOriginalConstructor()
             ->getMock();
         $user->expects($this->once())

tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php

@@ -28,7 +28,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -67,7 +67,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -106,7 +106,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -150,7 +150,7 @@ class ContentProxyTest extends TestCase
         $ruleBasedIgnoreOriginProcessor->expects($this->once())
             ->method('process');
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -195,7 +195,7 @@ class ContentProxyTest extends TestCase
         $ruleBasedIgnoreOriginProcessor->expects($this->once())
             ->method('process');
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -240,7 +240,7 @@ class ContentProxyTest extends TestCase
         $ruleBasedIgnoreOriginProcessor->expects($this->once())
             ->method('process');
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -284,7 +284,7 @@ class ContentProxyTest extends TestCase
         $ruleBasedIgnoreOriginProcessor->expects($this->once())
             ->method('process');
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -333,7 +333,7 @@ class ContentProxyTest extends TestCase
             ->method('validate')
             ->willReturn(new ConstraintViolationList([new ConstraintViolation('oops', 'oops', [], 'oops', 'language', 'dontexist')]));
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -383,7 +383,7 @@ class ContentProxyTest extends TestCase
                 new ConstraintViolationList([new ConstraintViolation('oops', 'oops', [], 'oops', 'url', 'https://')])
             ));
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -635,7 +635,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -678,7 +678,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -717,7 +717,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -755,7 +755,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -793,7 +793,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -831,7 +831,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();
@@ -870,7 +870,7 @@ class ContentProxyTest extends TestCase
 
         $ruleBasedIgnoreOriginProcessor = $this->getRuleBasedIgnoreOriginProcessorMock();
 
-        $graby = $this->getMockBuilder('Graby\Graby')
+        $graby = $this->getMockBuilder(Graby::class)
             ->setMethods(['fetchContent'])
             ->disableOriginalConstructor()
             ->getMock();

tests/Wallabag/CoreBundle/Helper/RedirectTest.php

@@ -3,6 +3,7 @@
 namespace Tests\Wallabag\CoreBundle\Helper;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Routing\Router;
 use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage;
 use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
 use Wallabag\CoreBundle\Entity\Config;
@@ -22,7 +23,7 @@ class RedirectTest extends TestCase
 
     protected function setUp(): void
     {
-        $this->routerMock = $this->getMockBuilder('Symfony\Component\Routing\Router')
+        $this->routerMock = $this->getMockBuilder(Router::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/CoreBundle/Helper/RuleBasedIgnoreOriginProcessorTest.php

@@ -5,11 +5,13 @@ namespace Tests\Wallabag\CoreBundle\Helper;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use RulerZ\RulerZ;
 use Wallabag\CoreBundle\Entity\Config;
 use Wallabag\CoreBundle\Entity\Entry;
 use Wallabag\CoreBundle\Entity\IgnoreOriginInstanceRule;
 use Wallabag\CoreBundle\Entity\IgnoreOriginUserRule;
 use Wallabag\CoreBundle\Helper\RuleBasedIgnoreOriginProcessor;
+use Wallabag\CoreBundle\Repository\IgnoreOriginInstanceRuleRepository;
 use Wallabag\UserBundle\Entity\User;
 
 class RuleBasedIgnoreOriginProcessorTest extends TestCase
@@ -193,14 +195,14 @@ class RuleBasedIgnoreOriginProcessorTest extends TestCase
 
     private function getRulerZMock()
     {
-        return $this->getMockBuilder('RulerZ\RulerZ')
+        return $this->getMockBuilder(RulerZ::class)
             ->disableOriginalConstructor()
             ->getMock();
     }
 
     private function getIgnoreOriginInstanceRuleRepositoryMock()
     {
-        return $this->getMockBuilder('Wallabag\CoreBundle\Repository\IgnoreOriginInstanceRuleRepository')
+        return $this->getMockBuilder(IgnoreOriginInstanceRuleRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
     }

tests/Wallabag/CoreBundle/Helper/RuleBasedTaggerTest.php

@@ -2,14 +2,19 @@
 
 namespace Tests\Wallabag\CoreBundle\Helper;
 
+use Doctrine\ORM\AbstractQuery;
+use Doctrine\ORM\QueryBuilder;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use RulerZ\RulerZ;
 use Wallabag\CoreBundle\Entity\Config;
 use Wallabag\CoreBundle\Entity\Entry;
 use Wallabag\CoreBundle\Entity\Tag;
 use Wallabag\CoreBundle\Entity\TaggingRule;
 use Wallabag\CoreBundle\Helper\RuleBasedTagger;
+use Wallabag\CoreBundle\Repository\EntryRepository;
+use Wallabag\CoreBundle\Repository\TagRepository;
 use Wallabag\UserBundle\Entity\User;
 
 class RuleBasedTaggerTest extends TestCase
@@ -202,7 +207,7 @@ class RuleBasedTaggerTest extends TestCase
             ->method('satisfies')
             ->willReturn(true);
 
-        $query = $this->getMockBuilder('Doctrine\ORM\AbstractQuery')
+        $query = $this->getMockBuilder(AbstractQuery::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -211,7 +216,7 @@ class RuleBasedTaggerTest extends TestCase
             ->method('getResult')
             ->willReturn([new Entry($user), new Entry($user)]);
 
-        $qb = $this->getMockBuilder('Doctrine\ORM\QueryBuilder')
+        $qb = $this->getMockBuilder(QueryBuilder::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -263,21 +268,21 @@ class RuleBasedTaggerTest extends TestCase
 
     private function getRulerZMock()
     {
-        return $this->getMockBuilder('RulerZ\RulerZ')
+        return $this->getMockBuilder(RulerZ::class)
             ->disableOriginalConstructor()
             ->getMock();
     }
 
     private function getTagRepositoryMock()
     {
-        return $this->getMockBuilder('Wallabag\CoreBundle\Repository\TagRepository')
+        return $this->getMockBuilder(TagRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
     }
 
     private function getEntryRepositoryMock()
     {
-        return $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        return $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
     }

tests/Wallabag/CoreBundle/ParamConverter/UsernameFeedTokenConverterTest.php

@@ -2,12 +2,16 @@
 
 namespace Tests\Wallabag\CoreBundle\ParamConverter;
 
+use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Common\Persistence\Mapping\ClassMetadata;
+use Doctrine\Common\Persistence\ObjectManager;
 use PHPUnit\Framework\TestCase;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Wallabag\CoreBundle\ParamConverter\UsernameFeedTokenConverter;
 use Wallabag\UserBundle\Entity\User;
+use Wallabag\UserBundle\Repository\UserRepository;
 
 class UsernameFeedTokenConverterTest extends TestCase
 {
@@ -21,7 +25,7 @@ class UsernameFeedTokenConverterTest extends TestCase
 
     public function testSupportsWithNoRegistryManagers()
     {
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -37,7 +41,7 @@ class UsernameFeedTokenConverterTest extends TestCase
 
     public function testSupportsWithNoConfigurationClass()
     {
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -53,7 +57,7 @@ class UsernameFeedTokenConverterTest extends TestCase
 
     public function testSupportsWithNotTheGoodClass()
     {
-        $meta = $this->getMockBuilder('Doctrine\Common\Persistence\Mapping\ClassMetadata')
+        $meta = $this->getMockBuilder(ClassMetadata::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -61,7 +65,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->method('getName')
             ->willReturn('nothingrelated');
 
-        $em = $this->getMockBuilder('Doctrine\Common\Persistence\ObjectManager')
+        $em = $this->getMockBuilder(ObjectManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -70,7 +74,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with('superclass')
             ->willReturn($meta);
 
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -91,15 +95,15 @@ class UsernameFeedTokenConverterTest extends TestCase
 
     public function testSupportsWithGoodClass()
     {
-        $meta = $this->getMockBuilder('Doctrine\Common\Persistence\Mapping\ClassMetadata')
+        $meta = $this->getMockBuilder(ClassMetadata::class)
             ->disableOriginalConstructor()
             ->getMock();
 
         $meta->expects($this->once())
             ->method('getName')
-            ->willReturn('Wallabag\UserBundle\Entity\User');
+            ->willReturn(User::class);
 
-        $em = $this->getMockBuilder('Doctrine\Common\Persistence\ObjectManager')
+        $em = $this->getMockBuilder(ObjectManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -108,7 +112,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with(User::class)
             ->willReturn($meta);
 
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -142,7 +146,7 @@ class UsernameFeedTokenConverterTest extends TestCase
         $this->expectException(NotFoundHttpException::class);
         $this->expectExceptionMessage('User not found');
 
-        $repo = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $repo = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -151,7 +155,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with('test', 'test')
             ->willReturn(null);
 
-        $em = $this->getMockBuilder('Doctrine\Common\Persistence\ObjectManager')
+        $em = $this->getMockBuilder(ObjectManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -160,7 +164,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with(User::class)
             ->willReturn($repo);
 
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -180,7 +184,7 @@ class UsernameFeedTokenConverterTest extends TestCase
     {
         $user = new User();
 
-        $repo = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $repo = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -189,7 +193,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with('test', 'test')
             ->willReturn($user);
 
-        $em = $this->getMockBuilder('Doctrine\Common\Persistence\ObjectManager')
+        $em = $this->getMockBuilder(ObjectManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -198,7 +202,7 @@ class UsernameFeedTokenConverterTest extends TestCase
             ->with(User::class)
             ->willReturn($repo);
 
-        $registry = $this->getMockBuilder('Doctrine\Common\Persistence\ManagerRegistry')
+        $registry = $this->getMockBuilder(ManagerRegistry::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/CoreBundle/Twig/WallabagExtensionTest.php

@@ -3,25 +3,29 @@
 namespace Tests\Wallabag\CoreBundle\Twig;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
+use Symfony\Component\Translation\TranslatorInterface;
+use Wallabag\CoreBundle\Repository\EntryRepository;
+use Wallabag\CoreBundle\Repository\TagRepository;
 use Wallabag\CoreBundle\Twig\WallabagExtension;
 
 class WallabagExtensionTest extends TestCase
 {
     public function testRemoveWww()
     {
-        $entryRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepository = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tagRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\TagRepository')
+        $tagRepository = $this->getMockBuilder(TagRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tokenStorage = $this->getMockBuilder('Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface')
+        $tokenStorage = $this->getMockBuilder(TokenStorageInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $translator = $this->getMockBuilder('Symfony\Component\Translation\TranslatorInterface')
+        $translator = $this->getMockBuilder(TranslatorInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -34,19 +38,19 @@ class WallabagExtensionTest extends TestCase
 
     public function testRemoveScheme()
     {
-        $entryRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepository = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tagRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\TagRepository')
+        $tagRepository = $this->getMockBuilder(TagRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tokenStorage = $this->getMockBuilder('Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface')
+        $tokenStorage = $this->getMockBuilder(TokenStorageInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $translator = $this->getMockBuilder('Symfony\Component\Translation\TranslatorInterface')
+        $translator = $this->getMockBuilder(TranslatorInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -59,19 +63,19 @@ class WallabagExtensionTest extends TestCase
 
     public function testRemoveSchemeAndWww()
     {
-        $entryRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepository = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tagRepository = $this->getMockBuilder('Wallabag\CoreBundle\Repository\TagRepository')
+        $tagRepository = $this->getMockBuilder(TagRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $tokenStorage = $this->getMockBuilder('Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface')
+        $tokenStorage = $this->getMockBuilder(TokenStorageInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $translator = $this->getMockBuilder('Symfony\Component\Translation\TranslatorInterface')
+        $translator = $this->getMockBuilder(TranslatorInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Command/RedisWorkerCommandTest.php

@@ -52,7 +52,7 @@ class RedisWorkerCommandTest extends WallabagCoreTestCase
         $application->add(new RedisWorkerCommand());
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $application->getKernel()->getContainer()->set(Client::class, $redisMock);
 

tests/Wallabag/ImportBundle/Consumer/AMQPEntryConsumerTest.php

@@ -2,17 +2,21 @@
 
 namespace Tests\Wallabag\ImportBundle\Consumer;
 
+use Doctrine\ORM\EntityManager;
 use PhpAmqpLib\Message\AMQPMessage;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
 use Wallabag\ImportBundle\Consumer\AMQPEntryConsumer;
+use Wallabag\ImportBundle\Import\AbstractImport;
 use Wallabag\UserBundle\Entity\User;
+use Wallabag\UserBundle\Repository\UserRepository;
 
 class AMQPEntryConsumerTest extends TestCase
 {
     public function testMessageOk()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -87,7 +91,7 @@ JSON;
         $user = new User();
         $entry = new Entry($user);
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -98,7 +102,7 @@ JSON;
             ->with(1)
             ->willReturn($user);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -113,7 +117,7 @@ JSON;
             ->with(json_decode($body, true))
             ->willReturn($entry);
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -135,7 +139,7 @@ JSON;
 
     public function testMessageWithBadUser()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -152,7 +156,7 @@ JSON;
         $user = new User();
         $entry = new Entry($user);
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -163,11 +167,11 @@ JSON;
             ->with(123)
             ->willReturn(null);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -191,7 +195,7 @@ JSON;
 
     public function testMessageWithEntryProcessed()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -207,7 +211,7 @@ JSON;
 
         $user = new User();
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -218,7 +222,7 @@ JSON;
             ->with(123)
             ->willReturn($user);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -233,7 +237,7 @@ JSON;
             ->with(json_decode($body, true))
             ->willReturn(null);
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Consumer/RedisEntryConsumerTest.php

@@ -2,16 +2,20 @@
 
 namespace Tests\Wallabag\ImportBundle\Consumer;
 
+use Doctrine\ORM\EntityManager;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
 use Wallabag\ImportBundle\Consumer\RedisEntryConsumer;
+use Wallabag\ImportBundle\Import\AbstractImport;
 use Wallabag\UserBundle\Entity\User;
+use Wallabag\UserBundle\Repository\UserRepository;
 
 class RedisEntryConsumerTest extends TestCase
 {
     public function testMessageOk()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -86,7 +90,7 @@ JSON;
         $user = new User();
         $entry = new Entry($user);
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -97,7 +101,7 @@ JSON;
             ->with(1)
             ->willReturn($user);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -112,7 +116,7 @@ JSON;
             ->with(json_decode($body, true))
             ->willReturn($entry);
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -134,7 +138,7 @@ JSON;
 
     public function testMessageWithBadUser()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -151,7 +155,7 @@ JSON;
         $user = new User();
         $entry = new Entry($user);
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -162,11 +166,11 @@ JSON;
             ->with(123)
             ->willReturn(null);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -188,7 +192,7 @@ JSON;
 
     public function testMessageWithEntryProcessed()
     {
-        $em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -204,7 +208,7 @@ JSON;
 
         $user = new User();
 
-        $userRepository = $this->getMockBuilder('Wallabag\UserBundle\Repository\UserRepository')
+        $userRepository = $this->getMockBuilder(UserRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -215,7 +219,7 @@ JSON;
             ->with(123)
             ->willReturn($user);
 
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\AbstractImport')
+        $import = $this->getMockBuilder(AbstractImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -230,7 +234,7 @@ JSON;
             ->with(json_decode($body, true))
             ->willReturn(null);
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Controller/ChromeControllerTest.php

@@ -122,7 +122,7 @@ class ChromeControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://www.20minutes.fr is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for https://www.20minutes.fr is ok');
         $this->assertCount(1, $content->getTags());

tests/Wallabag/ImportBundle/Controller/DeliciousControllerTest.php

@@ -122,7 +122,7 @@ class DeliciousControllerTest extends WallabagCoreTestCase
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
         $this->assertStringContainsString('flashes.import.notice.summary', $body[0]);
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
 
         $tags = $content->getTagsLabel();
         $this->assertContains('osx', $tags, 'It includes the "osx" tag');
@@ -161,7 +161,7 @@ class DeliciousControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content1);
+        $this->assertInstanceOf(Entry::class, $content1);
 
         $content2 = $client->getContainer()
             ->get(EntityManagerInterface::class)
@@ -171,7 +171,7 @@ class DeliciousControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content2);
+        $this->assertInstanceOf(Entry::class, $content2);
 
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
         $this->assertStringContainsString('flashes.import.notice.summary', $body[0]);

tests/Wallabag/ImportBundle/Controller/ElcuratorControllerTest.php

@@ -123,7 +123,7 @@ class ElcuratorControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
 
         $this->assertStringContainsString('Qualité de code - Intégration de php-git-hooks dans Symfony2', $content->getTitle());
         $this->assertSame('2015-09-09', $content->getCreatedAt()->format('Y-m-d'));

tests/Wallabag/ImportBundle/Controller/FirefoxControllerTest.php

@@ -122,7 +122,7 @@ class FirefoxControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for http://lexpansion.lexpress.fr is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for http://lexpansion.lexpress.fr is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for http://lexpansion.lexpress.fr is ok');
@@ -136,7 +136,7 @@ class FirefoxControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for https://www.lemonde.fr is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://www.lemonde.fr is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for https://www.lemonde.fr is ok');

tests/Wallabag/ImportBundle/Controller/InstapaperControllerTest.php

@@ -122,7 +122,7 @@ class InstapaperControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
 
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for https://www.liberation.fr is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://www.liberation.fr is ok');

tests/Wallabag/ImportBundle/Controller/PinboardControllerTest.php

@@ -122,7 +122,7 @@ class PinboardControllerTest extends WallabagCoreTestCase
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
         $this->assertStringContainsString('flashes.import.notice.summary', $body[0]);
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for https://ma.ttias.be is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://ma.ttias.be is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for https://ma.ttias.be is ok');
@@ -166,7 +166,7 @@ class PinboardControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content1);
+        $this->assertInstanceOf(Entry::class, $content1);
         $this->assertTrue($content1->isArchived());
 
         $content2 = $client->getContainer()
@@ -177,7 +177,7 @@ class PinboardControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content2);
+        $this->assertInstanceOf(Entry::class, $content2);
         $this->assertTrue($content2->isArchived());
 
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));

tests/Wallabag/ImportBundle/Controller/PocketControllerTest.php

@@ -66,7 +66,7 @@ class PocketControllerTest extends WallabagCoreTestCase
         $this->logInAs('admin');
         $client = $this->getClient();
 
-        $pocketImport = $this->getMockBuilder('Wallabag\ImportBundle\Import\PocketImport')
+        $pocketImport = $this->getMockBuilder(PocketImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -88,7 +88,7 @@ class PocketControllerTest extends WallabagCoreTestCase
         $this->logInAs('admin');
         $client = $this->getClient();
 
-        $pocketImport = $this->getMockBuilder('Wallabag\ImportBundle\Import\PocketImport')
+        $pocketImport = $this->getMockBuilder(PocketImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -111,7 +111,7 @@ class PocketControllerTest extends WallabagCoreTestCase
         $this->logInAs('admin');
         $client = $this->getClient();
 
-        $pocketImport = $this->getMockBuilder('Wallabag\ImportBundle\Import\PocketImport')
+        $pocketImport = $this->getMockBuilder(PocketImport::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Controller/ReadabilityControllerTest.php

@@ -122,7 +122,7 @@ class ReadabilityControllerTest extends WallabagCoreTestCase
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
         $this->assertStringContainsString('flashes.import.notice.summary', $body[0]);
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for https://www.20minutes.fr is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://www.20minutes.fr is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for https://www.20minutes.fr is ok');
@@ -164,7 +164,7 @@ class ReadabilityControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content1);
+        $this->assertInstanceOf(Entry::class, $content1);
         $this->assertTrue($content1->isArchived());
 
         $content2 = $client->getContainer()
@@ -175,7 +175,7 @@ class ReadabilityControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content2);
+        $this->assertInstanceOf(Entry::class, $content2);
         $this->assertTrue($content2->isArchived());
 
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));

tests/Wallabag/ImportBundle/Controller/WallabagV1ControllerTest.php

@@ -123,7 +123,7 @@ class WallabagV1ControllerTest extends WallabagCoreTestCase
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
         $this->assertStringContainsString('flashes.import.notice.summary', $body[0]);
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertEmpty($content->getMimetype(), 'Mimetype for http://www.framablog.org is empty');
         $this->assertSame($content->getPreviewPicture(), 'http://www.framablog.org/public/_img/framablog/wallaby_baby.jpg');
         $this->assertEmpty($content->getLanguage(), 'Language for http://www.framablog.org is empty');
@@ -165,7 +165,7 @@ class WallabagV1ControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content1);
+        $this->assertInstanceOf(Entry::class, $content1);
         $this->assertTrue($content1->isArchived());
 
         $content2 = $client->getContainer()
@@ -176,7 +176,7 @@ class WallabagV1ControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content2);
+        $this->assertInstanceOf(Entry::class, $content2);
         $this->assertTrue($content2->isArchived());
 
         $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));

tests/Wallabag/ImportBundle/Controller/WallabagV2ControllerTest.php

@@ -123,7 +123,7 @@ class WallabagV2ControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
 
         // empty because it wasn't re-imported
         $this->assertEmpty($content->getMimetype(), 'Mimetype for https://www.liberation.fr is empty');
@@ -142,7 +142,7 @@ class WallabagV2ControllerTest extends WallabagCoreTestCase
                 $this->getLoggedInUserId()
             );
 
-        $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+        $this->assertInstanceOf(Entry::class, $content);
         $this->assertNotEmpty($content->getMimetype(), 'Mimetype for https://www.mediapart.fr is ok');
         $this->assertNotEmpty($content->getPreviewPicture(), 'Preview picture for https://www.mediapart.fr is ok');
         $this->assertNotEmpty($content->getLanguage(), 'Language for https://www.mediapart.fr is ok');

tests/Wallabag/ImportBundle/Import/ChromeImportTest.php

@@ -2,12 +2,18 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\ChromeImport;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -34,7 +40,7 @@ class ChromeImportTest extends TestCase
         $chromeImport = $this->getChromeImport(false, 1);
         $chromeImport->setFilepath(__DIR__ . '/../fixtures/chrome-bookmarks');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -47,7 +53,7 @@ class ChromeImportTest extends TestCase
             ->method('getRepository')
             ->willReturn($entryRepo);
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -67,7 +73,7 @@ class ChromeImportTest extends TestCase
         $chromeImport = $this->getChromeImport(false, 1);
         $chromeImport->setFilepath(__DIR__ . '/../fixtures/chrome-bookmarks');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -105,7 +111,7 @@ class ChromeImportTest extends TestCase
         $chromeImport = $this->getChromeImport();
         $chromeImport->setFilepath(__DIR__ . '/../fixtures/chrome-bookmarks');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -116,7 +122,7 @@ class ChromeImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -124,7 +130,7 @@ class ChromeImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -145,7 +151,7 @@ class ChromeImportTest extends TestCase
         $chromeImport = $this->getChromeImport();
         $chromeImport->setFilepath(__DIR__ . '/../fixtures/chrome-bookmarks');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -156,7 +162,7 @@ class ChromeImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -165,7 +171,7 @@ class ChromeImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'chrome');
         $producer = new Producer($queue);
@@ -212,19 +218,19 @@ class ChromeImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/FirefoxImportTest.php

@@ -2,12 +2,18 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\FirefoxImport;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -34,7 +40,7 @@ class FirefoxImportTest extends TestCase
         $firefoxImport = $this->getFirefoxImport(false, 2);
         $firefoxImport->setFilepath(__DIR__ . '/../fixtures/firefox-bookmarks.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -47,7 +53,7 @@ class FirefoxImportTest extends TestCase
             ->method('getRepository')
             ->willReturn($entryRepo);
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -67,7 +73,7 @@ class FirefoxImportTest extends TestCase
         $firefoxImport = $this->getFirefoxImport(false, 1);
         $firefoxImport->setFilepath(__DIR__ . '/../fixtures/firefox-bookmarks.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -105,7 +111,7 @@ class FirefoxImportTest extends TestCase
         $firefoxImport = $this->getFirefoxImport();
         $firefoxImport->setFilepath(__DIR__ . '/../fixtures/firefox-bookmarks.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -116,7 +122,7 @@ class FirefoxImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -124,7 +130,7 @@ class FirefoxImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -145,7 +151,7 @@ class FirefoxImportTest extends TestCase
         $firefoxImport = $this->getFirefoxImport();
         $firefoxImport->setFilepath(__DIR__ . '/../fixtures/firefox-bookmarks.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -156,7 +162,7 @@ class FirefoxImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -165,7 +171,7 @@ class FirefoxImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'firefox');
         $producer = new Producer($queue);
@@ -212,19 +218,19 @@ class FirefoxImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/ImportChainTest.php

@@ -4,12 +4,13 @@ namespace Tests\Wallabag\ImportBundle\Import;
 
 use PHPUnit\Framework\TestCase;
 use Wallabag\ImportBundle\Import\ImportChain;
+use Wallabag\ImportBundle\Import\ImportInterface;
 
 class ImportChainTest extends TestCase
 {
     public function testGetAll()
     {
-        $import = $this->getMockBuilder('Wallabag\ImportBundle\Import\ImportInterface')
+        $import = $this->getMockBuilder(ImportInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/InstapaperImportTest.php

@@ -2,12 +2,19 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\UnitOfWork;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\InstapaperImport;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -35,7 +42,7 @@ class InstapaperImportTest extends TestCase
         $instapaperImport = $this->getInstapaperImport(false, 4);
         $instapaperImport->setFilepath(__DIR__ . '/../fixtures/instapaper-export.csv');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -48,7 +55,7 @@ class InstapaperImportTest extends TestCase
             ->method('getRepository')
             ->willReturn($entryRepo);
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -68,7 +75,7 @@ class InstapaperImportTest extends TestCase
         $instapaperImport = $this->getInstapaperImport(false, 1);
         $instapaperImport->setFilepath(__DIR__ . '/../fixtures/instapaper-export.csv');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -106,7 +113,7 @@ class InstapaperImportTest extends TestCase
         $instapaperImport = $this->getInstapaperImport();
         $instapaperImport->setFilepath(__DIR__ . '/../fixtures/instapaper-export.csv');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -117,7 +124,7 @@ class InstapaperImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -125,7 +132,7 @@ class InstapaperImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -146,7 +153,7 @@ class InstapaperImportTest extends TestCase
         $instapaperImport = $this->getInstapaperImport();
         $instapaperImport->setFilepath(__DIR__ . '/../fixtures/instapaper-export.csv');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -157,7 +164,7 @@ class InstapaperImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -166,7 +173,7 @@ class InstapaperImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'instapaper');
         $producer = new Producer($queue);
@@ -213,11 +220,11 @@ class InstapaperImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->uow = $this->getMockBuilder('Doctrine\ORM\UnitOfWork')
+        $this->uow = $this->getMockBuilder(UnitOfWork::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -231,15 +238,15 @@ class InstapaperImportTest extends TestCase
             ->method('getScheduledEntityInsertions')
             ->willReturn([]);
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/PocketImportTest.php

@@ -2,15 +2,22 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\UnitOfWork;
 use GuzzleHttp\Psr7\Response;
 use Http\Mock\Client as HttpMockClient;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Config;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\PocketImport;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -187,7 +194,7 @@ JSON
 
         $pocketImport = $this->getPocketImport('ConsumerKey', 1);
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -277,7 +284,7 @@ JSON
 
         $pocketImport = $this->getPocketImport('ConsumerKey', 2);
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -357,7 +364,7 @@ JSON
 
         $pocketImport = $this->getPocketImport();
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -374,7 +381,7 @@ JSON
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -440,7 +447,7 @@ JSON
 
         $pocketImport = $this->getPocketImport();
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -458,7 +465,7 @@ JSON
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'pocket');
         $producer = new Producer($queue);
@@ -517,7 +524,7 @@ JSON
 
         $pocketImport = $this->getPocketImport('ConsumerKey', 1);
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -555,19 +562,19 @@ JSON
 
         $this->user->setConfig($config);
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->uow = $this->getMockBuilder('Doctrine\ORM\UnitOfWork')
+        $this->uow = $this->getMockBuilder(UnitOfWork::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -581,7 +588,7 @@ JSON
             ->method('getScheduledEntityInsertions')
             ->willReturn([]);
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/ReadabilityImportTest.php

@@ -2,12 +2,18 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\ReadabilityImport;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -34,7 +40,7 @@ class ReadabilityImportTest extends TestCase
         $readabilityImport = $this->getReadabilityImport(false, 3);
         $readabilityImport->setFilepath(__DIR__ . '/../fixtures/readability.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -47,7 +53,7 @@ class ReadabilityImportTest extends TestCase
             ->method('getRepository')
             ->willReturn($entryRepo);
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -67,7 +73,7 @@ class ReadabilityImportTest extends TestCase
         $readabilityImport = $this->getReadabilityImport(false, 1);
         $readabilityImport->setFilepath(__DIR__ . '/../fixtures/readability-read.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -105,7 +111,7 @@ class ReadabilityImportTest extends TestCase
         $readabilityImport = $this->getReadabilityImport();
         $readabilityImport->setFilepath(__DIR__ . '/../fixtures/readability.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -116,7 +122,7 @@ class ReadabilityImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -124,7 +130,7 @@ class ReadabilityImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -145,7 +151,7 @@ class ReadabilityImportTest extends TestCase
         $readabilityImport = $this->getReadabilityImport();
         $readabilityImport->setFilepath(__DIR__ . '/../fixtures/readability.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -156,7 +162,7 @@ class ReadabilityImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -165,7 +171,7 @@ class ReadabilityImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'readability');
         $producer = new Producer($queue);
@@ -212,19 +218,19 @@ class ReadabilityImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/WallabagV1ImportTest.php

@@ -2,12 +2,19 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\UnitOfWork;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\WallabagV1Import;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -37,7 +44,7 @@ class WallabagV1ImportTest extends TestCase
         $wallabagV1Import = $this->getWallabagV1Import(false, 1);
         $wallabagV1Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v1.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -50,7 +57,7 @@ class WallabagV1ImportTest extends TestCase
             ->method('getRepository')
             ->willReturn($entryRepo);
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -70,7 +77,7 @@ class WallabagV1ImportTest extends TestCase
         $wallabagV1Import = $this->getWallabagV1Import(false, 3);
         $wallabagV1Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v1-read.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -108,7 +115,7 @@ class WallabagV1ImportTest extends TestCase
         $wallabagV1Import = $this->getWallabagV1Import();
         $wallabagV1Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v1.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -119,7 +126,7 @@ class WallabagV1ImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -127,7 +134,7 @@ class WallabagV1ImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -148,7 +155,7 @@ class WallabagV1ImportTest extends TestCase
         $wallabagV1Import = $this->getWallabagV1Import();
         $wallabagV1Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v1.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -159,7 +166,7 @@ class WallabagV1ImportTest extends TestCase
             ->expects($this->never())
             ->method('getRepository');
 
-        $entry = $this->getMockBuilder('Wallabag\CoreBundle\Entity\Entry')
+        $entry = $this->getMockBuilder(Entry::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -168,7 +175,7 @@ class WallabagV1ImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'wallabag_v1');
         $producer = new Producer($queue);
@@ -215,11 +222,11 @@ class WallabagV1ImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->uow = $this->getMockBuilder('Doctrine\ORM\UnitOfWork')
+        $this->uow = $this->getMockBuilder(UnitOfWork::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -233,15 +240,15 @@ class WallabagV1ImportTest extends TestCase
             ->method('getScheduledEntityInsertions')
             ->willReturn([]);
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/ImportBundle/Import/WallabagV2ImportTest.php

@@ -2,12 +2,19 @@
 
 namespace Tests\Wallabag\ImportBundle\Import;
 
+use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\UnitOfWork;
 use M6Web\Component\RedisMock\RedisMockFactory;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
+use Predis\Client;
 use Simpleue\Queue\RedisQueue;
+use Symfony\Component\EventDispatcher\EventDispatcher;
 use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Helper\ContentProxy;
+use Wallabag\CoreBundle\Helper\TagsAssigner;
+use Wallabag\CoreBundle\Repository\EntryRepository;
 use Wallabag\ImportBundle\Import\WallabagV2Import;
 use Wallabag\ImportBundle\Redis\Producer;
 use Wallabag\UserBundle\Entity\User;
@@ -35,7 +42,7 @@ class WallabagV2ImportTest extends TestCase
         $wallabagV2Import = $this->getWallabagV2Import(false, 2);
         $wallabagV2Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v2.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -64,7 +71,7 @@ class WallabagV2ImportTest extends TestCase
         $wallabagV2Import = $this->getWallabagV2Import(false, 2);
         $wallabagV2Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v2-read.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -102,7 +109,7 @@ class WallabagV2ImportTest extends TestCase
         $wallabagV2Import = $this->getWallabagV2Import();
         $wallabagV2Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v2.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -117,7 +124,7 @@ class WallabagV2ImportTest extends TestCase
             ->expects($this->never())
             ->method('updateEntry');
 
-        $producer = $this->getMockBuilder('OldSound\RabbitMqBundle\RabbitMq\Producer')
+        $producer = $this->getMockBuilder(\OldSound\RabbitMqBundle\RabbitMq\Producer::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -138,7 +145,7 @@ class WallabagV2ImportTest extends TestCase
         $wallabagV2Import = $this->getWallabagV2Import();
         $wallabagV2Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v2.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -154,7 +161,7 @@ class WallabagV2ImportTest extends TestCase
             ->method('updateEntry');
 
         $factory = new RedisMockFactory();
-        $redisMock = $factory->getAdapter('Predis\Client', true);
+        $redisMock = $factory->getAdapter(Client::class, true);
 
         $queue = new RedisQueue($redisMock, 'wallabag_v2');
         $producer = new Producer($queue);
@@ -213,7 +220,7 @@ class WallabagV2ImportTest extends TestCase
         $wallabagV2Import = $this->getWallabagV2Import(false, 2);
         $wallabagV2Import->setFilepath(__DIR__ . '/../fixtures/wallabag-v2.json');
 
-        $entryRepo = $this->getMockBuilder('Wallabag\CoreBundle\Repository\EntryRepository')
+        $entryRepo = $this->getMockBuilder(EntryRepository::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -241,11 +248,11 @@ class WallabagV2ImportTest extends TestCase
     {
         $this->user = new User();
 
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->uow = $this->getMockBuilder('Doctrine\ORM\UnitOfWork')
+        $this->uow = $this->getMockBuilder(UnitOfWork::class)
             ->disableOriginalConstructor()
             ->getMock();
 
@@ -259,15 +266,15 @@ class WallabagV2ImportTest extends TestCase
             ->method('getScheduledEntityInsertions')
             ->willReturn([]);
 
-        $this->contentProxy = $this->getMockBuilder('Wallabag\CoreBundle\Helper\ContentProxy')
+        $this->contentProxy = $this->getMockBuilder(ContentProxy::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->tagsAssigner = $this->getMockBuilder('Wallabag\CoreBundle\Helper\TagsAssigner')
+        $this->tagsAssigner = $this->getMockBuilder(TagsAssigner::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $dispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcher')
+        $dispatcher = $this->getMockBuilder(EventDispatcher::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/UserBundle/EventListener/AuthenticationFailureListenerTest.php

@@ -8,8 +8,10 @@ use PHPUnit\Framework\TestCase;
 use Symfony\Component\EventDispatcher\EventDispatcher;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
+use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
 use Symfony\Component\Security\Core\AuthenticationEvents;
 use Symfony\Component\Security\Core\Event\AuthenticationFailureEvent;
+use Symfony\Component\Security\Core\Exception\AuthenticationException;
 use Wallabag\UserBundle\EventListener\AuthenticationFailureListener;
 
 class AuthenticationFailureListenerTest extends TestCase
@@ -41,11 +43,11 @@ class AuthenticationFailureListenerTest extends TestCase
 
     public function testOnAuthenticationFailure()
     {
-        $token = $this->getMockBuilder('Symfony\Component\Security\Core\Authentication\Token\TokenInterface')
+        $token = $this->getMockBuilder(TokenInterface::class)
             ->disableOriginalConstructor()
             ->getMock();
 
-        $exception = $this->getMockBuilder('Symfony\Component\Security\Core\Exception\AuthenticationException')
+        $exception = $this->getMockBuilder(AuthenticationException::class)
             ->disableOriginalConstructor()
             ->getMock();
 

tests/Wallabag/UserBundle/EventListener/CreateConfigListenerTest.php

@@ -2,6 +2,7 @@
 
 namespace Tests\Wallabag\UserBundle\EventListener;
 
+use Doctrine\ORM\EntityManager;
 use FOS\UserBundle\Event\FilterUserResponseEvent;
 use FOS\UserBundle\FOSUserEvents;
 use PHPUnit\Framework\TestCase;
@@ -25,7 +26,7 @@ class CreateConfigListenerTest extends TestCase
     protected function setUp(): void
     {
         $session = new Session(new MockArraySessionStorage());
-        $this->em = $this->getMockBuilder('Doctrine\ORM\EntityManager')
+        $this->em = $this->getMockBuilder(EntityManager::class)
             ->disableOriginalConstructor()
             ->getMock();