-
Notifications
You must be signed in to change notification settings - Fork 891
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
Transaction.create is missing from @firebase/testing #2633
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
Why do you need a
|
Are we talking about different export class Transaction {
...
/**
* Create the document referred to by the provided `DocumentReference`.
* The operation will fail the transaction if a document exists at the
* specified location.
*
* @param documentRef A reference to the document to be create.
* @param data The object data to serialize as the document.
* @return This `Transaction` instance. Used for chaining method calls.
*/
create<T>(documentRef: DocumentReference<T>, data: T): Transaction;
}
|
Yes, we are talking about different transactions. The I see you were no happy with my reply earlier, I apologize for not explaining this with deserved details. |
It's unfortunate in this context that In some sense this is a duplicate of #1324. Rules testing is an interesting new motivating use case for this though so we'll bump our internal priority on this in the backlog. PRs welcome in the meantime :-). This is b/129358230 internally. |
@wu-hui, got it. Thank you. Honestly speaking it's not that obvious where But there're a bunch of issues when Only some issue on the github hinted me that firebase emulator does work well with |
[REQUIRED] Describe your environment
Linux 7u 5.4.5-050405-generic #201912181630 SMP Wed Dec 18 16:33:40 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
@firebase/testing
[REQUIRED] Describe the problem
Steps to reproduce:
Use
Transaction.create
on testing env.Transaction.set
works well.Relevant Code:
Approx. code:
The text was updated successfully, but these errors were encountered: