Skip to content

Update Lite tests with Query API v2 #3446

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Jul 23, 2020
Merged

Update Lite tests with Query API v2 #3446

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
596f9c6
Update and enable Lite tests
schmidt-sebastian Jul 21, 2020
b3a614c
Add Lite runner
schmidt-sebastian Jul 21, 2020
32a051b
Keep options optional
schmidt-sebastian Jul 21, 2020
b5eacf0
Fix compile
schmidt-sebastian Jul 21, 2020
4f4eb4e
Merge branch 'master' into mrschmidt/fixlitetests
schmidt-sebastian Jul 22, 2020
9f5ad5a
Wrap
schmidt-sebastian Jul 22, 2020
f370f53
Add Externs
schmidt-sebastian Jul 22, 2020
8f77f4b
Remove prebuilt
schmidt-sebastian Jul 22, 2020
9f88280
Skip update test
schmidt-sebastian Jul 22, 2020
70d39b4
Lint
schmidt-sebastian Jul 22, 2020
49e8689
Merge
schmidt-sebastian Jul 22, 2020
feb2cc2
Fix tests
schmidt-sebastian Jul 22, 2020
e3fa6a0
Fix import issue
schmidt-sebastian Jul 23, 2020
acde11c
Prettier
schmidt-sebastian Jul 23, 2020
1a07971
Run test in sequence
schmidt-sebastian Jul 23, 2020
dab045f
More test runner optimizations
schmidt-sebastian Jul 23, 2020
3fbacc2
run-s
schmidt-sebastian Jul 23, 2020
59f554b
WIP
schmidt-sebastian Jul 23, 2020
addcda1
Think it should work now
schmidt-sebastian Jul 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions packages/firestore/.idea/runConfigurations/firestore_exp_Tests__Emulator__.xml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions .../firestore/.idea/runConfigurations/firestore_exp_Tests__Emulator_w__Mock_Persistence_.xml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 19 additions & 0 deletions packages/firestore/.idea/runConfigurations/firestore_lite_Tests__Emulator_.xml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 5 additions & 2 deletions packages/firestore/exp-types/index.d.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ export interface FirestoreDataConverter<T> {
toFirestore(modelObject: Partial<T>, options: SetOptions): DocumentData;
fromFirestore(
snapshot: QueryDocumentSnapshot<DocumentData>,
options: SnapshotOptions
options?: SnapshotOptions
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed to make this interface compatible with the FirestoreDataConverter in lite.

): T;
}

Expand Down Expand Up @@ -242,9 +242,11 @@ export type SetOptions =
export class DocumentReference<T = DocumentData> {
private constructor();
readonly type: 'document';
readonly id: string;
readonly firestore: FirebaseFirestore;
readonly converter: FirestoreDataConverter<T> | null;
readonly path: string;
readonly id: string;

withConverter<U>(converter: FirestoreDataConverter<U>): DocumentReference<U>;
}

Expand Down Expand Up @@ -284,6 +286,7 @@ export class Query<T = DocumentData> {
protected constructor();
readonly type: 'query' | 'collection';
readonly firestore: FirebaseFirestore;
readonly converter: FirestoreDataConverter<T> | null;

withConverter<U>(converter: FirestoreDataConverter<U>): Query<U>;
}
Expand Down
Loading