Skip to content

PHPUnit 10 Shift #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 22, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions tests/Commands/MigrationCreatorTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

namespace Esign\DatabaseTrigger\Commands;

use PHPUnit\Framework\Attributes\Test;
use Esign\DatabaseTrigger\DatabaseTrigger;
use Esign\DatabaseTrigger\Enums\TriggerEvent;
use Esign\DatabaseTrigger\Enums\TriggerTiming;
Expand All @@ -10,10 +11,10 @@
use Mockery;
use PHPUnit\Framework\MockObject\MockObject;

class MigrationCreatorTest extends TestCase
final class MigrationCreatorTest extends TestCase
{
/** @test */
public function it_can_populate_the_stub_file()
#[Test]
public function it_can_populate_the_stub_file(): void
{
$creator = $this->getCreator();
$creator->expects($this->any())->method('getDatePrefix')->willReturn('foo');
Expand Down Expand Up @@ -44,8 +45,8 @@ public function it_can_populate_the_stub_file()
);
}

/** @test */
public function it_can_populate_the_stub_file_providing_a_statement()
#[Test]
public function it_can_populate_the_stub_file_providing_a_statement(): void
{
$creator = $this->getCreator();
$creator->expects($this->any())->method('getDatePrefix')->willReturn('foo');
Expand Down
7 changes: 4 additions & 3 deletions tests/Commands/TriggerListCommandTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,18 @@

namespace Esign\DatabaseTrigger\Tests\Commands;

use PHPUnit\Framework\Attributes\Test;
use Esign\DatabaseTrigger\Commands\TriggerListCommand;
use Esign\DatabaseTrigger\DatabaseTrigger;
use Esign\DatabaseTrigger\Enums\TriggerEvent;
use Esign\DatabaseTrigger\Enums\TriggerTiming;
use Esign\DatabaseTrigger\Facades\Schema;
use Esign\DatabaseTrigger\Tests\TestCase;

class TriggerListCommandTest extends TestCase
final class TriggerListCommandTest extends TestCase
{
/** @test */
public function it_can_display_a_list_of_triggers()
#[Test]
public function it_can_display_a_list_of_triggers(): void
{
Schema::createTrigger('my_trigger', function (DatabaseTrigger $trigger) {
$trigger->on('posts');
Expand Down
7 changes: 4 additions & 3 deletions tests/Commands/TriggerMakeCommandTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,18 @@

namespace Esign\DatabaseTrigger\Tests\Commands;

use PHPUnit\Framework\Attributes\Test;
use Esign\DatabaseTrigger\Commands\MigrationCreator;
use Esign\DatabaseTrigger\Commands\TriggerMakeCommand;
use Esign\DatabaseTrigger\Enums\TriggerEvent;
use Esign\DatabaseTrigger\Enums\TriggerTiming;
use Esign\DatabaseTrigger\Tests\TestCase;
use Illuminate\Support\Composer;

class TriggerMakeCommandTest extends TestCase
final class TriggerMakeCommandTest extends TestCase
{
/** @test */
public function it_can_create_the_trigger_migration()
#[Test]
public function it_can_create_the_trigger_migration(): void
{
$this->mock(MigrationCreator::class, function ($mock) {
$mock->shouldReceive('createTrigger')->once();
Expand Down
27 changes: 14 additions & 13 deletions tests/DatabaseTriggerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,46 +2,47 @@

namespace Esign\DatabaseTrigger\Tests;

use PHPUnit\Framework\Attributes\Test;
use Esign\DatabaseTrigger\DatabaseTrigger;
use Esign\DatabaseTrigger\Enums\TriggerEvent;
use Esign\DatabaseTrigger\Enums\TriggerTiming;

class DatabaseTriggerTest extends TestCase
final class DatabaseTriggerTest extends TestCase
{
/** @test */
public function it_can_set_the_name()
#[Test]
public function it_can_set_the_name(): void
{
$trigger = (new DatabaseTrigger())->name('my_trigger');

$this->assertEquals('my_trigger', $trigger->name);
}

/** @test */
public function it_can_set_the_table_name()
#[Test]
public function it_can_set_the_table_name(): void
{
$trigger = (new DatabaseTrigger())->on('posts');

$this->assertEquals('posts', $trigger->table);
}

/** @test */
public function it_can_set_the_timing()
#[Test]
public function it_can_set_the_timing(): void
{
$trigger = (new DatabaseTrigger())->timing(TriggerTiming::AFTER);

$this->assertEquals(TriggerTiming::AFTER->value, $trigger->timing);
}

/** @test */
public function it_can_set_the_event()
#[Test]
public function it_can_set_the_event(): void
{
$trigger = (new DatabaseTrigger())->event(TriggerEvent::INSERT);

$this->assertEquals(TriggerEvent::INSERT->value, $trigger->event);
}

/** @test */
public function it_can_set_the_statement_as_a_closure()
#[Test]
public function it_can_set_the_statement_as_a_closure(): void
{
$trigger = (new DatabaseTrigger())->statement(function () {
return "SET NEW.title = 'Default title';";
Expand All @@ -50,8 +51,8 @@ public function it_can_set_the_statement_as_a_closure()
$this->assertEquals("SET NEW.title = 'Default title';", $trigger->statement);
}

/** @test */
public function it_can_set_the_statement_as_a_string()
#[Test]
public function it_can_set_the_statement_as_a_string(): void
{
$trigger = (new DatabaseTrigger())->statement("SET NEW.title = 'Default title';");

Expand Down
19 changes: 10 additions & 9 deletions tests/Facades/SchemaTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,17 @@

namespace Esign\DatabaseTrigger\Tests\Facades;

use PHPUnit\Framework\Attributes\Test;
use Esign\DatabaseTrigger\DatabaseTrigger;
use Esign\DatabaseTrigger\Enums\TriggerEvent;
use Esign\DatabaseTrigger\Enums\TriggerTiming;
use Esign\DatabaseTrigger\Facades\Schema;
use Esign\DatabaseTrigger\Tests\TestCase;

class SchemaTest extends TestCase
final class SchemaTest extends TestCase
{
/** @test */
public function it_can_create_a_trigger()
#[Test]
public function it_can_create_a_trigger(): void
{
Schema::createTrigger('my_trigger', function (DatabaseTrigger $trigger) {
$trigger->on('posts');
Expand All @@ -23,8 +24,8 @@ public function it_can_create_a_trigger()
$this->assertTrue(Schema::hasTrigger('my_trigger'));
}

/** @test */
public function it_can_drop_a_trigger()
#[Test]
public function it_can_drop_a_trigger(): void
{
Schema::createTrigger('my_trigger', function (DatabaseTrigger $trigger) {
$trigger->on('posts');
Expand All @@ -38,16 +39,16 @@ public function it_can_drop_a_trigger()
$this->assertFalse(Schema::hasTrigger('my_trigger'));
}

/** @test */
public function it_can_drop_a_trigger_if_it_exists()
#[Test]
public function it_can_drop_a_trigger_if_it_exists(): void
{
Schema::dropTriggerIfExists('my_trigger');

$this->assertFalse(Schema::hasTrigger('my_trigger'));
}

/** @test */
public function it_can_check_if_a_trigger_exists()
#[Test]
public function it_can_check_if_a_trigger_exists(): void
{
$this->assertFalse(Schema::hasTrigger('my_trigger'));

Expand Down