Skip to content
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

Jump to bottom

Additional spanner integration tests. #4364

Closed
19 tasks done
tseaver opened this issue Nov 8, 2017 · 5 comments
Closed
19 tasks done

Additional spanner integration tests. #4364

tseaver opened this issue Nov 8, 2017 · 5 comments
Assignees
@tseaver
Labels
api: spanner Issues related to the Spanner API. testing type: process A process-related concern. May include testing, release, or the like.

Comments

@tseaver
Copy link
Contributor

tseaver commented Nov 8, 2017 •
edited by chemelnucfin
Loading

For spanner GA. Investigate whether these are already tested, and add tests as appropriate. Link either PR # or line reference of existing test.

/cc @lukesneeringer
@tseaver tseaver added api: spanner Issues related to the Spanner API. testing type: process A process-related concern. May include testing, release, or the like. labels Nov 8, 2017
@tseaver
Copy link
Contributor Author

tseaver commented Nov 9, 2017

@chemelnucfin Thanks for the updates! As we each begin adding one or more related tests, let's adopt the covention of marking the checkbox items with our handles (to avoid duplicating effort).

@tseaver
Copy link
Contributor Author

tseaver commented Nov 9, 2017

@chemelnucfin I had to uncheck some of the boxes :( -- the test description was missing "over index".

@tseaver tseaver mentioned this issue Nov 9, 2017
Merged
tseaver added a commit that referenced this issue Nov 9, 2017
@tseaver
Add spanner system tests:
dcf20cd 
- Write / read back null string.
- Write / read back empty array of string.
- Write / read back null array of string.
- Write / read back array of string with mixed null / non-null values.

Toward #4364.
@tseaver tseaver mentioned this issue Nov 9, 2017
Merged
tseaver added a commit that referenced this issue Nov 9, 2017
@tseaver
Add spanner system tests:
79922ca 
- Write / read back null string.
- Write / read back empty array of string.
- Write / read back null array of string.
- Write / read back array of string with mixed null / non-null values.

Toward #4364.
tseaver added a commit that referenced this issue Nov 10, 2017
@tseaver
Add spanner system tests: (#4372)
c353f2b 
- Write / read back null string.
- Write / read back empty array of string.
- Write / read back null array of string.
- Write / read back array of string with mixed null / non-null values.

Toward #4364.
@chemelnucfin
Copy link
Contributor

What is range read with partial key?

@chemelnucfin
Copy link
Contributor

Never mind, got it.

@chemelnucfin
Copy link
Contributor

Closed via PRs from above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tseaver tseaver

Labels
api: spanner Issues related to the Spanner API. testing type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tseaver @chemelnucfin