Skip to content

test: Ensure datastore transforms are working for transactions #1412

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 39 commits into from
Sep 15, 2025
Merged

test: Ensure datastore transforms are working for transactions #1412

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
b7f4a1c
Work in progress property transforms
danieljbruce Mar 18, 2025
73b8e15
Change entity back to any
danieljbruce Mar 18, 2025
1179e4e
Merge branch 'main' of https://github.com/googleapis/nodejs-datastore…
danieljbruce Mar 18, 2025
35ee5a4
Add the new interfaces and middleware code
danieljbruce Mar 18, 2025
f181a4c
Pull the build property transform into separate fn
danieljbruce Mar 18, 2025
66e0668
Add arrays to the build transform function
danieljbruce Mar 18, 2025
7abda1f
Add a test for property transforms
danieljbruce Mar 18, 2025
4ae772f
Correct test comments
danieljbruce Mar 18, 2025
c5c4484
More modular build function
danieljbruce Mar 18, 2025
a1e8e69
Update the test to check for the final result
danieljbruce Mar 18, 2025
ef41b73
Add the request spy
danieljbruce Mar 18, 2025
ceb3fc4
Fix the bug making property always increment
danieljbruce Mar 19, 2025
81f1898
Fix the test based on the most recent source code
danieljbruce Mar 19, 2025
c4ac639
Refactor code for the array transforms
danieljbruce Mar 19, 2025
a09119b
Add a guard in case the casted type doesn’t exist
danieljbruce Mar 19, 2025
409f6a6
Remove only
danieljbruce Mar 19, 2025
6f65b43
Add a header
danieljbruce Mar 24, 2025
55b1913
Remove TODO
danieljbruce Mar 24, 2025
2478abc
Remove TODO
danieljbruce Mar 24, 2025
901cbb5
Parameterize the test
danieljbruce May 29, 2025
c99af00
Merge branch 'main' of https://github.com/googleapis/nodejs-datastore…
danieljbruce May 29, 2025
a4710ed
Ran linter
danieljbruce May 29, 2025
ac96630
Run the linter
danieljbruce May 30, 2025
499eca9
Add a comment to buildPropertyTransforms
danieljbruce Sep 10, 2025
b75897e
Add comments for building the property transforms
danieljbruce Sep 10, 2025
c95ac45
Add two more test cases
danieljbruce Sep 10, 2025
a019f07
Build the 3 test cases
danieljbruce Sep 11, 2025
abcfa40
should process string inputs for max/min/increment
danieljbruce Sep 11, 2025
fc79c7e
Produce test cases for different situations
danieljbruce Sep 11, 2025
75a87bd
Delete invalid test cases and run linter
danieljbruce Sep 11, 2025
bc60583
Correct the order of operations test
danieljbruce Sep 11, 2025
0fdf74d
Remove only
danieljbruce Sep 11, 2025
c8801cc
Try mocking out request_
danieljbruce Sep 12, 2025
2c37e04
fix(transaction-spy): Correctly mock DatastoreRequest in Transaction
google-labs-jules[bot] Sep 12, 2025
5ec96d7
Solve transaction tests
danieljbruce Sep 15, 2025
07214b9
delete unused mocks
danieljbruce Sep 15, 2025
282edd2
Delete more unused mocks
danieljbruce Sep 15, 2025
cded5a1
no only
danieljbruce Sep 15, 2025
c7f36ca
Merge branch 'main' of https://github.com/googleapis/nodejs-datastore…
danieljbruce Sep 15, 2025
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
Loading
Loading