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

Split off ServerStreamingCallableTests into own files #435

Merged

Conversation

igorbernstein2
Copy link
Contributor

Nonfunctional change in preparation for #433.

Simply moves server streaming tests into their own files to cut down on the noise in the next PR

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #435   +/-   ##
=========================================
  Coverage     70.53%   70.53%           
  Complexity      634      634           
=========================================
  Files           136      136           
  Lines          2912     2912           
  Branches        220      220           
=========================================
  Hits           2054     2054           
  Misses          776      776           
  Partials         82       82

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 9b86a50...d458192. Read the comment docs.

.build();
}

private static class AccumulatingStreamObserver implements ApiStreamObserver<Integer> {

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

Copy link
Member

@garrettjonesgoogle garrettjonesgoogle left a comment

Choose a reason for hiding this comment

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

LGTM

@garrettjonesgoogle garrettjonesgoogle merged commit 459a80a into googleapis:master Nov 30, 2017
@igorbernstein2 igorbernstein2 deleted the ssc-1-test-split branch January 10, 2018 05:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants