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

Conversation

@wangbill-google
Copy link
Contributor

Upload system test case for text to speech API(both v1 and v1beta1).

Upload system test case for text to speech API v1
Upload system test case for text to speech API v1beta1
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 23, 2019
Copy link
Contributor

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

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

Can you move these tests into an it folder, change the package to include it and rename the files/classes to ITSystemTest?

We have common config across our projects that use this convention to limit which tests we run and when.

…cloud/texttospeech/v1/TextToSpeechSystemTest.java to google-cloud-texttospeech/src/test/java/com/google/cloud/texttospeech/v1/it/ITSystemTest.java

ITSystemTest
…cloud/texttospeech/v1beta1/TextToSpeechSystemTest.java to google-cloud-texttospeech/src/test/java/com/google/cloud/texttospeech/v1beta1/it/ITSystemTest.java

ITSystemTest
@codecov
Copy link

codecov bot commented Oct 23, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##             master      #5   +/-   ##
========================================
  Coverage      66.2%   66.2%           
  Complexity       72      72           
========================================
  Files            12      12           
  Lines           432     432           
  Branches          2       2           
========================================
  Hits            286     286           
  Misses          144     144           
  Partials          2       2

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 577c7dd...928f260. Read the comment docs.

@chingor13
Copy link
Contributor

Codecov can't possibly be correct about test coverage dropping - ignore that "failure".

Copy link
Contributor

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

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

Thank you!

@chingor13 chingor13 merged commit 81464e8 into googleapis:master Oct 23, 2019
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.

3 participants