Page MenuHomeDevCentral
Differential D281

assertEquals → assertSame in unit tests
ClosedPublic
Actions

Authored by dereckson on Feb 11 2016, 02:04.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 16:36
Unknown Object (File)
Sun, Nov 24, 16:36
Unknown Object (File)
Sun, Nov 24, 16:36
Unknown Object (File)
Tue, Nov 12, 15:35
Unknown Object (File)
Sun, Nov 10, 13:16
Unknown Object (File)
Thu, Nov 7, 22:15
Unknown Object (File)
Thu, Nov 7, 14:59
Unknown Object (File)
Oct 27 2024, 18:09
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIFead5f15a8796: assertEquals → assertSame in unit tests
Summary

The tests now use assertSame to assert type is respected too.

We keep assertEquals in EventServiceProviderTest::testOmittedFiles,
as we there compare two arrays, regardless of the order of the items.

Test Plan

phpunit

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 658.Feb 11 2016, 02:04
dereckson retitled this revision from to assertEquals → assertSame in unit tests.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
alken-orin added a subscriber: alken-orin.Feb 11 2016, 02:04

Build started: https://ci.nasqueron.org/job/test-notifications-phab/77/ (console: https://ci.nasqueron.org/job/test-notifications-phab/77/console)

dereckson planned changes to this revision.Feb 11 2016, 02:04
dereckson added inline comments.
tests/Actions/ActionsReportTest.php
61

This is an assertEquals too.

dereckson updated this revision to Diff 659.Feb 11 2016, 02:08
dereckson edited edge metadata.

assertEquals in suggestion to fix a testReport issue

alken-orin added a comment.Feb 11 2016, 02:10

Build is green https://ci.nasqueron.org/job/test-notifications-phab/77/ for more details.

alken-orin added a comment.Feb 11 2016, 02:10

Build started: https://ci.nasqueron.org/job/test-notifications-phab/78/ (console: https://ci.nasqueron.org/job/test-notifications-phab/78/console)

alken-orin added a comment.Feb 11 2016, 02:16

Build is green https://ci.nasqueron.org/job/test-notifications-phab/78/ for more details.

dereckson accepted this revision.Feb 11 2016, 04:36
dereckson edited edge metadata.
This revision is now accepted and ready to land.Feb 11 2016, 04:36
Closed by commit rNOTIFead5f15a8796: assertEquals → assertSame in unit tests (authored by dereckson, committed by dereckson). · Explain WhyFeb 11 2016, 04:37
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
tests/
Actions/
2 lines
10 lines
2 lines
Analyzers/
GitHub/
Events/
26 lines
4 lines
Notifications/
14 lines
Providers/
4 lines
Services/
12 lines

Diff 661

View Options

tests/Actions/AMQPActionTest.php

Loading...
View Options

tests/Actions/ActionsReportTest.php

Loading...
View Options

tests/Actions/NotifyNewCommitsActionTest.php

Loading...
View Options

tests/Analyzers/GitHub/Events/PushEventTest.php

Loading...
View Options

tests/Analyzers/GitHub/GitHubPayloadAnalyzerConfigurationTest.php

Loading...
View Options

tests/Notifications/DockerHubNotificationTest.php

Loading...
View Options

tests/Providers/ServicesServiceProviderTest.php

Loading...
View Options

tests/Services/ServicesTest.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator