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)
Jan 9 2024, 10:37 AM
Unknown Object (File)
Nov 30 2023, 4:09 AM
Unknown Object (File)
Nov 26 2023, 12:16 AM
Unknown Object (File)
Nov 17 2023, 12:11 AM
Unknown Object (File)
Sep 7 2023, 4:23 AM
Unknown Object (File)
Jul 23 2023, 8:10 PM
Unknown Object (File)
Jul 16 2023, 8:17 PM
Unknown Object (File)
Jul 10 2023, 4:44 PM
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