Skip to content

Commit 4313ad0

Browse files
committed
Migrate deprecated PHPUnit PHPDoc tags to PHPUnit attributes
1 parent 695244d commit 4313ad0

File tree

1 file changed

+9
-13
lines changed

1 file changed

+9
-13
lines changed

tests/SharedDatabasesTest.php

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
use ipl\Sql\Connection;
66
use ipl\Sql\Select;
77
use ipl\Sql\Test\SharedDatabases;
8+
use PHPUnit\Framework\Attributes\DataProvider;
9+
use PHPUnit\Framework\Attributes\Depends;
810

911
/**
1012
* A test for a test component! Yay!
@@ -13,7 +15,7 @@ class SharedDatabasesTest extends TestCase
1315
{
1416
use SharedDatabases;
1517

16-
/** @dataProvider sharedDatabases */
18+
#[DataProvider('sharedDatabases')]
1719
public function testInsert(Connection $db)
1820
{
1921
// This is the first case, so the table must have been dropped and be empty
@@ -24,10 +26,8 @@ public function testInsert(Connection $db)
2426
$db->insert('test', ['name' => 'test2']);
2527
}
2628

27-
/**
28-
* @depends testInsert
29-
* @dataProvider sharedDatabases
30-
*/
29+
#[Depends('testInsert')]
30+
#[DataProvider('sharedDatabases')]
3131
public function testSelect(Connection $db)
3232
{
3333
// The previous case inserts "name=test" but tearDown removes it
@@ -36,20 +36,16 @@ public function testSelect(Connection $db)
3636
$this->assertSame('test2', $result[0]['name']);
3737
}
3838

39-
/**
40-
* @depends testSelect
41-
* @dataProvider sharedDatabases
42-
*/
39+
#[Depends('testSelect')]
40+
#[DataProvider('sharedDatabases')]
4341
public function testUpdate(Connection $db)
4442
{
4543
$stmt = $db->update('test', ['name' => 'test3'], ['name = ?' => 'test2']);
4644
$this->assertEquals(1, $stmt->rowCount());
4745
}
4846

49-
/**
50-
* @depends testUpdate
51-
* @dataProvider sharedDatabases
52-
*/
47+
#[Depends('testUpdate')]
48+
#[DataProvider('sharedDatabases')]
5349
public function testDelete(Connection $db)
5450
{
5551
$stmt = $db->delete('test', ['name = ?' => 'test3']);

0 commit comments

Comments
 (0)