-
Notifications
You must be signed in to change notification settings - Fork 926
Add explicit tests for Compat #5870
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.
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
Conversation
|
@@ -0,0 +1,18 @@ | |||
<component name="ProjectRunConfigurationManager"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New IntelliJ test runner, inspired by the existing runner for Firestore.
@@ -0,0 +1,18 @@ | |||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
@@ -0,0 +1,7 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
@@ -0,0 +1,240 @@ | |||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
@@ -0,0 +1,378 @@ | |||
/** | |||
* @license |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
@@ -0,0 +1,82 @@ | |||
/** | |||
* @license | |||
* Copyright 2022 Google LLC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly copied
* limitations under the License. | ||
*/ | ||
|
||
import * as firestore from '@firebase/firestore-types'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
* limitations under the License. | ||
*/ | ||
|
||
export interface Resolver<R> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
* limitations under the License. | ||
*/ | ||
|
||
import * as firestore from '@firebase/firestore-types'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly copied
@@ -0,0 +1,1425 @@ | |||
/** | |||
* @license |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copied
Size Report 1Affected ProductsNo changes between base commit (f873ad4) and merge commit (fc5b806).Test Logs |
Size Analysis Report 1Affected ProductsNo changes between base commit (f873ad4) and merge commit (fc5b806).Test Logs |
This adds explicit integration tests for Firestore Compat, as I am rewriting the existing integration tests to use the v9 API. Most of the files are just copied. Exceptions are noted.