Yiisoft\ActiveRecord\Tests\Driver\Mssql\ActiveRecordTest::testSaveWithTrigger()

Complexity

Conditions	1
Paths	1

Size

Total Lines	36
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function testSaveWithTrigger(): void
    {
        $this->checkFixture($this->db(), 'test_trigger');

        // drop trigger if exist
        $sql = <<<SQL
        IF (OBJECT_ID('[dbo].[test_alert]') IS NOT NULL)
        BEGIN
            DROP TRIGGER [dbo].[test_alert];
        END
        SQL;
        $this->db()->createCommand($sql)->execute();

        // create trigger
        $sql = <<<SQL
        CREATE TRIGGER [dbo].[test_alert] ON [dbo].[test_trigger]
        AFTER INSERT
        AS
        BEGIN
            INSERT INTO [dbo].[test_trigger_alert] ( [stringcol] )
            SELECT [stringcol]
            FROM [inserted]
        END
        SQL;
        $this->db()->createCommand($sql)->execute();

        $record = new TestTrigger($this->db());

The call to Yiisoft\ActiveRecord\Tests\Stubs\ActiveRecord\TestTrigger::__construct() has too many arguments starting with $this->db().

        $record->stringcol = 'test';

        $this->assertTrue($record->save());
        $this->assertEquals(1, $record->id);

        $testRecordQuery = new ActiveQuery(TestTriggerAlert::class);

        $this->assertEquals('test', $testRecordQuery->findOne(1)->stringcol);

Accessing stringcol on the interface Yiisoft\ActiveRecord\ActiveRecordInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }