diff --git a/src/Wallabag/ApiBundle/Controller/UserRestController.php b/src/Wallabag/ApiBundle/Controller/UserRestController.php index 675c2292a..5d91cd582 100644 --- a/src/Wallabag/ApiBundle/Controller/UserRestController.php +++ b/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, ]); diff --git a/src/Wallabag/ApiBundle/Form/Type/ClientType.php b/src/Wallabag/ApiBundle/Form/Type/ClientType.php index 14dc5c44f..3d00b2ac8 100644 --- a/src/Wallabag/ApiBundle/Form/Type/ClientType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Controller/IgnoreOriginInstanceRuleController.php b/src/Wallabag/CoreBundle/Controller/IgnoreOriginInstanceRuleController.php index a3cd1caf0..22fbf5fa0 100644 --- a/src/Wallabag/CoreBundle/Controller/IgnoreOriginInstanceRuleController.php +++ b/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()) { diff --git a/src/Wallabag/CoreBundle/Controller/SiteCredentialController.php b/src/Wallabag/CoreBundle/Controller/SiteCredentialController.php index da19d940a..67b262947 100644 --- a/src/Wallabag/CoreBundle/Controller/SiteCredentialController.php +++ b/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()) { diff --git a/src/Wallabag/CoreBundle/Form/Type/ConfigType.php b/src/Wallabag/CoreBundle/Form/Type/ConfigType.php index af916b29f..2ea776512 100644 --- a/src/Wallabag/CoreBundle/Form/Type/ConfigType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/EditEntryType.php b/src/Wallabag/CoreBundle/Form/Type/EditEntryType.php index 2fc4c204b..911616594 100644 --- a/src/Wallabag/CoreBundle/Form/Type/EditEntryType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/FeedType.php b/src/Wallabag/CoreBundle/Form/Type/FeedType.php index 9b34daf4c..b2ebddb38 100644 --- a/src/Wallabag/CoreBundle/Form/Type/FeedType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginInstanceRuleType.php b/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginInstanceRuleType.php index d2e414fb7..608cbf7cf 100644 --- a/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginInstanceRuleType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginUserRuleType.php b/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginUserRuleType.php index b9110f17b..7e96b144f 100644 --- a/src/Wallabag/CoreBundle/Form/Type/IgnoreOriginUserRuleType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/NewEntryType.php b/src/Wallabag/CoreBundle/Form/Type/NewEntryType.php index 7af1e5895..ecbd68ba7 100644 --- a/src/Wallabag/CoreBundle/Form/Type/NewEntryType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/NewTagType.php b/src/Wallabag/CoreBundle/Form/Type/NewTagType.php index e830ade48..ed4fa67a2 100644 --- a/src/Wallabag/CoreBundle/Form/Type/NewTagType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/RenameTagType.php b/src/Wallabag/CoreBundle/Form/Type/RenameTagType.php index e62700487..a639b4e9e 100644 --- a/src/Wallabag/CoreBundle/Form/Type/RenameTagType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/SiteCredentialType.php b/src/Wallabag/CoreBundle/Form/Type/SiteCredentialType.php index fd409ad2c..0bf1acb60 100644 --- a/src/Wallabag/CoreBundle/Form/Type/SiteCredentialType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/TaggingRuleType.php b/src/Wallabag/CoreBundle/Form/Type/TaggingRuleType.php index 732606c91..1a356df53 100644 --- a/src/Wallabag/CoreBundle/Form/Type/TaggingRuleType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Form/Type/UserInformationType.php b/src/Wallabag/CoreBundle/Form/Type/UserInformationType.php index 6e4c9154c..c8544e428 100644 --- a/src/Wallabag/CoreBundle/Form/Type/UserInformationType.php +++ b/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, ]); } diff --git a/src/Wallabag/CoreBundle/Helper/DownloadImages.php b/src/Wallabag/CoreBundle/Helper/DownloadImages.php index a8a6ccad8..b74f63027 100644 --- a/src/Wallabag/CoreBundle/Helper/DownloadImages.php +++ b/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()); diff --git a/src/Wallabag/UserBundle/Controller/ManageController.php b/src/Wallabag/UserBundle/Controller/ManageController.php index 1a2edfb95..1d2bcbb40 100644 --- a/src/Wallabag/UserBundle/Controller/ManageController.php +++ b/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 diff --git a/src/Wallabag/UserBundle/Form/NewUserType.php b/src/Wallabag/UserBundle/Form/NewUserType.php index 03df4e338..ff63e2906 100644 --- a/src/Wallabag/UserBundle/Form/NewUserType.php +++ b/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, ]); } diff --git a/src/Wallabag/UserBundle/Form/UserType.php b/src/Wallabag/UserBundle/Form/UserType.php index 03fad9717..ab0db2033 100644 --- a/src/Wallabag/UserBundle/Form/UserType.php +++ b/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, ]); } } diff --git a/tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php b/tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php index c753de01e..7cdf20ab6 100644 --- a/tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php +++ b/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); diff --git a/tests/Wallabag/CoreBundle/Event/Listener/LocaleListenerTest.php b/tests/Wallabag/CoreBundle/Event/Listener/LocaleListenerTest.php index bc35d1000..2d0b87ae1 100644 --- a/tests/Wallabag/CoreBundle/Event/Listener/LocaleListenerTest.php +++ b/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(); diff --git a/tests/Wallabag/CoreBundle/Event/Subscriber/TablePrefixSubscriberTest.php b/tests/Wallabag/CoreBundle/Event/Subscriber/TablePrefixSubscriberTest.php index 976462a13..6a22a84e1 100644 --- a/tests/Wallabag/CoreBundle/Event/Subscriber/TablePrefixSubscriberTest.php +++ b/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_', 'Wallabag\UserBundle\Entity\User', '`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 PostgreSqlPlatform()], + ['wallabag_', User::class, '`user`', 'user', 'wallabag_user', '`wallabag_user`', new MySqlPlatform()], + ['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_', 'Wallabag\UserBundle\Entity\User', '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 PostgreSqlPlatform()], + ['wallabag_', User::class, 'user', 'user', 'wallabag_user', 'wallabag_user', new MySqlPlatform()], + ['wallabag_', User::class, 'user', 'user', 'wallabag_user', 'wallabag_user', new SqlitePlatform()], - ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '"user"', new PostgreSqlPlatform()], - ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '`user`', new MySqlPlatform()], - ['', 'Wallabag\UserBundle\Entity\User', '`user`', 'user', 'user', '"user"', new SqlitePlatform()], + ['', User::class, '`user`', 'user', 'user', '"user"', new PostgreSqlPlatform()], + ['', User::class, '`user`', 'user', 'user', '`user`', new MySqlPlatform()], + ['', User::class, '`user`', 'user', 'user', '"user"', new SqlitePlatform()], - ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new PostgreSqlPlatform()], - ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new MySqlPlatform()], - ['', 'Wallabag\UserBundle\Entity\User', 'user', 'user', 'user', 'user', new SqlitePlatform()], + ['', User::class, 'user', 'user', 'user', 'user', new PostgreSqlPlatform()], + ['', User::class, 'user', 'user', 'user', 'user', new MySqlPlatform()], + ['', 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(); diff --git a/tests/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilderTest.php b/tests/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilderTest.php index 2b7840b0c..d22764250 100644 --- a/tests/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilderTest.php +++ b/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()) diff --git a/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php b/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php index 4644912fc..ef6e7aebe 100644 --- a/tests/Wallabag/CoreBundle/Helper/ContentProxyTest.php +++ b/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(); diff --git a/tests/Wallabag/CoreBundle/Helper/RedirectTest.php b/tests/Wallabag/CoreBundle/Helper/RedirectTest.php index 4a8864e3c..57a46dc14 100644 --- a/tests/Wallabag/CoreBundle/Helper/RedirectTest.php +++ b/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(); diff --git a/tests/Wallabag/CoreBundle/Helper/RuleBasedIgnoreOriginProcessorTest.php b/tests/Wallabag/CoreBundle/Helper/RuleBasedIgnoreOriginProcessorTest.php index afe8502f7..4bd62ba4e 100644 --- a/tests/Wallabag/CoreBundle/Helper/RuleBasedIgnoreOriginProcessorTest.php +++ b/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(); } diff --git a/tests/Wallabag/CoreBundle/Helper/RuleBasedTaggerTest.php b/tests/Wallabag/CoreBundle/Helper/RuleBasedTaggerTest.php index f85b8d07b..da11737ca 100644 --- a/tests/Wallabag/CoreBundle/Helper/RuleBasedTaggerTest.php +++ b/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(); } diff --git a/tests/Wallabag/CoreBundle/ParamConverter/UsernameFeedTokenConverterTest.php b/tests/Wallabag/CoreBundle/ParamConverter/UsernameFeedTokenConverterTest.php index f39372caf..e8a82e49a 100644 --- a/tests/Wallabag/CoreBundle/ParamConverter/UsernameFeedTokenConverterTest.php +++ b/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(); diff --git a/tests/Wallabag/CoreBundle/Twig/WallabagExtensionTest.php b/tests/Wallabag/CoreBundle/Twig/WallabagExtensionTest.php index db02cb9cf..ddb4806b0 100644 --- a/tests/Wallabag/CoreBundle/Twig/WallabagExtensionTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Command/RedisWorkerCommandTest.php b/tests/Wallabag/ImportBundle/Command/RedisWorkerCommandTest.php index 16e11c555..f27439af3 100644 --- a/tests/Wallabag/ImportBundle/Command/RedisWorkerCommandTest.php +++ b/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); diff --git a/tests/Wallabag/ImportBundle/Consumer/AMQPEntryConsumerTest.php b/tests/Wallabag/ImportBundle/Consumer/AMQPEntryConsumerTest.php index b7f6192d3..c02bb366f 100644 --- a/tests/Wallabag/ImportBundle/Consumer/AMQPEntryConsumerTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Consumer/RedisEntryConsumerTest.php b/tests/Wallabag/ImportBundle/Consumer/RedisEntryConsumerTest.php index e1bd88272..7213c27cc 100644 --- a/tests/Wallabag/ImportBundle/Consumer/RedisEntryConsumerTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Controller/ChromeControllerTest.php b/tests/Wallabag/ImportBundle/Controller/ChromeControllerTest.php index 6318137f8..997318e6a 100644 --- a/tests/Wallabag/ImportBundle/Controller/ChromeControllerTest.php +++ b/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()); diff --git a/tests/Wallabag/ImportBundle/Controller/DeliciousControllerTest.php b/tests/Wallabag/ImportBundle/Controller/DeliciousControllerTest.php index 88cdbd809..bb710b18f 100644 --- a/tests/Wallabag/ImportBundle/Controller/DeliciousControllerTest.php +++ b/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]); diff --git a/tests/Wallabag/ImportBundle/Controller/ElcuratorControllerTest.php b/tests/Wallabag/ImportBundle/Controller/ElcuratorControllerTest.php index ff67a823d..188d2798e 100644 --- a/tests/Wallabag/ImportBundle/Controller/ElcuratorControllerTest.php +++ b/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')); diff --git a/tests/Wallabag/ImportBundle/Controller/FirefoxControllerTest.php b/tests/Wallabag/ImportBundle/Controller/FirefoxControllerTest.php index 43abe5a6e..1c744eae8 100644 --- a/tests/Wallabag/ImportBundle/Controller/FirefoxControllerTest.php +++ b/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'); diff --git a/tests/Wallabag/ImportBundle/Controller/InstapaperControllerTest.php b/tests/Wallabag/ImportBundle/Controller/InstapaperControllerTest.php index a1243b0b4..8a1616564 100644 --- a/tests/Wallabag/ImportBundle/Controller/InstapaperControllerTest.php +++ b/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'); diff --git a/tests/Wallabag/ImportBundle/Controller/PinboardControllerTest.php b/tests/Wallabag/ImportBundle/Controller/PinboardControllerTest.php index f8eca5e5a..ce030e471 100644 --- a/tests/Wallabag/ImportBundle/Controller/PinboardControllerTest.php +++ b/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'])); diff --git a/tests/Wallabag/ImportBundle/Controller/PocketControllerTest.php b/tests/Wallabag/ImportBundle/Controller/PocketControllerTest.php index e17dac376..1bd57a62b 100644 --- a/tests/Wallabag/ImportBundle/Controller/PocketControllerTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Controller/ReadabilityControllerTest.php b/tests/Wallabag/ImportBundle/Controller/ReadabilityControllerTest.php index df7e44ea1..75134a247 100644 --- a/tests/Wallabag/ImportBundle/Controller/ReadabilityControllerTest.php +++ b/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'])); diff --git a/tests/Wallabag/ImportBundle/Controller/WallabagV1ControllerTest.php b/tests/Wallabag/ImportBundle/Controller/WallabagV1ControllerTest.php index 0973b6786..0fa171f66 100644 --- a/tests/Wallabag/ImportBundle/Controller/WallabagV1ControllerTest.php +++ b/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'])); diff --git a/tests/Wallabag/ImportBundle/Controller/WallabagV2ControllerTest.php b/tests/Wallabag/ImportBundle/Controller/WallabagV2ControllerTest.php index 7cc02c897..31824f437 100644 --- a/tests/Wallabag/ImportBundle/Controller/WallabagV2ControllerTest.php +++ b/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'); diff --git a/tests/Wallabag/ImportBundle/Import/ChromeImportTest.php b/tests/Wallabag/ImportBundle/Import/ChromeImportTest.php index 66e72a8d8..2829466c0 100644 --- a/tests/Wallabag/ImportBundle/Import/ChromeImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/FirefoxImportTest.php b/tests/Wallabag/ImportBundle/Import/FirefoxImportTest.php index d3b3ee1ae..362c2ee05 100644 --- a/tests/Wallabag/ImportBundle/Import/FirefoxImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/ImportChainTest.php b/tests/Wallabag/ImportBundle/Import/ImportChainTest.php index f27826f40..c3984fa63 100644 --- a/tests/Wallabag/ImportBundle/Import/ImportChainTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/InstapaperImportTest.php b/tests/Wallabag/ImportBundle/Import/InstapaperImportTest.php index 051c7c68a..84900d3fb 100644 --- a/tests/Wallabag/ImportBundle/Import/InstapaperImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/PocketImportTest.php b/tests/Wallabag/ImportBundle/Import/PocketImportTest.php index 84eb80ec7..fe92b34c1 100644 --- a/tests/Wallabag/ImportBundle/Import/PocketImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/ReadabilityImportTest.php b/tests/Wallabag/ImportBundle/Import/ReadabilityImportTest.php index c9baabd69..6b2622ccb 100644 --- a/tests/Wallabag/ImportBundle/Import/ReadabilityImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/WallabagV1ImportTest.php b/tests/Wallabag/ImportBundle/Import/WallabagV1ImportTest.php index 165fe1466..5da4aa658 100644 --- a/tests/Wallabag/ImportBundle/Import/WallabagV1ImportTest.php +++ b/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(); diff --git a/tests/Wallabag/ImportBundle/Import/WallabagV2ImportTest.php b/tests/Wallabag/ImportBundle/Import/WallabagV2ImportTest.php index 4c52fa1c8..59173f58a 100644 --- a/tests/Wallabag/ImportBundle/Import/WallabagV2ImportTest.php +++ b/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(); diff --git a/tests/Wallabag/UserBundle/EventListener/AuthenticationFailureListenerTest.php b/tests/Wallabag/UserBundle/EventListener/AuthenticationFailureListenerTest.php index 54384f26b..8518373c2 100644 --- a/tests/Wallabag/UserBundle/EventListener/AuthenticationFailureListenerTest.php +++ b/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(); diff --git a/tests/Wallabag/UserBundle/EventListener/CreateConfigListenerTest.php b/tests/Wallabag/UserBundle/EventListener/CreateConfigListenerTest.php index 7a685f990..dedd22f3f 100644 --- a/tests/Wallabag/UserBundle/EventListener/CreateConfigListenerTest.php +++ b/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();