document.get() does not support argument 'transactions'

[agherasim]

Reference:

python-mock-firestore/mockfirestore/document.py

Line 42 in 0de34b1

def get(self, field_path: str) -> Any:

While mocking fireo ORM, get queries cannot execute due to missing transactions argument for document.get() method. Google Firestore v1 library provides transactions argument: https://github.com/googleapis/python-firestore/blob/2d42731996586fd63e9b8453b0eb627d3e23a310/google/cloud/firestore_v1/document.py#L362

[ignasbol]

Having the same issue. @agherasim your solution works. Should also update get in DocumentReference class as well, if you are planning to use transaction with references.

[agherasim]

@ignasbol forgot to merge the changes. I pushed the changes addressing DocumentReference, querying works now as well.

[agherasim]

@mdowds would you be able to review this PR and merge the changes?

[alexjolig]

@mdowds Please checkout this PR. I really need this fix to solve this issue in our project.

[agherasim]

@alexjolig, this repo seems unmaintained, unfortunately. What I ended up doing was to create a fork of it, fix issues, and use it as a local dependency.