fix: LogMode isEnabled behavior (#1038)

Author: lauzadis

.changes/01ea7858-c080-447f-9bd1-93bdec3f0161.json

@@ -0,0 +1,5 @@
+{
+    "id": "01ea7858-c080-447f-9bd1-93bdec3f0161",
+    "type": "bugfix",
+    "description": "Fix LogMode unintentionally enabling *WithBody modes"
+}

runtime/smithy-client/common/src/aws/smithy/kotlin/runtime/client/LogMode.kt

@@ -62,7 +62,7 @@ public sealed class LogMode(private val mask: Int) {
     /**
      * Test if a particular [LogMode] is enabled
      */
-    public fun isEnabled(mode: LogMode): Boolean = mask and mode.mask != 0
+    public fun isEnabled(mode: LogMode): Boolean = mask and mode.mask == mode.mask
 
     public companion object {
         /**

runtime/smithy-client/common/test/aws/smithy/kotlin/runtime/client/LogModeTest.kt

@@ -66,4 +66,13 @@ class LogModeTest {
         assertTrue(LogMode.LogResponseWithBody.isEnabled(LogMode.LogResponse))
         assertFalse(LogMode.LogResponseWithBody.isEnabled(LogMode.LogRequest))
     }
+
+    @Test
+    fun testWithoutBodyDoesNotImplyWith() {
+        // LogRequest does NOT imply LogRequestWithBody
+        assertFalse(LogMode.LogRequest.isEnabled(LogMode.LogRequestWithBody))
+
+        // LogResponse does NOT imply LogResponseWithBody
+        assertFalse(LogMode.LogResponse.isEnabled(LogMode.LogResponseWithBody))
+    }
 }