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.

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

System test coverage for Cloud Spanner #2197

Closed
stephenplusplus opened this issue Apr 10, 2017 · 3 comments
Closed

System test coverage for Cloud Spanner #2197

stephenplusplus opened this issue Apr 10, 2017 · 3 comments
Assignees
Labels
api: spanner Issues related to the Spanner API.

Comments

@stephenplusplus
Copy link
Contributor

From @vkedia on January 19, 2017 7:20

System tests for Cloud Spanner should cover all these scenarios:

  1. Basic read,query and writes for all the data types.
  2. Read with indexes, read limits, different concurrency options, range reads.
  3. Reading back large amount of data (lot of rows, large rows) to exercise the stream resume, flow control and chunking logic.
  4. Parameter binding in query for all the data types including nulls, null arrays, arrays containing null, array of structs, special values for double type (Nan, Infinity,-Infinity).
  5. Read/modify/write using transactions.
  6. Tests for admin operations such as creating/deleting a database, updating database schema, pagination for list operations.

Copied from original issue: GoogleCloudPlatform/google-cloud-node-private#20

@stephenplusplus
Copy link
Contributor Author

From @callmehiphop on February 2, 2017 23:59

@stephenplusplus is there anything in here that we don't already cover via system tests?

@stephenplusplus
Copy link
Contributor Author

Bold = needs tests

  1. Basic read,query and writes for all the data types. (We're missing null)
  2. Read with indexes, read limits, different concurrency options, range reads.
  3. Reading back large amount of data (lot of rows, large rows) to exercise the stream resume, flow control and chunking logic. (only tested in unit tests)
  4. Parameter binding in query for all the data types including nulls, null arrays, arrays containing null, array of structs, special values for double type (Nan, Infinity,-Infinity).
  5. Read/modify/write using transactions. (Not sure about the transactions)
  6. Tests for admin operations such as creating/deleting a database, updating database schema, pagination for list operations.

@stephenplusplus stephenplusplus added api: spanner Issues related to the Spanner API. testing and removed spanner labels Apr 10, 2017
@Chipintoza
Copy link

It will be good if you add some tests for this issues #2176

@lukesneeringer lukesneeringer added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 13, 2017
@lukesneeringer lukesneeringer removed the priority: p2 Moderately-important priority. Fix may not be included in next release. label May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the Spanner API.
Projects
None yet
Development

No branches or pull requests

4 participants