Page MenuHomeDevCentral

[DO NOT MERGE] Use Laravel WithoutEvents test trait
AbandonedPublic

Authored by dereckson on Jan 9 2016, 15:16.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 10, 04:28
Unknown Object (File)
Fri, Mar 7, 18:39
Unknown Object (File)
Wed, Mar 5, 09:59
Unknown Object (File)
Tue, Mar 4, 06:16
Unknown Object (File)
Thu, Feb 27, 04:32
Unknown Object (File)
Tue, Feb 25, 15:55
Unknown Object (File)
Fri, Feb 21, 21:32
Unknown Object (File)
Fri, Feb 21, 16:03
Subscribers
None

Details

Reviewers
None
Summary

D195 introduced TestCase::disableEvents, as the Laravel methods
MocksApplicationServices::expectEvents and ::withoutEvents don't
mock the listen method.

We can use the Laravel implementation now it has been fixed at
the framework level.

Test Plan

phpunit --no-coverage

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
use-laravel-events-mock
Build Status
Buildable 261
Build 261: arc lint + arc unit

Event Timeline

dereckson retitled this revision from to [DO NOT MERGE] Use Laravel WithoutEvents test trait.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)

Upstream not interested by the change, but provides a simpler solution.

TestCase::disableEvents()
$this->withoutEvents();
Events::shouldReceive('listen');
[RuntimeException]                                                                             
Error Output: PHP Fatal error:  Class 'Events' not found in /home/ubuntu/workspace/notificati  
ons/tests/TestCase.php on line 44