Page MenuHomePhabricator

Lift bulk tests for "many users against one object" capabilities into "PolicyFilterSet"
ClosedPublic

Authored by epriestley on Feb 5 2021, 1:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 9:02 AM
Unknown Object (File)
Fri, Mar 29, 10:30 PM
Unknown Object (File)
Feb 4 2024, 12:40 AM
Unknown Object (File)
Jan 25 2024, 1:05 PM
Unknown Object (File)
Jan 25 2024, 2:32 AM
Unknown Object (File)
Jan 24 2024, 8:22 PM
Unknown Object (File)
Jan 16 2024, 9:14 PM
Unknown Object (File)
Nov 29 2023, 11:55 PM
Subscribers
None

Details

Summary

Ref T13602. Currently, the policy framework can not execute "test if many users can see one object" particluarly efficiently. This test must be executed more broadly to implement the changes in T13602.

To avoid making this any worse than it already is, lift this block into a wrapper class that has a bulk queue + fetch API and could eventually be optimized.

Test Plan

Viewed a task with an @mention of a user without permission to see it in the summary, saw it rendered in a disabled style.

Diff Detail

Repository
rP Phabricator
Branch
policy1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25100
Build 34635: Run Core Tests
Build 34634: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 13 2021, 9:37 PM
This revision was automatically updated to reflect the committed changes.