diff options
author | Nguyen Tuan Khang Phan <phan.khang5@gmail.com> | 2021-03-15 15:38:03 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2021-03-15 15:38:03 +0000 |
commit | e2d71aee1efa53b524f2bfb65f6965c0a3355655 (patch) | |
tree | 907671e99ca10ccce4c77cbb43af3ef62a3857bc | |
parent | dba338517e59b329faf7fbe3a1687b2251f3998a (diff) | |
parent | 868a034f1780578c6c83344ff8007f9d57e5f09d (diff) |
Merge "e2e-tests: Refactor duplicated "number" key name" into stable-3.1
8 files changed, 18 insertions, 17 deletions
diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/AbandonChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/AbandonChange.scala index 51bcb9e76b..d387a3e3b9 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/AbandonChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/AbandonChange.scala @@ -43,9 +43,9 @@ class AbandonChange extends GerritSimulation { numbersCopy = createChange.get.numbers.clone() } } - session.set("number", numbersCopy.dequeue()) + session.set(numberKey, numbersCopy.dequeue()) }) - .exec(http(uniqueName).post("${url}${number}/abandon")) + .exec(http(uniqueName).post("${url}${" + numberKey + "}/abandon")) private val createProject = new CreateProject(projectName) private val deleteProject = new DeleteProject(projectName) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ApproveChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ApproveChange.scala index 5e4f6718eb..9a91153c2d 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ApproveChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ApproveChange.scala @@ -40,13 +40,13 @@ class ApproveChange extends GerritSimulation { if (numbersCopy.isEmpty) { numbersCopy = createChange.get.numbers.clone() } - session.set("number", numbersCopy.dequeue()) + session.set(numberKey, numbersCopy.dequeue()) } else { session } }) .exec(http(uniqueName) - .post("${url}${number}/revisions/current/review") + .post("${url}${" + numberKey + "}/revisions/current/review") .body(ElFileBody(body)).asJson) setUp( diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateChange.scala index b28edb5fc5..fb410753fb 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateChange.scala @@ -24,7 +24,6 @@ import scala.concurrent.duration._ class CreateChange extends ProjectSimulation { private val data: FeederBuilder = jsonFile(resource).convert(keys).circular - private val numberKey = "_number" private val weightPerUser = 0.1 private var createBranch: Option[CreateBranch] = None private var branchesCopy: mutable.Queue[String] = mutable.Queue[String]() @@ -58,7 +57,7 @@ class CreateChange extends ProjectSimulation { }) .exec(httpRequest .body(ElFileBody(body)).asJson - .check(regex("\"" + numberKey + "\":(\\d+),").saveAs(numberKey))) + .check(regex("\"_" + numberKey + "\":(\\d+),").saveAs(numberKey))) .exec(session => { number = session(numberKey).as[Int] numbers += number diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteChange.scala index e47108f04d..743219f4e4 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteChange.scala @@ -34,12 +34,12 @@ class DeleteChange extends GerritSimulation { .feed(data) .exec(session => { if (createChange.nonEmpty) { - session.set("number", createChange.get.numbers.dequeue()) + session.set(numberKey, createChange.get.numbers.dequeue()) } else { session } }) - .exec(http(uniqueName).delete("${url}${number}")) + .exec(http(uniqueName).delete("${url}${" + numberKey + "}")) setUp( test.inject( diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GerritSimulation.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GerritSimulation.scala index b11c87ced0..f8e3faa00f 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GerritSimulation.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GerritSimulation.scala @@ -23,6 +23,8 @@ import io.gatling.http.request.builder.HttpRequestBuilder class GerritSimulation extends Simulation { implicit val conf: GatlingGitConfiguration = GatlingGitConfiguration() + protected val numberKey: String = "number" + private val packageName = getClass.getPackage.getName private val path = packageName.replaceAllLiterally(".", "/") @@ -64,9 +66,9 @@ class GerritSimulation extends Simulation { protected val keys: PartialFunction[(String, Any), Any] = { case ("entries", entries) => replaceProperty("projects_entries", "1", entries.toString) - case ("number", number) => - val precedes = replaceKeyWith("_number", 0, number.toString) - replaceProperty("number", 1, precedes) + case (`numberKey`, number) => + val precedes = replaceKeyWith("_" + numberKey, 0, number.toString) + replaceProperty(numberKey, 1, precedes) case ("parent", parent) => replaceProperty("parent", "All-Projects", parent.toString) case ("project", project) => diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/RestoreChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/RestoreChange.scala index b952670d10..81096b0c09 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/RestoreChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/RestoreChange.scala @@ -35,9 +35,9 @@ class RestoreChange extends GerritSimulation { if (numbersCopy.isEmpty) { numbersCopy = createChange.numbers.clone() } - session.set("number", numbersCopy.dequeue()) + session.set(numberKey, numbersCopy.dequeue()) } - ).exec(http(uniqueName).post("${url}${number}/restore")) + ).exec(http(uniqueName).post("${url}${" + numberKey + "}/restore")) private val createProject = new CreateProject(projectName) private val createChange = new CreateChange(projectName) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChange.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChange.scala index 067496ac5e..20be28aa82 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChange.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChange.scala @@ -36,9 +36,9 @@ class SubmitChange extends GerritSimulation { val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { - session.set("number", createChange.number) + session.set(numberKey, createChange.number) }) - .exec(http(uniqueName).post("${url}${number}/submit")) + .exec(http(uniqueName).post("${url}${" + numberKey + "}/submit")) private val createProject = new CreateProject(projectName) private val approveChange = new ApproveChange(createChange) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChangeInBranch.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChangeInBranch.scala index 1b88503fb4..9e1431b11d 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChangeInBranch.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/SubmitChangeInBranch.scala @@ -35,9 +35,9 @@ class SubmitChangeInBranch extends GerritSimulation { if (changesCopy.isEmpty) { changesCopy = createChange.numbers.clone() } - session.set("number", changesCopy.dequeue()) + session.set(numberKey, changesCopy.dequeue()) }) - .exec(http(uniqueName).post("${url}${number}/submit")) + .exec(http(uniqueName).post("${url}${" + numberKey + "}/submit")) private val createProject = new CreateProject(projectName) private val createBranch = new CreateBranch(projectName) |