Skip to content

Commit af584b9

Browse files
committed
protosanitizer: another test for oneof
During PR review there were concerns about support for oneof fields, with this example suggested as test case.
1 parent 43143fa commit af584b9

File tree

1 file changed

+25
-0
lines changed

1 file changed

+25
-0
lines changed

protosanitizer/protosanitizer_test.go

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,31 @@ func TestStripSecrets(t *testing.T) {
7070
{"hello world", `"hello world"`},
7171
{true, "true"},
7272
{false, "false"},
73+
// Test case from https://github.com/kubernetes-csi/csi-lib-utils/pull/1#pullrequestreview-180126394.
74+
{&csi.CreateVolumeRequest{
75+
Name: "test-volume",
76+
CapacityRange: &csi.CapacityRange{
77+
RequiredBytes: int64(1024),
78+
LimitBytes: int64(1024),
79+
},
80+
VolumeCapabilities: []*csi.VolumeCapability{
81+
&csi.VolumeCapability{
82+
AccessType: &csi.VolumeCapability_Mount{
83+
Mount: &csi.VolumeCapability_MountVolume{
84+
FsType: "ext4",
85+
MountFlags: []string{"flag1", "flag2", "flag3"},
86+
},
87+
},
88+
AccessMode: &csi.VolumeCapability_AccessMode{
89+
Mode: csi.VolumeCapability_AccessMode_MULTI_NODE_MULTI_WRITER,
90+
},
91+
},
92+
},
93+
Secrets: map[string]string{"secret1": "secret1", "secret2": "secret2"},
94+
Parameters: map[string]string{"param1": "param1", "param2": "param2"},
95+
VolumeContentSource: &csi.VolumeContentSource{},
96+
AccessibilityRequirements: &csi.TopologyRequirement{},
97+
}, `{"accessibility_requirements":{},"capacity_range":{"limit_bytes":1024,"required_bytes":1024},"name":"test-volume","parameters":{"param1":"param1","param2":"param2"},"secrets":"***stripped***","volume_capabilities":[{"AccessType":{"Mount":{"fs_type":"ext4","mount_flags":["flag1","flag2","flag3"]}},"access_mode":{"mode":5}}],"volume_content_source":{"Type":null}}`},
7398
{createVolume, `{"accessibility_requirements":{"requisite":[{"segments":{"foo":"bar","x":"y"}},{"segments":{"a":"b"}}]},"capacity_range":{"required_bytes":1024},"name":"foo","secrets":"***stripped***","volume_capabilities":[{"AccessType":{"Mount":{"fs_type":"ext4"}}}]}`},
7499

75100
// There is currently no test case that can verify

0 commit comments

Comments
 (0)