Skip to content

Commit 804ad63

Browse files
committed
Add more tests
1 parent 3101d82 commit 804ad63

File tree

4 files changed

+1330
-1253
lines changed

4 files changed

+1330
-1253
lines changed

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

+10-87
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@
1616
*/
1717

1818
import { expect } from 'chai';
19-
20-
import { RealtimePipeline } from '../../../src/api/realtime_pipeline';
2119
import {
2220
RealtimePipelineSnapshot,
2321
ResultChange
@@ -31,6 +29,7 @@ import {
3129
Bytes,
3230
collection,
3331
collectionGroup,
32+
CollectionReference,
3433
deleteDoc,
3534
disableNetwork,
3635
doc,
@@ -41,6 +40,7 @@ import {
4140
enableNetwork,
4241
endAt,
4342
endBefore,
43+
Firestore,
4444
GeoPoint,
4545
getDocs as getDocsProd,
4646
limit,
@@ -51,109 +51,32 @@ import {
5151
query,
5252
QuerySnapshot,
5353
setDoc,
54+
setLogLevel,
5455
startAfter,
5556
startAt,
5657
Timestamp,
5758
updateDoc,
5859
where,
5960
writeBatch,
60-
CollectionReference,
61-
WriteBatch,
62-
Firestore,
63-
Query,
64-
Unsubscribe,
65-
setLogLevel,
66-
getDocsFromCache
61+
WriteBatch
6762
} from '../util/firebase_export';
6863
import {
6964
apiDescribe,
65+
apiPipelineDescribe,
66+
checkOnlineAndOfflineResultsMatchWithPipelineMode,
67+
getDocs,
68+
onSnapshot,
69+
PERSISTENCE_MODE_UNSPECIFIED,
7070
RetryError,
7171
toChangesArray,
7272
toDataArray,
73-
PERSISTENCE_MODE_UNSPECIFIED,
7473
withEmptyTestCollection,
7574
withRetry,
7675
withTestCollection,
77-
withTestDb,
78-
apiPipelineDescribe,
79-
PipelineMode,
80-
checkOnlineAndOfflineResultsMatchWithPipelineMode
76+
withTestDb
8177
} from '../util/helpers';
8278
import { USE_EMULATOR } from '../util/settings';
8379
import { captureExistenceFilterMismatches } from '../util/testing_hooks_util';
84-
import { pipelineFromStages } from '../../util/pipelines';
85-
86-
function getDocs(
87-
pipelineMode: PipelineMode,
88-
queryOrPipeline: Query | RealtimePipeline
89-
) {
90-
if (pipelineMode === 'query-to-pipeline') {
91-
if (queryOrPipeline instanceof Query) {
92-
const ppl = queryOrPipeline.firestore
93-
.pipeline()
94-
.createFrom(queryOrPipeline);
95-
return getDocsProd(
96-
new RealtimePipeline(
97-
ppl._db,
98-
ppl.userDataReader,
99-
ppl._userDataWriter,
100-
ppl.stages
101-
)
102-
);
103-
} else {
104-
return getDocsProd(queryOrPipeline);
105-
}
106-
}
107-
108-
return getDocsProd(queryOrPipeline as Query);
109-
}
110-
111-
function onSnapshot(
112-
pipelineMode: PipelineMode,
113-
queryOrPipeline: Query | RealtimePipeline,
114-
observer: unknown
115-
): Unsubscribe;
116-
function onSnapshot(
117-
pipelineMode: PipelineMode,
118-
queryOrPipeline: Query | RealtimePipeline,
119-
options: unknown,
120-
observer: unknown
121-
): Unsubscribe;
122-
function onSnapshot(
123-
pipelineMode: PipelineMode,
124-
queryOrPipeline: Query | RealtimePipeline,
125-
optionsOrObserver: unknown,
126-
observer?: unknown
127-
): Unsubscribe {
128-
const obs = observer || optionsOrObserver;
129-
const options = observer
130-
? optionsOrObserver
131-
: {
132-
includeMetadataChanges: false,
133-
source: 'default'
134-
};
135-
if (pipelineMode === 'query-to-pipeline') {
136-
if (queryOrPipeline instanceof Query) {
137-
const ppl = queryOrPipeline.firestore
138-
.pipeline()
139-
.createFrom(queryOrPipeline);
140-
return onSnapshotProd(
141-
new RealtimePipeline(
142-
ppl._db,
143-
ppl.userDataReader,
144-
ppl._userDataWriter,
145-
ppl.stages
146-
),
147-
options as any,
148-
obs as any
149-
);
150-
} else {
151-
return onSnapshotProd(queryOrPipeline, options as any, obs as any);
152-
}
153-
}
154-
155-
return onSnapshotProd(queryOrPipeline as Query, options as any, obs as any);
156-
}
15780

15881
function results(outputs: RealtimePipelineSnapshot | QuerySnapshot) {
15982
if (outputs instanceof RealtimePipelineSnapshot) {

0 commit comments

Comments
 (0)