DigitaleAgentur___ALT
/
registration
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Correctly run the phpunit tests again 

Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
Joas Schilling 2020-07-10 13:12:20 +02:00
parent 92e79fa024
commit 9085f08269
No known key found for this signature in database
GPG Key ID: 7076EA9751AACDDA
5 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tests/Integration/Controller/RegisterControllerTest.php
View File

@ -2,6 +2,7 @@
namespace OCA\Registration\Tests\Integration\Controller; namespace OCA\Registration\Tests\Integration\Controller;
use OCA\Registration\Controller\RegisterController;
use OCA\Registration\Db\RegistrationMapper; use OCA\Registration\Db\RegistrationMapper;
use OCA\Registration\Service\MailService; use OCA\Registration\Service\MailService;
use OCA\Registration\Service\RegistrationService; use OCA\Registration\Service\RegistrationService;

9
tests/Integration/Service/RegistrationServiceTest.php
View File

@ -5,6 +5,7 @@ namespace OCA\Registration\Tests\Integration\Service;
use OCA\Registration\Db\Registration; use OCA\Registration\Db\Registration;
use OCA\Registration\Db\RegistrationMapper; use OCA\Registration\Db\RegistrationMapper;
use OCA\Registration\Service\MailService; use OCA\Registration\Service\MailService;
use OCA\Registration\Service\RegistrationException;
use OCA\Registration\Service\RegistrationService; use OCA\Registration\Service\RegistrationService;
use OCP\Defaults; use OCP\Defaults;
use OCP\IConfig; use OCP\IConfig;
@ -133,11 +134,11 @@ class RegistrationServiceTest extends TestCase {
} }
/** /**
* @depends testValidateNewEmailWithinAllowedDomain * @depends testValidateNewEmailWithinAllowedDomain
* @expectedException OCA\Registration\Service\RegistrationException
*/ */
public function testValidateNewEmailNotWithinAllowedDomain() { public function testValidateNewEmailNotWithinAllowedDomain() {
$email2 = 'bbbb@gmail.com'; $email2 = 'bbbb@gmail.com';
$this->expectException(RegistrationException::class);
$this->service->validateEmail($email2); $this->service->validateEmail($email2);
} }
@ -155,11 +156,11 @@ class RegistrationServiceTest extends TestCase {
} }
/** /**
* @depends testValidateNewEmailWithinMultipleAllowedDomain * @depends testValidateNewEmailWithinMultipleAllowedDomain
* @expectedException OCA\Registration\Service\RegistrationException
*/ */
public function testValidateNewEmailNotWithinMultipleAllowedDomain() { public function testValidateNewEmailNotWithinMultipleAllowedDomain() {
$email2 = 'cccc@yahoo.com'; $email2 = 'cccc@yahoo.com';
$this->expectException(RegistrationException::class);
$this->service->validateEmail($email2); $this->service->validateEmail($email2);
} }
@ -213,7 +214,6 @@ class RegistrationServiceTest extends TestCase {
/** /**
* @depends testCreateAccountWebForm * @depends testCreateAccountWebForm
* @expectedException OCA\Registration\Service\RegistrationException
*/ */
public function testDuplicateUsernameWebForm() { public function testDuplicateUsernameWebForm() {
$reg = new Registration(); $reg = new Registration();
@ -223,6 +223,7 @@ class RegistrationServiceTest extends TestCase {
//$reg->setPassword("asdf"); //$reg->setPassword("asdf");
$reg->setEmailConfirmed(true); $reg->setEmailConfirmed(true);
$this->expectException(RegistrationException::class);
$resulting_user = $this->service->createAccount($reg, 'alice1', 'asdf'); $resulting_user = $this->service->createAccount($reg, 'alice1', 'asdf');
} }
@ -236,7 +237,6 @@ class RegistrationServiceTest extends TestCase {
/** /**
* @depends testCreateAccountWebForm * @depends testCreateAccountWebForm
* @expectedException OCA\Registration\Service\RegistrationException
*/ */
public function testDuplicateUsernameApi() { public function testDuplicateUsernameApi() {
$reg = new Registration(); $reg = new Registration();
@ -246,6 +246,7 @@ class RegistrationServiceTest extends TestCase {
$reg->setPassword("asdf"); $reg->setPassword("asdf");
$reg->setEmailConfirmed(true); $reg->setEmailConfirmed(true);
$this->expectException(RegistrationException::class);
$resulting_user = $this->service->createAccount($reg); $resulting_user = $this->service->createAccount($reg);
} }

1
tests/Unit/Controller/ApiControllerTest.php
View File

@ -11,6 +11,7 @@
namespace OCA\Registration\Tests\Unit\Controller; namespace OCA\Registration\Tests\Unit\Controller;
use OCA\Registration\Controller\ApiController;
use OCA\Registration\Db\Registration; use OCA\Registration\Db\Registration;
use OCA\Registration\Service\MailService; use OCA\Registration\Service\MailService;
use OCA\Registration\Service\RegistrationService; use OCA\Registration\Service\RegistrationService;

2
tests/phpunit.integration.xml
View File

@ -3,7 +3,7 @@
verbose="true" verbose="true"
> >
<testsuite name='Registration app tests'> <testsuite name='Registration app tests'>
<directory suffix='Test.php'>integration</directory> <directory suffix='Test.php'>Integration</directory>
</testsuite> </testsuite>
<!-- filters for code coverage --> <!-- filters for code coverage -->
<filter> <filter>

2
tests/phpunit.unit.xml
View File

@ -3,7 +3,7 @@
verbose="true" verbose="true"
> >
<testsuite name='Registration app tests'> <testsuite name='Registration app tests'>
<directory suffix='Test.php'>unit</directory> <directory suffix='Test.php'>Unit</directory>
</testsuite> </testsuite>
<!-- filters for code coverage --> <!-- filters for code coverage -->
<filter> <filter>