Page MenuHomePhabricator

Correct behavior of "writable" Almanac service binding for repository services
ClosedPublic

Authored by epriestley on Feb 25 2021, 8:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 2:42 PM
Unknown Object (File)
Thu, Nov 14, 6:53 AM
Unknown Object (File)
Thu, Nov 14, 6:41 AM
Unknown Object (File)
Thu, Nov 14, 6:40 AM
Unknown Object (File)
Thu, Nov 14, 6:20 AM
Unknown Object (File)
Wed, Nov 6, 12:25 AM
Unknown Object (File)
Wed, Nov 6, 12:25 AM
Unknown Object (File)
Wed, Nov 6, 12:25 AM
Subscribers
None

Details

Summary

Ref T13611. This property worked correctly when implemented in D19357. The behavior was broken by D20775, which tested node-level routing but did not specifically re-test the "writable" property. This was difficult to spot because ref query outcomes weren't observable in the UI, and the ref itself had the correct property value.

Test Plan

See D21575. After this change, the UI shows the correct state, rather than showing a read-only service ref as writable:

Screen Shot 2021-02-25 at 12.13.24 PM.png (384×721 px, 43 KB)

Diff Detail

Repository
rP Phabricator
Branch
writable2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25170
Build 34738: Run Core Tests
Build 34737: arc lint + arc unit