Page MenuHomePhabricator

Give "FileAttachment" policy support and a query object
ClosedPublic

Authored by epriestley on May 23 2022, 5:24 PM.
Tags
None
Referenced Files
F13099618: D21835.id52039.diff
Fri, Apr 26, 3:44 PM
Unknown Object (File)
Thu, Apr 25, 3:25 AM
Unknown Object (File)
Thu, Apr 25, 2:22 AM
Unknown Object (File)
Wed, Apr 24, 1:13 PM
Unknown Object (File)
Tue, Apr 23, 4:44 PM
Unknown Object (File)
Sat, Apr 20, 5:59 PM
Unknown Object (File)
Sat, Apr 20, 3:35 PM
Unknown Object (File)
Sat, Apr 20, 12:21 AM
Subscribers
None

Details

Summary

Ref T13682. This supports an "Attached Files" curtain UI element.

Test Plan

See next change.

Diff Detail

Repository
rP Phabricator
Branch
file4
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 25739
Build 35567: arc lint + arc unit

Unit TestsFailed

TimeTest
45 msPhabricatorFileTestCase::testFileIndirectScramble
Assertion failed, expected 'true' (at PhabricatorFileTestCase.php:140): Changing attached object view policy should scramble secret. ACTUAL VALUE
213 msPhabricatorCelerityTestCase::testCelerityMaps
3 assertions passed.
9 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
34 msPhabricatorFileTestCase::testFileDirectScramble
2 assertions passed.
10 msPhabricatorFileTestCase::testFileStorageDelete
1 assertion passed.
View Full Test Results (1 Failed · 16 Passed)

Event Timeline

epriestley created this revision.
  • Also fix indirect reference attachment/scrambling test.
  • Further improve unit test coverage and test "reference doesn't grant view permission".
This revision was not accepted when it landed; it landed in state Needs Review.May 23 2022, 9:08 PM
This revision was automatically updated to reflect the committed changes.