Skip to content

Commit 5f48169

Browse files
committed
test(firestore): update nested fields test to reflect backend change
The backend now supports nested field modification in the addFields and select changes. All aliases with a `.` are now treated as a nested field path. This change updates the tests to reflect this new behaviour. This change is still in nightly. Do not merge until this is in prod.
1 parent d7b1826 commit 5f48169

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

packages/firestore/test/integration/api/pipeline.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2879,7 +2879,9 @@ apiDescribe.skipClassic('Pipelines', persistence => {
28792879
snapshot,
28802880
{
28812881
title: "The Hitchhiker's Guide to the Galaxy",
2882-
'awards.hugo': true
2882+
awards: {
2883+
hugo: true
2884+
}
28832885
},
28842886
{ title: 'Dune', 'awards.hugo': true }
28852887
);

packages/firestore/test/lite/pipeline.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2851,9 +2851,11 @@ describe.skipClassic('Firestore Pipelines', () => {
28512851
snapshot,
28522852
{
28532853
title: "The Hitchhiker's Guide to the Galaxy",
2854-
'awards.hugo': true
2854+
awards: {
2855+
hugo: true
2856+
}
28552857
},
2856-
{ title: 'Dune', 'awards.hugo': true }
2858+
{ title: 'Dune', awards: { hugo: true } }
28572859
);
28582860
});
28592861

0 commit comments

Comments
 (0)