From 18cf594f8ad4bb5236274cf1326c5e6ea549a570 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicolas=20L=C5=93uillet?= Date: Thu, 15 Oct 2015 13:17:21 +0200 Subject: move 2factor activation in parameters --- .../Tests/Controller/SecurityControllerTest.php | 62 ++++++++++++---------- 1 file changed, 34 insertions(+), 28 deletions(-) (limited to 'src/Wallabag/CoreBundle/Tests') diff --git a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php index 3402b340..b9f5d835 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php @@ -19,40 +19,46 @@ class SecurityControllerTest extends WallabagCoreTestCase public function testLoginWith2Factor() { $client = $this->getClient(); - $client->followRedirects(); - $em = $client->getContainer()->get('doctrine.orm.entity_manager'); - $user = $em - ->getRepository('WallabagUserBundle:User') - ->findOneByUsername('admin'); - $user->setTwoFactorAuthentication(true); - $em->persist($user); - $em->flush(); + if ($client->getContainer()->getParameter('twofactor_auth')) { + $client->followRedirects(); - $this->logInAs('admin'); - $client->request('GET', '/config'); - $this->assertContains('trusted computer', $client->getResponse()->getContent()); - - // restore user - $user = $em - ->getRepository('WallabagUserBundle:User') - ->findOneByUsername('admin'); - $user->setTwoFactorAuthentication(false); - $em->persist($user); - $em->flush(); + $em = $client->getContainer()->get('doctrine.orm.entity_manager'); + $user = $em + ->getRepository('WallabagUserBundle:User') + ->findOneByUsername('admin'); + $user->setTwoFactorAuthentication(true); + $em->persist($user); + $em->flush(); + + $this->logInAs('admin'); + $client->request('GET', '/config'); + $this->assertContains('trusted computer', $client->getResponse()->getContent()); + + // restore user + $user = $em + ->getRepository('WallabagUserBundle:User') + ->findOneByUsername('admin'); + $user->setTwoFactorAuthentication(false); + $em->persist($user); + $em->flush(); + } } public function testTrustedComputer() { $client = $this->getClient(); - $em = $client->getContainer()->get('doctrine.orm.entity_manager'); - $user = $em - ->getRepository('WallabagUserBundle:User') - ->findOneByUsername('admin'); - - $date = new \DateTime(); - $user->addTrustedComputer('ABCDEF', $date->add(new \DateInterval('P1M'))); - $this->assertTrue($user->isTrustedComputer('ABCDEF')); - $this->assertFalse($user->isTrustedComputer('FEDCBA')); + + if ($client->getContainer()->getParameter('twofactor_auth')) { + $em = $client->getContainer()->get('doctrine.orm.entity_manager'); + $user = $em + ->getRepository('WallabagUserBundle:User') + ->findOneByUsername('admin'); + + $date = new \DateTime(); + $user->addTrustedComputer('ABCDEF', $date->add(new \DateInterval('P1M'))); + $this->assertTrue($user->isTrustedComputer('ABCDEF')); + $this->assertFalse($user->isTrustedComputer('FEDCBA')); + } } } -- cgit v1.2.3