This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new 5f75b36ad36 [regression](framework) add config caseNamePrefix (#32266)
5f75b36ad36 is described below
commit 5f75b36ad36c0e91f48a9757cf9e6d79d635c8a8
Author: Dongyang Li <[email protected]>
AuthorDate: Thu Mar 14 21:48:28 2024 +0800
[regression](framework) add config caseNamePrefix (#32266)
---
.../main/groovy/org/apache/doris/regression/Config.groovy | 12 +++++++++++-
.../groovy/org/apache/doris/regression/ConfigOptions.groovy | 9 +++++++++
.../groovy/org/apache/doris/regression/RegressionTest.groovy | 4 ++++
.../org/apache/doris/regression/util/TeamcityUtils.groovy | 11 +++++++----
4 files changed, 31 insertions(+), 5 deletions(-)
diff --git
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy
index e240264060f..b1c9f5822ec 100644
---
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy
+++
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy
@@ -98,15 +98,17 @@ class Config {
public Integer actionParallel
public Integer times
public boolean withOutLoadData
+ public String caseNamePrefix
Config() {}
- Config(String defaultDb, String jdbcUrl, String jdbcUser, String
jdbcPassword,
+ Config(String caseNamePrefix, String defaultDb, String jdbcUrl, String
jdbcUser, String jdbcPassword,
String feSourceThriftAddress, String feTargetThriftAddress, String
feSyncerUser, String feSyncerPassword,
String syncerPassword, String feHttpAddress, String feHttpUser,
String feHttpPassword, String metaServiceHttpAddress,
String suitePath, String dataPath, String realDataPath, String
cacheDataPath, Boolean enableCacheData,
String testGroups, String excludeGroups, String testSuites, String
excludeSuites,
String testDirectories, String excludeDirectories, String
pluginPath, String sslCertificatePath) {
+ this.caseNamePrefix = caseNamePrefix
this.defaultDb = defaultDb
this.jdbcUrl = jdbcUrl
this.jdbcUser = jdbcUser
@@ -264,11 +266,13 @@ class Config {
config.randomOrder = cmd.hasOption(randomOrderOpt)
config.stopWhenFail = cmd.hasOption(stopWhenFailOpt)
config.withOutLoadData = cmd.hasOption(withOutLoadDataOpt)
+ config.caseNamePrefix = cmd.getOptionValue(caseNamePrefixOpt,
config.caseNamePrefix)
config.dryRun = cmd.hasOption(dryRunOpt)
log.info("randomOrder is ${config.randomOrder}".toString())
log.info("stopWhenFail is ${config.stopWhenFail}".toString())
log.info("withOutLoadData is ${config.withOutLoadData}".toString())
+ log.info("caseNamePrefix is ${config.caseNamePrefix}".toString())
log.info("dryRun is ${config.dryRun}".toString())
Properties props = cmd.getOptionProperties("conf")
@@ -282,6 +286,7 @@ class Config {
static Config fromConfigObject(ConfigObject obj) {
def config = new Config(
+ configToString(obj.caseNamePrefix),
configToString(obj.defaultDb),
configToString(obj.jdbcUrl),
configToString(obj.jdbcUser),
@@ -329,6 +334,11 @@ class Config {
}
static void fillDefaultConfig(Config config) {
+ if (config.caseNamePrefix == null) {
+ config.caseNamePrefix = ""
+ log.info("set caseNamePrefix to '' because not
specify.".toString())
+ }
+
if (config.defaultDb == null) {
config.defaultDb = "regression_test"
log.info("Set defaultDb to '${config.defaultDb}' because not
specify.".toString())
diff --git
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/ConfigOptions.groovy
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/ConfigOptions.groovy
index 36648dbe61d..0f5b63de5bf 100644
---
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/ConfigOptions.groovy
+++
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/ConfigOptions.groovy
@@ -65,6 +65,7 @@ class ConfigOptions {
static Option stopWhenFailOpt
static Option timesOpt
static Option withOutLoadDataOpt
+ static Option caseNamePrefixOpt
static Option dryRunOpt
static CommandLine initCommands(String[] args) {
@@ -376,6 +377,13 @@ class ConfigOptions {
.longOpt("withOutLoadData")
.desc("do not run load.groovy to reload data to Doris.")
.build()
+ caseNamePrefixOpt = Option.builder("cnp")
+ .required(false)
+ .hasArg(true)
+ .type(String.class)
+ .longOpt("caseNamePrefix")
+ .desc("add prefix to each case name")
+ .build()
dryRunOpt = Option.builder("dryRun")
.required(false)
.hasArg(false)
@@ -418,6 +426,7 @@ class ConfigOptions {
.addOption(stopWhenFailOpt)
.addOption(timesOpt)
.addOption(withOutLoadDataOpt)
+ .addOption(caseNamePrefixOpt)
.addOption(dryRunOpt)
CommandLine cmd = new DefaultParser().parse(options, args, true)
diff --git
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/RegressionTest.groovy
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/RegressionTest.groovy
index 4f0515c382d..feafabaddb4 100644
---
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/RegressionTest.groovy
+++
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/RegressionTest.groovy
@@ -97,6 +97,10 @@ class RegressionTest {
TeamcityUtils.postfix = i.toString()
}
+ if (config.caseNamePrefix) {
+ TeamcityUtils.prefix = config.caseNamePrefix.toString()
+ }
+
Recorder recorder = runScripts(config)
success = (success && printResult(config, recorder))
diff --git
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/TeamcityUtils.groovy
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/TeamcityUtils.groovy
index 61e6ff2eed3..fae3cdad846 100644
---
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/TeamcityUtils.groovy
+++
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/TeamcityUtils.groovy
@@ -25,13 +25,16 @@ import org.apache.tools.ant.util.DateUtils
@CompileStatic
class TeamcityUtils {
static String postfix = ""
+ static String prefix = ""
static String getSuiteName(String name) {
- if (postfix == "") {
- return name
- } else {
- return name+"-"+postfix
+ if (prefix != "") {
+ name = prefix + "-" + name
}
+ if (postfix != "") {
+ name = name + "-" + postfix
+ }
+ return name
}
static String formatNow() {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
