Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-13240. TestAclCommands.testSetfaclValidations fail. Contributed by John Zhuge. 

(cherry picked from commit 43cf6b101d)
(cherry picked from commit 382bcf21e7)
This commit is contained in:
Chris Nauroth 2016-07-21 14:12:31 -07:00
parent 9442e1d48e
commit af3c860bb1
2 changed files with 24 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java
View File

@ -213,6 +213,10 @@ class AclCommands extends FsCommand {
"Missing either <acl_spec> or <path>");
}
aclEntries = AclEntry.parseAclSpec(args.removeFirst(), !cf.getOpt("x"));
if (aclEntries.isEmpty()) {
throw new HadoopIllegalArgumentException(
"Missing <acl_spec> entry");
}
}
if (args.isEmpty()) {

26
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/shell/TestAclCommands.java
View File

@ -43,15 +43,22 @@ import org.apache.hadoop.ipc.RpcNoSuchMethodException;
import org.apache.hadoop.util.Progressable;
import org.apache.hadoop.util.ToolRunner;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
public class TestAclCommands {
@Rule
public TemporaryFolder testFolder = new TemporaryFolder();
private String path;
private Configuration conf = null;
@Before
public void setup() throws IOException {
conf = new Configuration();
path = testFolder.newFile("file").getPath();
}
@Test
@ -59,32 +66,31 @@ public class TestAclCommands {
assertFalse("getfacl should fail without path",
0 == runCommand(new String[] {"-getfacl"}));
assertFalse("getfacl should fail with extra argument",
0 == runCommand(new String[] { "-getfacl", "/test", "extraArg" }));
0 == runCommand(new String[] {"-getfacl", path, "extraArg"}));
}
@Test
public void testSetfaclValidations() throws Exception {
assertFalse("setfacl should fail without options",
0 == runCommand(new String[] { "-setfacl", "/" }));
0 == runCommand(new String[] {"-setfacl", path}));
assertFalse("setfacl should fail without options -b, -k, -m, -x or --set",
0 == runCommand(new String[] { "-setfacl", "-R", "/" }));
0 == runCommand(new String[] {"-setfacl", "-R", path}));
assertFalse("setfacl should fail without path",
0 == runCommand(new String[] {"-setfacl"}));
assertFalse("setfacl should fail without aclSpec",
0 == runCommand(new String[] { "-setfacl", "-m", "/path" }));
0 == runCommand(new String[] {"-setfacl", "-m", path}));
assertFalse("setfacl should fail with conflicting options",
0 == runCommand(new String[] { "-setfacl", "-m", "/path" }));
0 == runCommand(new String[] {"-setfacl", "-m", path}));
assertFalse("setfacl should fail with extra arguments",
0 == runCommand(new String[] { "-setfacl", "/path", "extra" }));
0 == runCommand(new String[] {"-setfacl", path, "extra"}));
assertFalse("setfacl should fail with extra arguments",
0 == runCommand(new String[] {"-setfacl", "--set",
"default:user::rwx", "/path", "extra" }));
"default:user::rwx", path, "extra"}));
assertFalse("setfacl should fail with permissions for -x",
0 == runCommand(new String[] {"-setfacl", "-x", "user:user1:rwx",
"/path" }));
path}));
assertFalse("setfacl should fail ACL spec missing",
0 == runCommand(new String[] { "-setfacl", "-m",
"", "/path" }));
0 == runCommand(new String[] {"-setfacl", "-m", "", path}));
}
@Test