Skip to content

Commit

Permalink
tests conf updated
Browse files Browse the repository at this point in the history
  • Loading branch information
@s3b4stian
s3b4stian committed Dec 13, 2023
1 parent afc112c commit d3ee80a
Show file tree
Hide file tree
Showing 3 changed files with 61 additions and 63 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/tests.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ jobs:
PGPASSWORD=password psql -U postgres -h0.0.0.0 -d linna_db -c "SELECT user_id, name from public.user"
- name: Execute tests
run: vendor/bin/paratest --coverage-text
run: vendor/bin/phpunit

static-analysis:
runs-on: ubuntu-latest
Expand All @@ -91,7 +91,7 @@ jobs:
strategy:
fail-fast: true
matrix:
php: [8.1, 8.2]
php: [8.1, 8.2, 8.3]

name: Code analysis with PHPStan PHP ${{ matrix.php }}

Expand Down
2 changes: 1 addition & 1 deletion phpunit.dist.xml
View file
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.2/phpunit.xsd" colors="true" executionOrder="default" bootstrap="vendor/autoload.php" cacheDirectory=".phpunit.cache">
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.4/phpunit.xsd" colors="true" bootstrap="vendor/autoload.php" cacheDirectory=".phpunit.cache">
<coverage>
<report>
<clover outputFile="clover.xml"/>
Expand Down
118 changes: 58 additions & 60 deletions tests/Linna/Authentication/EnhancedAuthenticationTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -116,80 +116,78 @@ public function testNewInstance(): void
}

/**
* Wrong arguments router class provider.
* Test ge attempts with the same user.
*
* @return array
* @return void
*/
public static function wrongCredentialProvider(): array
public function testGetAttemptsLeftWithSameUser(): void
{
return [
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 4, 9, 19, false, false, false],
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 3, 8, 18, false, false, false],
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 2, 7, 17, false, false, false],
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 1, 6, 16, false, false, false],
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 0, 5, 15, true, false, false],
['root', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 0, 4, 14, true, false, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 4, 3, 13, false, false, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 3, 2, 12, false, false, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 2, 1, 11, false, false, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 1, 0, 10, false, true, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 0, 0, 9, true, true, false],
['admin', 'mbvi2lgdpcj6vp3qemh2estei2', '192.168.1.2', 0, 0, 8, true, true, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 4, 9, 7, false, false, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 3, 8, 6, false, false, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 2, 7, 5, false, false, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 1, 6, 4, false, false, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 0, 5, 3, true, false, false],
['administrator', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 0, 4, 2, true, false, false],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 4, 3, 1, false, false, false],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 3, 2, 0, false, false, true],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 2, 1, 0, false, false, true],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 1, 0, 0, false, true, true],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 0, 0, 0, true, true, true],
['poweruser', 'vaqgvpochtif8gh888q6vnlch5', '192.168.1.2', 0, 0, 0, true, true, true],
['fooroot', '3hto06tko273jjc1se0v1aqvvn', '192.168.1.3', 4, 9, 19, false, false, false],
['fooroot', '3hto06tko273jjc1se0v1aqvvn', '192.168.1.3', 3, 8, 18, false, false, false],
['fooroot', '3hto06tko273jjc1se0v1aqvvn', '192.168.1.3', 2, 7, 17, false, false, false],
['fooroot', '3hto06tko273jjc1se0v1aqvvn', '192.168.1.3', 1, 6, 16, false, false, false],
];
$user = 'root';
$sessionId = 'mbvi2lgdpcj6vp3qemh2estei2';
$ipAddress = '192.168.1.2';

for ($i = 0; $i < 4; $i++){
$this->storeLoginAttempt($user, $sessionId, $ipAddress);
}

$this->assertEquals(1, self::$enhancedAuthentication->getAttemptsLeftWithSameUser($user));
$this->assertFalse(self::$enhancedAuthentication->isUserBanned($user));

//pass the threshold
$this->storeLoginAttempt($user, $sessionId, $ipAddress);

$this->assertEquals(0, self::$enhancedAuthentication->getAttemptsLeftWithSameUser($user));
$this->assertTrue(self::$enhancedAuthentication->isUserBanned($user));
}

/**
* Test login.
*
* @dataProvider wrongCredentialProvider
* Test ge attempts with the same session.
*
* @param string $user User name.
* @param string $sessionId Session id.
* @param string $ipAddress Ip address.
* @param int $awsU Attempts with same user.
* @param int $awsS Attempts with same session id.
* @param int $awsI Attempts with same ip
* @param bool $banU Is user banned?.
* @param bool $banS Is session id banned?.
* @param bool $banI Is ip banned?.
* @return void
*/
public function testGetAttemptsLeftWithSameSession(): void
{
$user = 'admin';
$sessionId = 'vaqgvpochtif8gh888q6vnlch5';
$ipAddress = '192.168.1.3';

for ($i = 0; $i < 9; $i++){
$this->storeLoginAttempt($user, $sessionId, $ipAddress);
}

$this->assertEquals(1, self::$enhancedAuthentication->getAttemptsLeftWithSameSession($sessionId));
$this->assertFalse(self::$enhancedAuthentication->isSessionBanned($sessionId));

//pass the threshold
$this->storeLoginAttempt($user, $sessionId, $ipAddress);

$this->assertEquals(0, self::$enhancedAuthentication->getAttemptsLeftWithSameSession($sessionId));
$this->assertTrue(self::$enhancedAuthentication->isSessionBanned($sessionId));
}

/**
* Test ge attempts with the same ip.
*
* @return void
*/
public function testLogin(string $user, string $sessionId, string $ipAddress, int $awsU, int $awsS, int $awsI, bool $banU, bool $banS, bool $banI): void
public function testGetAttemptsLeftWithSameIp(): void
{
$this->assertFalse(self::$enhancedAuthentication->login($user, 'passwor', $user, '$2y$11$4IAn6SRaB0osPz8afZC5D.CmTrBGxnb5FQEygPjDirK9SWE/u8YuO', 1));
$user = 'user';
$sessionId = '3hto06tko273jjc1se0v1aqvvn';
$ipAddress = '192.168.1.4';

for ($i = 0; $i < 19; $i++){
$this->storeLoginAttempt($user, $sessionId, $ipAddress);
}

$this->assertEquals(1, self::$enhancedAuthentication->getAttemptsLeftWithSameIp($ipAddress));
$this->assertFalse(self::$enhancedAuthentication->isIpBanned($ipAddress));

//pass the threshold
$this->storeLoginAttempt($user, $sessionId, $ipAddress);

//Access with user
$this->assertEquals($awsU, self::$enhancedAuthentication->getAttemptsLeftWithSameUser($user));
//Access with session
$this->assertEquals($awsS, self::$enhancedAuthentication->getAttemptsLeftWithSameSession($sessionId));
//Access with ip
$this->assertEquals($awsI, self::$enhancedAuthentication->getAttemptsLeftWithSameIp($ipAddress));

//User Banned
$this->assertEquals($banU, self::$enhancedAuthentication->isUserBanned($user));
//Session Banned
$this->assertEquals($banS, self::$enhancedAuthentication->isSessionBanned($sessionId));
//Ip Banned
$this->assertEquals($banI, self::$enhancedAuthentication->isIpBanned($ipAddress));
$this->assertEquals(0, self::$enhancedAuthentication->getAttemptsLeftWithSameIp($ipAddress));
$this->assertTrue(self::$enhancedAuthentication->isIpBanned($ipAddress));
}

/**
Expand Down

0 comments on commit d3ee80a

Please sign in to comment.