Skip to content

Commit 857509b

Browse files
Update cedar-policy-core/src/entities protobuf tests to take Arc wrapped entities into account.
Signed-off-by: Thomas Hill <[email protected]>
1 parent 11e70bd commit 857509b

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

cedar-policy-core/src/entities.rs

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ impl From<&proto::Entities> for Entities {
369369
// PANIC SAFETY: experimental feature
370370
#[allow(clippy::expect_used)]
371371
fn from(v: &proto::Entities) -> Self {
372-
let entities: Vec<Entity> = v.entities.iter().map(Entity::from).collect();
372+
let entities: Vec<Arc<Entity>> = v.entities.iter().map(|e| Arc::new(Entity::from(e))).collect();
373373

374374
#[cfg(not(feature = "partial-eval"))]
375375
let result = Entities::new();
@@ -3546,14 +3546,13 @@ pub mod protobuf_tests {
35463546
let attrs = (1..=7)
35473547
.map(|id| (format!("{id}").into(), RestrictedExpr::val(true)))
35483548
.collect::<HashMap<SmolStr, _>>();
3549-
let entity: Entity = Entity::new(
3549+
let entity: Arc<Entity> = Arc::new(Entity::new(
35503550
r#"Foo::"bar""#.parse().unwrap(),
35513551
attrs.clone(),
35523552
HashSet::new(),
35533553
BTreeMap::new(),
35543554
&Extensions::none(),
3555-
)
3556-
.unwrap();
3555+
).unwrap());
35573556
let mut entities2: Entities = Entities::new();
35583557
entities2 = entities2
35593558
.add_entities(
@@ -3569,14 +3568,13 @@ pub mod protobuf_tests {
35693568
);
35703569

35713570
// Two Element Test
3572-
let entity2: Entity = Entity::new(
3571+
let entity2: Arc<Entity> = Arc::new(Entity::new(
35733572
r#"Bar::"foo""#.parse().unwrap(),
35743573
attrs.clone(),
35753574
HashSet::new(),
35763575
BTreeMap::new(),
35773576
&Extensions::none(),
3578-
)
3579-
.unwrap();
3577+
).unwrap());
35803578
let mut entities3: Entities = Entities::new();
35813579
entities3 = entities3
35823580
.add_entities(

0 commit comments

Comments
 (0)