github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #91741 from oomichi/nit-ExpectError

Browse Source 
Replace framework.Failf with ExpectNoError
This commit is contained in:
Kubernetes Prow Robot
2020-06-04 13:53:05 -07:00
committed by GitHub
parent 4dfa0912a0 0ebaae88b1
commit 64bba294ae
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
test/e2e/storage/volume_provisioning.go
View File

@@ -673,10 +673,10 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() {
// Modifying the default storage class can be disruptive to other tests that depend on it
ginkgo.It("should be disabled by changing the default annotation [Serial] [Disruptive]", func() {
e2eskipper.SkipUnlessProviderIs("openstack", "gce", "aws", "gke", "vsphere", "azure")
scName, scErr := e2epv.GetDefaultStorageClassName(c)
if scErr != nil {
framework.Failf(scErr.Error())
}
framework.ExpectNoError(scErr)
test := testsuites.StorageClassTest{
Name: "default",
ClaimSize: "2Gi",
@@ -710,10 +710,10 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() {
// Modifying the default storage class can be disruptive to other tests that depend on it
ginkgo.It("should be disabled by removing the default annotation [Serial] [Disruptive]", func() {
e2eskipper.SkipUnlessProviderIs("openstack", "gce", "aws", "gke", "vsphere", "azure")
scName, scErr := e2epv.GetDefaultStorageClassName(c)
if scErr != nil {
framework.Failf(scErr.Error())
}
framework.ExpectNoError(scErr)
test := testsuites.StorageClassTest{
Name: "default",
ClaimSize: "2Gi",