Skip to content
This repository has been archived by the owner on Dec 23, 2023. It is now read-only.

Disable integration tests in the Agent. Always fail. #556

Merged
merged 1 commit into from Aug 25, 2017

Conversation

bogdandrutu
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Aug 25, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##             master   #556   +/-   ##
=======================================
  Coverage        90%    90%           
  Complexity      566    566           
=======================================
  Files           110    110           
  Lines          2361   2361           
  Branches        211    211           
=======================================
  Hits           2125   2125           
  Misses          180    180           
  Partials         56     56

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 efb18be...a24c066. Read the comment docs.

@sebright
Copy link
Contributor

Is this change related to #540? I think it would be better to just add an Ignore annotation to the failing test.

@sebright sebright assigned bogdandrutu and unassigned sebright Aug 25, 2017
@bogdandrutu
Copy link
Contributor Author

That would be an option if the test would finish and not break everything. The test pauses for a long time (more than a minute) and I don't want to wait that much every time when I build locally.

@bogdandrutu
Copy link
Contributor Author

This is an workaround for #557 until a fix lands.

@bogdandrutu bogdandrutu merged commit ee067d7 into census-instrumentation:master Aug 25, 2017
@bogdandrutu bogdandrutu deleted the disabletests branch August 25, 2017 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants