Paths

Table of Contentst

Differential D216

Not operational broker
ClosedPublic
Actions

Authored by dereckson on Jan 10 2016, 03:48.

Details

Reviewers

dereckson

Commits

Unknown Object (Diffusion Commit)
rNOTIF85bf57953e2f: Not operational broker

Summary

If something wrong occurs at the broker level, ensure we've a nice
exception attached to our report and a 503 status code.

This ensures a full coverage of the SendMessageToBroker job, as we
test sendMessage stores the exception and report attaches it to the
event to add to the report.

Test Plan

phpunit

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Passed

Unit

Tests Passed

Branch

test/broker-throws-exception

Event Timeline

dereckson updated this revision to Diff 510.Jan 10 2016, 03:48

dereckson retitled this revision from to Not operational broker.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson updated this object.

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

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

dereckson accepted this revision.Jan 10 2016, 03:55

dereckson updated this object.

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jan 10 2016, 03:55

Closed by commit rNOTIF85bf57953e2f: Not operational broker (authored by dereckson, committed by dereckson). · Explain WhyJan 10 2016, 03:56

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

tests/

Http/

PayloadFullTest.php

23 lines

TestCase.php

13 lines

Diff 510

View Options

tests/Http/PayloadFullTest.php