Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

chore: update nock path in system tests#216

Merged
DominicKramer merged 4 commits intogoogleapis:masterfrom
DominicKramer:chore/update-nock-in-system-tests
Oct 22, 2018
Merged

chore: update nock path in system tests#216
DominicKramer merged 4 commits intogoogleapis:masterfrom
DominicKramer:chore/update-nock-in-system-tests

Conversation

@DominicKramer
Copy link
Copy Markdown
Contributor

@DominicKramer DominicKramer commented Sep 25, 2018

The updated path and the following environment variables

GCLOUD_TESTS_PROJECT_ID
GCLOUD_PROJECT
GCLOUD_TESTS_API_KEY
GCLOUD_TESTS_PROJECT_NUMBER
GCLOUD_TESTS_KEY
GOOGLE_APPLICATION_CREDENTIALS

are needed for the systems tests to pass.

@DominicKramer DominicKramer requested a review from a team September 25, 2018 20:42
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 25, 2018
@ghost ghost assigned DominicKramer Sep 25, 2018
Copy link
Copy Markdown
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Do you want to push this as a branch so that the system tests get exercised?

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 25, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #216   +/-   ##
=======================================
  Coverage   88.79%   88.79%           
=======================================
  Files          29       29           
  Lines        1276     1276           
  Branches      151      151           
=======================================
  Hits         1133     1133           
  Misses         87       87           
  Partials       56       56
Impacted Files Coverage Δ
test/unit/request-extractors/express.ts 100% <ø> (ø) ⬆️

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 981f5df...a364940. Read the comment docs.

@ghost ghost assigned jkwlui Sep 25, 2018
@jkwlui jkwlui force-pushed the chore/update-nock-in-system-tests branch from eaf6f23 to de58222 Compare September 25, 2018 21:34
@DominicKramer DominicKramer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 25, 2018
@jkwlui jkwlui added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 28, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 28, 2018
@ghost ghost assigned JustinBeckwith Oct 1, 2018
@JustinBeckwith JustinBeckwith added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 1, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 1, 2018
@JustinBeckwith
Copy link
Copy Markdown
Contributor

@kinwa91 this is still needing a little love

@DominicKramer DominicKramer force-pushed the chore/update-nock-in-system-tests branch from c8acddf to 62074e1 Compare October 3, 2018 17:18
@DominicKramer DominicKramer force-pushed the chore/update-nock-in-system-tests branch 2 times, most recently from eec4048 to a2de43e Compare October 22, 2018 17:58
DominicKramer and others added 4 commits October 22, 2018 13:05
commit 2679b06
Author: Jonathan Lui <[email protected]>
Date:   Tue Sep 25 14:21:51 2018 -0700

    use kokoros stored system key

commit 3ead9c1
Merge: b761863 4aba4cd
Author: Justin Beckwith <[email protected]>
Date:   Tue Sep 25 14:10:23 2018 -0700

    Merge branch 'master' into autosynth

commit b761863
Author: Jonathan Lui <[email protected]>
Date:   Tue Sep 25 14:06:11 2018 -0700

    prevent credential from leaking

commit 29e034c
Author: Jonathan Lui <[email protected]>
Date:   Tue Sep 25 13:56:53 2018 -0700

    add GCLOUD_TESTS_API_KEY before system test

commit d5efc07
Author: Jonathan Lui <[email protected]>
Date:   Tue Sep 25 13:54:39 2018 -0700

    add repo-specific system-test env vars to kokoro pre hook

commit 25e3347
Author: dpebot <[email protected]>
Date:   Fri Sep 21 14:34:37 2018 -0700

    Re-generate library using /synth.py
The package appears to be broken and is not needed anyway.
@DominicKramer DominicKramer force-pushed the chore/update-nock-in-system-tests branch from acac9fb to a364940 Compare October 22, 2018 20:06
@DominicKramer DominicKramer merged commit d497a14 into googleapis:master Oct 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants