From e862fe7f02143deef97a4df832a91103f87d1b63 Mon Sep 17 00:00:00 2001
From: Kihwal Lee <kihwal@apache.org>
Date: Fri, 26 Feb 2016 10:23:45 -0600
Subject: [PATCH] HDFS-9855. Modify TestAuditLoggerWithCommands to workaround
 the absence of HDFS-8332. Contributed by Kuhu Shukla.

(cherry picked from commit 03afcdd39d61ccebd4e7b7903115213cee1bf92d)
---
 hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt         |  3 +++
 .../namenode/TestAuditLoggerWithCommands.java       | 13 +++++++------
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
index 88e3db201a3..0bccefb8e6a 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -1810,6 +1810,9 @@ Release 2.8.0 - UNRELEASED
     HDFS-9549. TestCacheDirectives#testExceedsCapacity is flaky (Xiao Chen via
     cmccabe)
 
+    HDFS-9855. Modify TestAuditLoggerWithCommands to workaround the absence
+    of HDFS-8332 (Kuhu Shukla via kihwal)
+
 Release 2.7.3 - UNRELEASED
 
   INCOMPATIBLE CHANGES
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestAuditLoggerWithCommands.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestAuditLoggerWithCommands.java
index 8b06b0bccd6..1449690fe48 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestAuditLoggerWithCommands.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestAuditLoggerWithCommands.java
@@ -121,7 +121,6 @@ public class TestAuditLoggerWithCommands {
   public void testSetQuota() throws Exception {
     Path path = new Path("/testdir/testdir1");
     fs.mkdirs(path);
-    fileSys = DFSTestUtil.getFileSystemAs(user1, conf);
     try {
       ((DistributedFileSystem)fileSys).setQuota(path, 10l, 10l);
       fail("The operation should have failed with AccessControlException");
@@ -129,15 +128,17 @@ public class TestAuditLoggerWithCommands {
     }
     String acePattern =
         ".*allowed=false.*ugi=theDoctor.*cmd=setQuota.*";
-    int length = verifyAuditLogs(acePattern);
-    fileSys.close();
+    verifyAuditLogs(acePattern);
+    String ioePattern =
+        ".*allowed=true.*ugi=" + fs.getFileStatus(path).getOwner().toString() +
+            ".*cmd=delete.*";
+    fs.delete(path, true);
     try {
-      ((DistributedFileSystem)fileSys).setQuota(path, 10l, 10l);
+      ((DistributedFileSystem)fs).setQuota(path, 10l, 10l);
       fail("The operation should have failed with IOException");
     } catch (IOException ace) {
     }
-    assertTrue("Unexpected log from getContentSummary",
-        length == auditlog.getOutput().split("\n").length);
+    verifyAuditLogs(ioePattern);
   }
 
   @Test