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

chore: resolve TODO in test#259

Merged
ofrobots merged 2 commits intomasterfrom
fix-fixme
Feb 12, 2019
Merged

chore: resolve TODO in test#259
ofrobots merged 2 commits intomasterfrom
fix-fixme

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 12, 2019
assert.strictEqual(passedEmitRequestLog, undefined);
});
});
[GCPEnv.APP_ENGINE, GCPEnv.CLOUD_FUNCTIONS].forEach(env => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So not directly related to this PR - but I typically prefer to just copy the code for tests like this. Debugging a failure is hard because the stack trace becomes a guess, and things like .only and .skip become harder.

@ofrobots ofrobots added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2019
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 12, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   91.78%   91.78%           
=======================================
  Files           2        2           
  Lines          73       73           
  Branches        6        6           
=======================================
  Hits           67       67           
  Misses          6        6

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 68d14b9...5ac57eb. Read the comment docs.

@ofrobots ofrobots merged commit 6665195 into master Feb 12, 2019
@ofrobots ofrobots deleted the fix-fixme branch February 12, 2019 22:00
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.

5 participants