diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java index b8cc387..95cc301 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java @@ -5,7 +5,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.bootstrap.context.InternalInspectitContext; import rocks.inspectit.gepard.bootstrap.instrumentation.IMethodHook; diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java index 7075a30..f4c4377 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java @@ -8,8 +8,8 @@ import net.bytebuddy.matcher.ElementMatcher; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration; import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration; import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.resolver.ClassHookConfigurationResolver; import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.resolver.MethodHookConfigurationResolver; import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration; diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java index 4ba106c..94a4e1d 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java @@ -10,9 +10,9 @@ import java.util.concurrent.atomic.AtomicInteger; import java.util.stream.Collectors; import net.bytebuddy.description.method.MethodDescription; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration; import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.agent.instrumentation.hook.util.MethodHookFactory; /** Stores the method hook configurations of all instrumented classes. */ diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfiguration.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfiguration.java similarity index 90% rename from inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfiguration.java rename to inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfiguration.java index 2752016..872b390 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfiguration.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfiguration.java @@ -1,5 +1,5 @@ /* (C) 2024 */ -package rocks.inspectit.gepard.agent.instrumentation.hook.configuration; +package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model; import java.util.HashMap; import java.util.Map; @@ -9,12 +9,11 @@ import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration; /** - * Stores the hook configuration of all methods for a specific class. Currently, there is no complex - * hook configuration, thus we only use booleans. + * Stores the hook configuration of all methods for a specific class, which is used to update hooks. */ public class ClassHookConfiguration { - /** Set of methods and their hook configuration. Currently, just true. */ + /** Set of methods and their hook configuration. */ private final Map hookConfigurations; private final MethodHookConfigurationResolver hookResolver; diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfiguration.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfiguration.java similarity index 87% rename from inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfiguration.java rename to inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfiguration.java index 5a6417e..d12024f 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfiguration.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfiguration.java @@ -1,12 +1,12 @@ /* (C) 2024 */ -package rocks.inspectit.gepard.agent.instrumentation.hook.configuration; +package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model; import java.util.Objects; import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration; /** - * Configuration for one specific method hook. Currently just for tracing. Later we will add - * entry-/exit-actions and metrics. + * Configuration for one specific method hook, which is used to update hooks. Currently just for + * tracing. Later we will add entry-/exit-actions and metrics. */ public class MethodHookConfiguration { diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java index 8ceec49..a8381ec 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java @@ -5,8 +5,8 @@ import net.bytebuddy.description.method.MethodDescription; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration; /** diff --git a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java index 05d7d0e..ad492d5 100644 --- a/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java +++ b/inspectit-gepard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java @@ -3,7 +3,7 @@ import rocks.inspectit.gepard.agent.instrumentation.hook.MethodHook; import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; /** Creates method hook objects */ public class MethodHookFactory { diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java index 10a34de..10c2519 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java @@ -15,9 +15,9 @@ import org.junit.jupiter.api.extension.ExtendWith; import org.mockito.Mock; import org.mockito.junit.jupiter.MockitoExtension; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration; import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration; @ExtendWith(MockitoExtension.class) diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java index 82249d1..5438c18 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java @@ -13,7 +13,7 @@ import org.mockito.junit.jupiter.MockitoExtension; import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction; import rocks.inspectit.gepard.agent.instrumentation.hook.action.exception.CouldNotCloseSpanScopeException; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.bootstrap.context.InternalInspectitContext; @ExtendWith(MockitoExtension.class) diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfigurationTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfigurationTest.java similarity index 99% rename from inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfigurationTest.java rename to inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfigurationTest.java index 0ccfb65..b95c990 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/ClassHookConfigurationTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/ClassHookConfigurationTest.java @@ -1,5 +1,5 @@ /* (C) 2024 */ -package rocks.inspectit.gepard.agent.instrumentation.hook.configuration; +package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model; import static org.junit.jupiter.api.Assertions.*; import static org.mockito.Mockito.when; diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfigurationTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfigurationTest.java similarity index 98% rename from inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfigurationTest.java rename to inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfigurationTest.java index 2ef5f81..00ed051 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/MethodHookConfigurationTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/model/MethodHookConfigurationTest.java @@ -1,5 +1,5 @@ /* (C) 2024 */ -package rocks.inspectit.gepard.agent.instrumentation.hook.configuration; +package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotEquals; diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java index 90c8e51..58d674f 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java @@ -13,7 +13,7 @@ import org.junit.jupiter.api.extension.ExtendWith; import org.mockito.Mock; import org.mockito.junit.jupiter.MockitoExtension; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration; import rocks.inspectit.gepard.agent.internal.instrumentation.model.rules.InstrumentationRule; import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration; diff --git a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java index cc9a65b..251fd11 100644 --- a/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java +++ b/inspectit-gepard-agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java @@ -9,7 +9,7 @@ import org.mockito.Mock; import org.mockito.junit.jupiter.MockitoExtension; import rocks.inspectit.gepard.agent.instrumentation.hook.MethodHook; -import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration; +import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration; import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration; @ExtendWith(MockitoExtension.class)