From 909048d87d168628ac1e19004b7d1f53863eef7b Mon Sep 17 00:00:00 2001 From: Mehakmeet Singh Date: Fri, 4 Mar 2022 00:14:28 +0530 Subject: [PATCH] HADOOP-18150. Fix ITestAuditManagerDisabled test in S3A. (#4044) Contributed by Mehakmeet Singh Change-Id: I25c10844e4ad64b1fd7af9a02018220a611c85e0 --- .../hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java index 033d793a6cb..68aaef51d60 100644 --- a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java +++ b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java @@ -28,9 +28,10 @@ import static org.apache.hadoop.fs.s3a.audit.AuditTestSupport.NOOP_SPAN; import static org.apache.hadoop.fs.s3a.audit.AuditTestSupport.resetAuditOptions; +import static org.apache.hadoop.fs.s3a.audit.S3AAuditConstants.AUDIT_ENABLED; /** - * Verify that by default audit managers are disabled. + * Verify that audit managers are disabled if set to false. */ public class ITestAuditManagerDisabled extends AbstractS3ACostTest { @@ -42,11 +43,12 @@ public ITestAuditManagerDisabled() { public Configuration createConfiguration() { Configuration conf = super.createConfiguration(); resetAuditOptions(conf); + conf.setBoolean(AUDIT_ENABLED, false); return conf; } /** - * The default auditor is the no-op auditor. + * Verify that the auditor is the no-op auditor if auditing is disabled. */ @Test public void testAuditorDisabled() {