diff --git a/tests/SPT/Domain/Account/Services/AccountTest.php b/tests/SPT/Domain/Account/Services/AccountTest.php index 9163730b..4e624177 100644 --- a/tests/SPT/Domain/Account/Services/AccountTest.php +++ b/tests/SPT/Domain/Account/Services/AccountTest.php @@ -116,7 +116,7 @@ class AccountTest extends UnitaryTestCase /** * @throws ServiceException */ - public function testUpdateUserCannotChangePermissionsWithoutAdminApp() + public function testUpdateUserCannotChangePermissionsWithoutPermission() { $id = self::$faker->randomNumber(); $accountDataGenerator = AccountDataGenerator::factory(); @@ -124,10 +124,14 @@ class AccountTest extends UnitaryTestCase $this->context->setUserData( new UserDataDto( - UserDataGenerator::factory()->buildUserData()->mutate(['isAdminApp' => false]) + UserDataGenerator::factory() + ->buildUserData() + ->mutate(['isAdminApp' => false, 'isAdminAcc' => false]) ) ); + $this->context->getUserProfile()->setAccPermission(false); + $this->configService->expects(self::once())->method('getByParam') ->with('masterPwd')->willReturn(self::$faker->password); $this->accountHistoryService->expects(self::once())->method('create'); diff --git a/tests/SPT/Util/HttpUtilTest.php b/tests/SPT/Util/HttpUtilTest.php new file mode 100644 index 00000000..d143ba9f --- /dev/null +++ b/tests/SPT/Util/HttpUtilTest.php @@ -0,0 +1,116 @@ +. + */ + +namespace SPT\Util; + +use PHPUnit\Framework\Attributes\Group; +use PHPUnit\Framework\MockObject\Exception; +use PHPUnit\Framework\TestCase; +use SP\Domain\Config\Ports\ConfigDataInterface; +use SP\Domain\Http\RequestInterface; +use SP\Util\HttpUtil; + +/** + * Class HttpUtilTest + */ +#[Group('unitary')] +class HttpUtilTest extends TestCase +{ + + /** + * @throws Exception + */ + public function testCheckHttps() + { + $configData = $this->createMock(ConfigDataInterface::class); + $request = $this->createMock(RequestInterface::class); + + $configData->expects($this->once()) + ->method('isHttpsEnabled') + ->willReturn(true); + + $request->expects($this->once()) + ->method('isHttps') + ->willReturn(false); + + $request->expects($this->once()) + ->method('getServerPort') + ->willReturn(8080); + + $request->expects($this->once()) + ->method('getHttpHost') + ->willReturn('localhost'); + + HttpUtil::checkHttps($configData, $request); + } + + /** + * @throws Exception + */ + public function testCheckHttpsWithNoHttpsEnabled() + { + $configData = $this->createMock(ConfigDataInterface::class); + $request = $this->createMock(RequestInterface::class); + + $configData->expects($this->once()) + ->method('isHttpsEnabled') + ->willReturn(false); + + $request->expects($this->never()) + ->method('isHttps'); + + $request->expects($this->never()) + ->method('getServerPort'); + + $request->expects($this->never()) + ->method('getHttpHost'); + + HttpUtil::checkHttps($configData, $request); + } + + /** + * @throws Exception + */ + public function testCheckHttpsWithHttpsEnabledAndHttpsRequest() + { + $configData = $this->createMock(ConfigDataInterface::class); + $request = $this->createMock(RequestInterface::class); + + $configData->expects($this->once()) + ->method('isHttpsEnabled') + ->willReturn(true); + + $request->expects($this->once()) + ->method('isHttps') + ->willReturn(true); + + $request->expects($this->never()) + ->method('getServerPort'); + + $request->expects($this->never()) + ->method('getHttpHost'); + + HttpUtil::checkHttps($configData, $request); + } +}