AG-23697 'log-addEventListener' prints a lot of errors to console. #335

Squashed commit of the following:

commit a8d88dd
Author: Adam Wróblewski <adam@adguard.com>
Date:   Thu Jun 29 15:04:02 2023 +0200

    Fix log-addEventListener
    do not print unnecessary logs to the console

CHANGELOG.md

@@ -11,6 +11,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- issue with printing unnecessary logs to the console in `log-addEventListener` scriptlet
+  [#335](https://github.com/AdguardTeam/Scriptlets/issues/335)
 - error throwing in `prevent-fetch` and `prevent-xhr` scriptlets when a request is blocked
   [#334](https://github.com/AdguardTeam/Scriptlets/issues/334)
 

src/scriptlets/log-addEventListener.js

@@ -35,14 +35,14 @@ export function logAddEventListener(source) {
             const message = `addEventListener("${type}", ${listenerToString(listener)})`;
             logMessage(source, message, true);
             hit(source);
+        } else {
+            // logging while debugging
+            const message = `Invalid event type or listener passed to addEventListener:
+        type: ${convertTypeToString(type)}
+        listener: ${convertTypeToString(listener)}`;
+            logMessage(source, message, true);
         }
 
-        // logging while debugging
-        const message = `Invalid event type or listener passed to addEventListener:
-type: ${convertTypeToString(type)}
-listener: ${convertTypeToString(listener)}`;
-        logMessage(source, message, true);
-
         // Avoid illegal invocations due to lost context
         // https://github.com/AdguardTeam/Scriptlets/issues/271
         let context = this;

tests/scriptlets/log-addEventListener.test.js

@@ -60,6 +60,7 @@ test('logs events to console', (assert) => {
         }
         assert.ok(input.includes(eventName), 'console.hit input should be equal');
         assert.ok(input.includes(callback.toString()), 'console.hit input should be equal');
+        assert.notOk(input.includes(INVALID_MESSAGE_START), 'Invalid message should not be displayed');
     };
 
     runScriptlet(name);