Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Using query order in test_projection_query.#1372

Merged
dhermes merged 1 commit intogoogleapis:masterfrom
dhermes:test-proj-query-order
Jan 8, 2016
Merged

Using query order in test_projection_query.#1372
dhermes merged 1 commit intogoogleapis:masterfrom
dhermes:test-proj-query-order

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 8, 2016

This way, the results are in the same order in both production and the emulator.

See comment in #1369 for context.

This way, the results are in the same order in both
production and the emulator.
@dhermes dhermes added api: datastore Issues related to the Datastore API. testing labels Jan 8, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 8, 2016
@tseaver
Copy link
Contributor

tseaver commented Jan 8, 2016

LGTM

dhermes added a commit that referenced this pull request Jan 8, 2016
Using query order in test_projection_query.
@dhermes dhermes merged commit 0c74a59 into googleapis:master Jan 8, 2016
@dhermes dhermes deleted the test-proj-query-order branch January 8, 2016 22:34
parthea pushed a commit that referenced this pull request Mar 6, 2026
parthea pushed a commit that referenced this pull request Mar 9, 2026
* test: add explicit credentials and project to test

Tests need to be able to run in environments without any default credentials.
This requires the test to explictly set the credentials and the project that
should be used.

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants