Skip to content

Commit 79f9a8c

Browse files
committed
Only sanity-check when necessary
1 parent 4eddd42 commit 79f9a8c

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

e2etest/scenario_helpers.go

+9-7
Original file line numberDiff line numberDiff line change
@@ -686,16 +686,18 @@ func (scenarioHelper) generateAzureFilesFromList(c asserter, options *generateAz
686686
_, err := dir.SetProperties(ctx, ad.toHeaders(c, options.shareURL).SMBProperties)
687687
c.AssertNoErr(err)
688688

689-
prop, err := dir.GetProperties(ctx)
690-
c.AssertNoErr(err)
689+
if f.creationProperties.smbPermissionsSddl != nil {
690+
prop, err := dir.GetProperties(ctx)
691+
c.AssertNoErr(err)
691692

692-
perm, err := options.shareURL.GetPermission(ctx, prop.FilePermissionKey())
693-
c.AssertNoErr(err)
693+
perm, err := options.shareURL.GetPermission(ctx, prop.FilePermissionKey())
694+
c.AssertNoErr(err)
694695

695-
dest, _ := sddl.ParseSDDL(perm.Permission)
696-
source, _ := sddl.ParseSDDL(*f.creationProperties.smbPermissionsSddl)
696+
dest, _ := sddl.ParseSDDL(perm.Permission)
697+
source, _ := sddl.ParseSDDL(*f.creationProperties.smbPermissionsSddl)
697698

698-
c.Assert(dest.Compare(source), equals(), true)
699+
c.Assert(dest.Compare(source), equals(), true)
700+
}
699701
}
700702

701703
// set other properties

0 commit comments

Comments
 (0)