Skip to content

Commit 833cd52

Browse files
Fix deprecation warnings in phpunit, strangely causing phpunit failures on PHP 8.1 and 8.2
1 parent 693a255 commit 833cd52

10 files changed

Lines changed: 10 additions & 29 deletions

tests/src/Auth/Source/PasswordVerify1CompatTest.php

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,7 @@
44

55
namespace SimpleSAML\Test\Module\sqlauth\Auth\Source;
66

7-
/**
8-
* Test for the core:AttributeLimit filter.
9-
*
10-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
11-
*/
7+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\PasswordVerify1Compat::class)
128
class PasswordVerify1CompatTest extends PasswordVerifyTest
139
{
1410
protected string $wrapperClassName = '\SimpleSAML\Test\Module\sqlauth\Auth\Source\PasswordVerify1CompatWrapper';

tests/src/Auth/Source/PasswordVerifyTest.php

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,7 @@
77
use PDO;
88
use PHPUnit\Framework\TestCase;
99

10-
/**
11-
* Test for the core:AttributeLimit filter.
12-
*
13-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
14-
*/
10+
#CoversClass(\SimpleSAML\Module\sqlauth\Auth\Source\PasswordVerify::class)
1511
class PasswordVerifyTest extends TestCase
1612
{
1713
// Subclasses can override this to test other wrapper classes

tests/src/Auth/Source/SQL1CompatTest.php

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,7 @@
44

55
namespace SimpleSAML\Test\Module\sqlauth\Auth\Source;
66

7-
/**
8-
* Test for the core:AttributeLimit filter.
9-
*
10-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
11-
*/
7+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL1Compat::class)
128
class SQL1CompatTest extends SQLTest
139
{
1410
protected string $wrapperClassName = '\SimpleSAML\Test\Module\sqlauth\Auth\Source\SQL1CompatWrapper';

tests/src/Auth/Source/SQL2MultipleAuthTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@
1919
* Attributes for students come from just the student database, whereas the staff attributes
2020
* come from both staff databases if the user is in both, and only the main staff database
2121
* if they are not in the physics department.
22-
*
23-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
2422
*/
23+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
2524
class SQL2MultipleAuthTest extends TestCase
2625
{
2726
/** @var array<string, string> */

tests/src/Auth/Source/SQL2PasswordVerifyMultipleAuthTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,8 @@
88

99
/**
1010
* The scenario is SQL2MultipleAuthTest but with passwords hashed using password_hash()
11-
*
12-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1311
*/
12+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
1413
class SQL2PasswordVerifyMultipleAuthTest extends SQL2MultipleAuthTest
1514
{
1615
// We need to return password column for password_verify() to use.

tests/src/Auth/Source/SQL2PasswordVerifySimpleTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,8 @@
1313
*
1414
* The tests are essentially the same tests as SQLTest, but using the SQLv2
1515
* configuration and code, not the SQL1 code or SQL1Compat interface.
16-
*
17-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1816
*/
17+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
1918
class SQL2PasswordVerifySimpleTest extends SQL2SimpleTest
2019
{
2120
// We need to return password column for password_verify() to use.

tests/src/Auth/Source/SQL2PasswordVerifySingleAuthTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,8 @@
1313
* but the common identifier across all databases is the userid (uid).
1414
*
1515
* The attributes then come from multiple databases.
16-
*
17-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1816
*/
17+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
1918
class SQL2PasswordVerifySingleAuthTest extends SQL2SingleAuthTest
2019
{
2120
// We need to return password column for password_verify() to use.

tests/src/Auth/Source/SQL2SimpleTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,8 @@
1111
* The scenario for this test case is a single database of customers who have their
1212
* metadata in a single database. This is essentially the same tests as SQLTest, but
1313
* using the SQLv2 configuration and code, not the SQL1 code or SQL1Compat interface.
14-
*
15-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1614
*/
15+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
1716
class SQL2SimpleTest extends TestCase
1817
{
1918
/** @var array<string, string> */

tests/src/Auth/Source/SQL2SingleAuthTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,8 @@
1313
* but the common identifier across all databases is the userid (uid).
1414
*
1515
* The attributes then come from multiple databases.
16-
*
17-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1816
*/
17+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL2::class)
1918
class SQL2SingleAuthTest extends TestCase
2019
{
2120
/** @var array<string, string> */

tests/src/Auth/Source/SQLTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,8 @@
99

1010
/**
1111
* Test for the core:AttributeLimit filter.
12-
*
13-
* @covers \SimpleSAML\Module\core\Auth\Process\AttributeLimit
1412
*/
13+
#CoversClass(SimpleSAML\Module\sqlauth\Auth\Source\SQL::class)
1514
class SQLTest extends TestCase
1615
{
1716
// Subclasses can override this to test other wrapper classes

0 commit comments

Comments
 (0)