Skip to content

Bigtable: remove integration tests from the default failsafe execution - #5955

Merged
igorbernstein2 merged 1 commit into
googleapis:masterfrom
igorbernstein2:no-default-its
Aug 1, 2019
Merged

Bigtable: remove integration tests from the default failsafe execution#5955
igorbernstein2 merged 1 commit into
googleapis:masterfrom
igorbernstein2:no-default-its

Conversation

@igorbernstein2

Copy link
Copy Markdown
Contributor

This was a bug introduced in #5756. When integration profiles were introduced,
I forgot to remove the test from the default execution of the failsafe plugin.
The new way integration tests are run are via profile activation: by default
the bigtable-emulator-it profile is active.

This was a bug introduced in googleapis#5756. When integration profiles were introduced,
I forgot to remove the test from the default execution of the failsafe plugin.
The new way integration tests are run are via profile activation: by default
the bigtable-emulator-it profile is active.
@igorbernstein2igorbernstein2 added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: bigtable Issues related to the Bigtable API. labels Aug 1, 2019
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 1, 2019
@codecov

codecovBot commented Aug 1, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5955 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #5955 +/- ##
=========================================
Coverage 46.79% 46.79% - Complexity 25648 25663 +15 
=========================================
Files 2456 2456 Lines 267613 267613 Branches 30563 30558 -5 =========================================
Hits 125241 125241 Misses 133113 133113 Partials 9259 9259

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0880b1a...227eaaf. Read the comment docs.

@igorbernstein2
igorbernstein2 merged commit 1aeaf18 into googleapis:masterAug 1, 2019
@igorbernstein2
igorbernstein2 deleted the no-default-its branch August 1, 2019 17:37
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
googleapis#5955)
This was a bug introduced in googleapis#5756. When integration profiles were introduced,
I forgot to remove the test from the default execution of the failsafe plugin.
The new way integration tests are run are via profile activation: by default
the bigtable-emulator-it profile is active.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtableIssues related to the Bigtable API.cla: yesThis human has signed the Contributor License Agreement.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@igorbernstein2@kolea2@googlebot