chore: add tests for complex resource IDs #3190
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3190 +/- ##
=========================================
Coverage 87.12% 87.12%
Complexity 6077 6077
=========================================
Files 494 494
Lines 24052 24052
Branches 2613 2613
=========================================
Hits 20956 20956
Misses 2234 2234
Partials 862 862
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm but the reformatting of the test data library.proto
makes it really hard to see what actually changed in the PR. WDYT about move the formatting changes to another PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Reverted proto formatting changes. |
It sounds like 2.7 is no longer supported, and it's currently breaking [tests](https://app.circleci.com/pipelines/github/googleapis/gapic-generator/255/workflows/b2063a3b-2ad2-4641-8952-0bc16673a242/jobs/47732/steps) for [this unrelated PR](#3190).
* chore: remove 2.7 from python showcase tests It sounds like 2.7 is no longer supported, and it's currently breaking [tests](https://app.circleci.com/pipelines/github/googleapis/gapic-generator/255/workflows/b2063a3b-2ad2-4641-8952-0bc16673a242/jobs/47732/steps) for [this unrelated PR](#3190). * fix: remove python 2.7 from .circleci/config.yml as well
Update tests and baseline protos to support non-slash resource name separators. That is, "-", "~", "_", ".". Tests will pass once this PR is submitted and a new release is cut.
Relevant issue here.