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

Commit d5f88f6

Browse files
feat!: remove legacy resource name (#125)
* chore: Update gapic-generator version The updated version is required to migrate Python clients generation from artman to bazel. PiperOrigin-RevId: 307739946 Source-Author: Google APIs <noreply@google.com> Source-Date: Tue Apr 21 21:16:30 2020 -0700 Source-Repo: googleapis/googleapis Source-Sha: 862cd840dbb0101a0630913217bf8b6d4101c84c Source-Link: googleapis/googleapis@862cd84 * chore: allow breaking resource name changes * chore: remove legacy resource name types Co-authored-by: Jeff Ching <chingor@google.com>
1 parent 5d62223 commit d5f88f6

File tree

11 files changed

+140
-619
lines changed

11 files changed

+140
-619
lines changed

google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1/SecurityCenterClientTest.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -1285,7 +1285,8 @@ public void updateSourceExceptionTest() throws Exception {
12851285
@Test
12861286
@SuppressWarnings("all")
12871287
public void updateSecurityMarksTest() {
1288-
SecurityMarksName name = SecurityMarksName.ofOrganizationAssetName("[ORGANIZATION]", "[ASSET]");
1288+
SecurityMarksName name =
1289+
SecurityMarksName.ofOrganizationAssetSecurityMarksName("[ORGANIZATION]", "[ASSET]");
12891290
SecurityMarks expectedResponse = SecurityMarks.newBuilder().setName(name.toString()).build();
12901291
mockSecurityCenter.addResponse(expectedResponse);
12911292

google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1p1beta1/SecurityCenterClientTest.java

+4-2
Original file line numberDiff line numberDiff line change
@@ -1374,7 +1374,8 @@ public void updateSourceExceptionTest2() throws Exception {
13741374
@Test
13751375
@SuppressWarnings("all")
13761376
public void updateSecurityMarksTest() {
1377-
SecurityMarksName name = SecurityMarksName.ofOrganizationAssetName("[ORGANIZATION]", "[ASSET]");
1377+
SecurityMarksName name =
1378+
SecurityMarksName.ofOrganizationAssetSecurityMarksName("[ORGANIZATION]", "[ASSET]");
13781379
SecurityMarks expectedResponse = SecurityMarks.newBuilder().setName(name.toString()).build();
13791380
mockSecurityCenter.addResponse(expectedResponse);
13801381

@@ -1413,7 +1414,8 @@ public void updateSecurityMarksExceptionTest() throws Exception {
14131414
@Test
14141415
@SuppressWarnings("all")
14151416
public void updateSecurityMarksTest2() {
1416-
SecurityMarksName name = SecurityMarksName.ofOrganizationAssetName("[ORGANIZATION]", "[ASSET]");
1417+
SecurityMarksName name =
1418+
SecurityMarksName.ofOrganizationAssetSecurityMarksName("[ORGANIZATION]", "[ASSET]");
14171419
SecurityMarks expectedResponse = SecurityMarks.newBuilder().setName(name.toString()).build();
14181420
mockSecurityCenter.addResponse(expectedResponse);
14191421

proto-google-cloud-securitycenter-v1/clirr-ignored-differences.xml

+43-3
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,55 @@
1717
<method>boolean has*(*)</method>
1818
</difference>
1919

20-
<!-- TODO: remove after 0.121.0 -->
20+
<!-- TODO: remove after 0.122.0 -->
2121
<difference>
2222
<differenceType>7002</differenceType>
2323
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
24-
<method>java.util.List parse(java.util.List)</method>
24+
<method>java.lang.String formatOrganizationAssetName(java.lang.String, java.lang.String)</method>
2525
</difference>
2626
<difference>
2727
<differenceType>7002</differenceType>
2828
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
29-
<method>java.util.List toStrings(java.util.List)</method>
29+
<method>java.lang.String formatOrganizationSourceFindingName(java.lang.String, java.lang.String, java.lang.String)</method>
30+
</difference>
31+
<difference>
32+
<differenceType>7002</differenceType>
33+
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
34+
<method>com.google.cloud.securitycenter.v1.SecurityMarksName$Builder newOrganizationAssetBuilder()</method>
35+
</difference>
36+
<difference>
37+
<differenceType>7002</differenceType>
38+
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
39+
<method>com.google.cloud.securitycenter.v1.SecurityMarksName$OrganizationSourceFindingBuilder newOrganizationSourceFindingBuilder()</method>
40+
</difference>
41+
<difference>
42+
<differenceType>7002</differenceType>
43+
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
44+
<method>com.google.cloud.securitycenter.v1.SecurityMarksName ofOrganizationAssetName(java.lang.String, java.lang.String)</method>
45+
</difference>
46+
<difference>
47+
<differenceType>7002</differenceType>
48+
<className>com/google/cloud/securitycenter/v1/SecurityMarksName</className>
49+
<method>com.google.cloud.securitycenter.v1.SecurityMarksName ofOrganizationSourceFindingName(java.lang.String, java.lang.String, java.lang.String)</method>
50+
</difference>
51+
<difference>
52+
<differenceType>8001</differenceType>
53+
<className>com/google/cloud/securitycenter/v1/SecurityMarksName$OrganizationSourceFindingBuilder</className>
54+
</difference>
55+
<difference>
56+
<differenceType>8001</differenceType>
57+
<className>com/google/cloud/securitycenter/v1/AssetSecurityMarksName*</className>
58+
</difference>
59+
<difference>
60+
<differenceType>8001</differenceType>
61+
<className>com/google/cloud/securitycenter/v1/FindingSecurityMarksName*</className>
62+
</difference>
63+
<difference>
64+
<differenceType>8001</differenceType>
65+
<className>com/google/cloud/securitycenter/v1/SecurityMarksNames</className>
66+
</difference>
67+
<difference>
68+
<differenceType>8001</differenceType>
69+
<className>com/google/cloud/securitycenter/v1/UntypedSecurityMarksName</className>
3070
</difference>
3171
</differences>

proto-google-cloud-securitycenter-v1/src/main/java/com/google/cloud/securitycenter/v1/AssetSecurityMarksName.java

-187
This file was deleted.

0 commit comments

Comments
 (0)