Extend test coverage

Author: timokoessler

library/agent/hooks/instrumentation/codeTransformation.test.ts

@@ -449,7 +449,7 @@ t.test("modify rest parameter args", async (t) => {
         {
           nodeType: "MethodDefinition",
           name: "testFunction",
-          identifier: "testmodule.test.js.testFunction.MethodDefinition.v1.0.0",
+          identifier: "testpkg.test.js.testFunction.MethodDefinition.v1.0.0",
           inspectArgs: false,
           modifyArgs: true,
           modifyReturnValue: false,

library/agent/hooks/instrumentation/loadHook.test.ts

@@ -5,6 +5,7 @@ import { Hooks } from "../Hooks";
 import * as mod from "node:module";
 import { registerNodeHooks } from ".";
 import { Agent } from "../../Agent";
+import { onModuleLoad } from "./loadHook";
 
 t.test(
   "it throws an error if Node.js version is not supported",
@@ -190,3 +191,10 @@ t.test(
     t.equal(typeof fastify, "function");
   }
 );
+
+t.test("call with invalid args", async (t) => {
+  // @ts-expect-error Invalid args
+  t.same(onModuleLoad("test", undefined, "test"), "test");
+  // @ts-expect-error Invalid args
+  t.same(onModuleLoad("test", undefined, undefined), undefined);
+});

library/agent/hooks/instrumentation/processGetBuiltin.test.ts

@@ -17,6 +17,9 @@ t.test(
   async (t) => {
     createTestAgent();
 
+    // Before patched
+    t.same(getBuiltinModuleWithoutPatching("http"), require("http"));
+
     patchProcessGetBuiltinModule();
 
     const hooks = new Hooks();
@@ -53,5 +56,7 @@ t.test(
       // @ts-expect-error Ignore original types
       getBuiltinModuleWithoutPatching(undefined);
     });
+
+    t.same(getBuiltinModuleWithoutPatching("sqlite"), require("node:sqlite"));
   }
 );