Skip to content

Add VPCSC tests for cloud asset client lib - #5695

Merged
chingor13 merged 1 commit into
googleapis:masterfrom
gaogaogiraffe:master
Jul 8, 2019
Merged

Add VPCSC tests for cloud asset client lib#5695
chingor13 merged 1 commit into
googleapis:masterfrom
gaogaogiraffe:master

Conversation

@gaogaogiraffe

Copy link
Copy Markdown
Contributor

The tests can be run inside or outside of VPC service perimeter. The input to the script should be the following environment variables.

PROJECT_ID: a project that is inside the VPC perimeter.
GOOGLE_CLOUD_TESTS_VPCSC_OUTSIDE_PERIMETER_PROJECT: a project that is outside the VPC perimeter.

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for context and/or discussion)

The tests can be run inside or outside of VPC service perimeter. The input to the script should be the following environment variables.
PROJECT_ID: a project that is inside the VPC perimeter.
GOOGLE_CLOUD_TESTS_VPCSC_OUTSIDE_PERIMETER_PROJECT: a project that is outside the VPC perimeter.
@gaogaogiraffe
gaogaogiraffe requested a review from a teamJuly 6, 2019 23:38
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 6, 2019
@codecov

codecovBot commented Jul 7, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5695 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #5695 +/- ##
============================================
- Coverage 46.71% 46.71% -0.01% + Complexity 24630 24629 -1 
============================================
Files 2351 2351 Lines 256127 256127 Branches 29325 29325 ============================================
- Hits 119650 119648 -2 - Misses 127558 127559 +1 - Partials 8919 8920 +1
Impacted FilesCoverage ΔComplexity Δ
...able/gaxx/reframing/ReframingResponseObserver.java88.99% <0%> (-1.84%)29% <0%> (-1%)

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 c5d709c...2484f02. Read the comment docs.

@chingor13
chingor13 merged commit 06e7b08 into googleapis:masterJul 8, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@gaogaogiraffe@chingor13@googlebot