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

'Config.IN_EMULATOR' is a boolean. #2381

Merged
merged 1 commit into from
Sep 21, 2016
Merged

'Config.IN_EMULATOR' is a boolean. #2381

merged 1 commit into from
Sep 21, 2016

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Sep 21, 2016

Five tests were being skipped due to false positves (testing if Config.IN_EMULATOR is not None: vs. if Config.IN_EMULATOR:).

Five tests were being skipped due to false positves (testing
'if Config.IN_EMULATOR is not None:' vs. 'if Config.IN_EMULATOR:'.
@tseaver tseaver added testing api: bigtable Issues related to the Bigtable API. labels Sep 21, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 21, 2016
@dhermes
Copy link
Contributor

dhermes commented Sep 21, 2016

DERP! Sorry @tseaver. Should've been a boolean all along. My bad on not being careful with my find and replace

LGTM

@tseaver tseaver merged commit c8a2517 into googleapis:master Sep 21, 2016
@tseaver tseaver deleted the system_tests-bigtable-false_positive_emulator_test branch September 21, 2016 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the Bigtable 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.

4 participants