Skip to content

'Config.IN_EMULATOR' is a boolean.#2381

Merged
tseaver merged 1 commit intogoogleapis:masterfrom
tseaver:system_tests-bigtable-false_positive_emulator_test
Sep 21, 2016
Merged

'Config.IN_EMULATOR' is a boolean.#2381
tseaver merged 1 commit intogoogleapis:masterfrom
tseaver:system_tests-bigtable-false_positive_emulator_test

Conversation

@tseaver
Copy link
Copy Markdown
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
Copy Markdown
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
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Apr 1, 2026
…ps (#2381)

This change adds null literal checks during the compilation of eq, ne
and map ops. This aims to resolve the
`test_series_replace_nans_with_pd_na` failure reported in #2248.

Fixes internal issue 417774347 🦕
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