Page MenuHomeDevCentral
Differential D333

APIClient and APIFactory contracts
ClosedPublic
Actions

Authored by dereckson on Mar 9 2016, 05:56.
Tags
None
Referenced Files
F10826253: D333.id1477.diff
Mon, Jul 28, 01:10
F10825335: D333.id.diff
Mon, Jul 28, 00:15
F10823320: D333.id1477.diff
Sun, Jul 27, 22:29
F10820797: D333.id1475.diff
Sun, Jul 27, 19:50
F10819314: D333.id793.diff
Sun, Jul 27, 18:07
F10818649: D333.diff
Sun, Jul 27, 17:40
Unknown Object (File)
Sat, Jul 26, 21:22
Unknown Object (File)
Sat, Jul 26, 20:18
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIFab12e8289918: APIClient and APIFactory contracts
Summary

To improve dependency injection in ProjectsMap, we create two
interface contracts, APIClient and APIFactory, which abstract
an API client and a class to instantiate them.

That allow to easily inject an API client and increase components
independance.

Test Plan

A variation of fea5b9c has successfully been tested.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
api-contracts

Event Timeline

dereckson updated this revision to Diff 793.Mar 9 2016, 05:56
dereckson retitled this revision from to APIClient and APIFactory contracts.
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.Mar 9 2016, 05:57

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

alken-orin added a comment.Mar 9 2016, 06:09

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

dereckson updated this revision to Diff 1475.Aug 10 2016, 19:17
dereckson edited edge metadata.

Rebased against master.

dereckson mentioned this in D334: API dependency injection for ProjectsMap.Aug 10 2016, 19:24
dereckson accepted this revision.Aug 10 2016, 19:47
dereckson edited edge metadata.
This revision is now accepted and ready to land.Aug 10 2016, 19:47
Closed by commit rNOTIFab12e8289918: APIClient and APIFactory contracts (authored by dereckson, committed by dereckson). · Explain WhyAug 10 2016, 19:49
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
app/
Contracts/
24 lines--
15 lines--
Phabricator/
34 lines--
4 lines100%
Loading...
29 lines91%
Loading...

Diff 1475

View Options

app/Contracts/APIClient.php

Loading...
View Options

app/Contracts/APIFactory.php

Loading...
View Options

app/Phabricator/PhabricatorAPI.php

Loading...
View Options

app/Phabricator/PhabricatorAPIFactory.php

Loading...
View Options

app/Phabricator/ProjectsMap.php

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