move test suite constant definitions into ModuleDefinition
... so that users of the constants can reference ModuleDefinition
instead, avoiding dependency on compatibility test harness
Bug: 36140955
Bug: 35360169
Change-Id: I60763d102b51a30d512ceff604a3cd49c8954e42
Test: new and existing unit test
diff --git a/common/host-side/tradefed/src/com/android/compatibility/common/tradefed/testtype/IModuleDef.java b/common/host-side/tradefed/src/com/android/compatibility/common/tradefed/testtype/IModuleDef.java
index 69e8393..7a2c44d 100644
--- a/common/host-side/tradefed/src/com/android/compatibility/common/tradefed/testtype/IModuleDef.java
+++ b/common/host-side/tradefed/src/com/android/compatibility/common/tradefed/testtype/IModuleDef.java
@@ -24,6 +24,7 @@
import com.android.tradefed.testtype.IRemoteTest;
import com.android.tradefed.testtype.IRuntimeHintProvider;
import com.android.tradefed.testtype.ITestCollector;
+import com.android.tradefed.testtype.suite.ModuleDefinition;
import java.util.List;
import java.util.Set;
@@ -35,8 +36,10 @@
IRemoteTest, IRuntimeHintProvider, ITestCollector {
/** key names used for saving module info into {@link IInvocationContext} */
- public static String MODULE_NAME = "module-name";
- public static String MODULE_ABI = "module-abi";
+ // This currently references ModuleDefinition so that there's only once source for String
+ // literals and making it easier to converge IModuleDef and ModuleDefinition later
+ public static String MODULE_NAME = ModuleDefinition.MODULE_NAME;
+ public static String MODULE_ABI = ModuleDefinition.MODULE_ABI;
/**
* @return The name of this module.