Page MenuHomePhabricator

Make the Pholio Mock "getImages" / "getAllImages" API more clear

Authored by epriestley on Dec 20 2018, 7:48 PM.



Depends on D19920. Ref T11351. Currently, "images" and "all images" are attached to Mocks separately, and getImages() gets you only some images.

Clean this up slightly:

  • One attach method; attach everything.
  • Two getters, one for "images" (returns all images); one for "active images" (returns active images).
Test Plan

Browsed around Pholio without any apparent behavioral changes.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Dec 20 2018, 7:48 PM
epriestley requested review of this revision.Dec 20 2018, 7:50 PM
epriestley added inline comments.Dec 20 2018, 7:53 PM

This was a bug, should have asserted allImages instead.

amckinley accepted this revision.Dec 20 2018, 10:00 PM
This revision is now accepted and ready to land.Dec 20 2018, 10:00 PM
This revision was automatically updated to reflect the committed changes.