From 9eb8682ec085c1836f7ab42ba86ad1483f983bbc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=87=91=E6=88=9F?= <jinji.lf@alibaba-inc.com>
Date: Mon, 15 Feb 2021 14:51:28 +0800
Subject: [PATCH] warn for non-static inner mock class

---
 .../agent/transformer/TestableClassTransformer.java      | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/testable-agent/src/main/java/com/alibaba/testable/agent/transformer/TestableClassTransformer.java b/testable-agent/src/main/java/com/alibaba/testable/agent/transformer/TestableClassTransformer.java
index fbc4dcf..6050910 100644
--- a/testable-agent/src/main/java/com/alibaba/testable/agent/transformer/TestableClassTransformer.java
+++ b/testable-agent/src/main/java/com/alibaba/testable/agent/transformer/TestableClassTransformer.java
@@ -270,9 +270,12 @@ public class TestableClassTransformer implements ClassFileTransformer {
         }
         // look for Mock inner class
         for (InnerClassNode ic : cn.innerClasses) {
-            if ((ic.access & ACC_PUBLIC) != 0 && (ic.access & ACC_STATIC) != 0 &&
-                ic.name.equals(getInnerMockClassName(className))) {
-                return ic.name;
+            if ((ic.access & ACC_PUBLIC) != 0 && ic.name.equals(getInnerMockClassName(className))) {
+                if ((ic.access & ACC_STATIC) != 0) {
+                    return ic.name;
+                } else {
+                    LogUtil.warn(String.format("Mock class in \"%s\" is not static", className));
+                }
             }
         }
         return null;