Differential D3880

Avoid notices in database tests
ClosedPublic
Actions

Authored by dereckson on Tue, Nov 11, 18:08.

Details

Reviewers

dereckson

Maniphest Tasks

T2188: Fix tests for PHPUnit 12

Commits

rKDBdb0b20fcbf9c: Avoid notices in database tests
rKERUALDba7ec81ee0d3: Avoid notices in database tests

Summary

This fixes the following issue:

notice when trying to read "errno" key
PHPUnit notice and "N" test status, when using createMock but not calling expect

Diff Detail

Repository

rKERUALD Keruald libraries development repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson created this revision.Tue, Nov 11, 18:08

Harbormaster completed remote builds in B6183: Diff 10046.Tue, Nov 11, 18:08

dereckson requested review of this revision.Tue, Nov 11, 18:08

dereckson accepted this revision.Tue, Nov 11, 18:09

This revision is now accepted and ready to land.Tue, Nov 11, 18:09

dereckson edited the summary of this revision. (Show Details)Tue, Nov 11, 18:09

This revision was landed with ongoing or failed builds.Tue, Nov 11, 18:10

Closed by commit rKERUALDba7ec81ee0d3: Avoid notices in database tests (authored by dereckson). · Explain Why

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rKERUALDba7ec81ee0d3: Avoid notices in database tests.

dereckson added a commit: rKDBdb0b20fcbf9c: Avoid notices in database tests.

dereckson added a task: T2188: Fix tests for PHPUnit 12.Tue, Nov 11, 18:38

Revision Contents
Changeset List

Path

Size

database/

tests/

Exceptions/

SqlExceptionTest.php

10 lines

Query/

PDOQueryTest.php

4 lines

Diff 10047

View Options

database/tests/Exceptions/SqlExceptionTest.php