diff options
author | Marco Miller <marco.miller@ericsson.com> | 2020-09-21 17:28:29 -0400 |
---|---|---|
committer | Marco Miller <marco.mmiller@gmail.com> | 2020-09-22 17:33:29 +0000 |
commit | b314e7b64ae519655a7b85beb334a459f4b8de4d (patch) | |
tree | 64c20466b92f13894b733a3a63e5637978ebd3c1 | |
parent | f9c1300bfeb1e8b74216c2d24b3cb0113f028bca (diff) |
e2e-tests: Rename 'unique' to uniqueName for explicitness
While still keeping a relatively short name for this highly reused
value. Preserve a fair enough balance between footprint and meaning.
Change-Id: I229976f502f4187938303053fd81d686f1730d3c
17 files changed, 26 insertions, 26 deletions
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 e808d0d673..8ae69d70d7 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 @@ -28,7 +28,7 @@ class ApproveChange extends GerritSimulation { this.createChange = Some(createChange) } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { if (createChange.nonEmpty) { @@ -37,7 +37,7 @@ class ApproveChange extends GerritSimulation { session } }) - .exec(http(unique) + .exec(http(uniqueName) .post("${url}${number}/revisions/current/review") .body(ElFileBody(body)).asJson) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckMasterBranchReplica1.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckMasterBranchReplica1.scala index 96269f28b8..81274de566 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckMasterBranchReplica1.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckMasterBranchReplica1.scala @@ -39,12 +39,12 @@ class CheckMasterBranchReplica1 extends ProjectSimulation { private val submitChange = new SubmitChange(createChange) private val getBranch = new GetMasterBranchRevision - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { session.set(getBranch.revisionKey, getBranch.revision.get) }) - .exec(http(unique).get("${url}") + .exec(http(uniqueName).get("${url}") .check(regex(getBranch.revisionPattern) .is(session => session(getBranch.revisionKey).as[String]))) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckNewProjectReplica1.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckNewProjectReplica1.scala index 13e88b7588..60b5a243da 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckNewProjectReplica1.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckNewProjectReplica1.scala @@ -34,7 +34,7 @@ class CheckNewProjectReplica1 extends GitSimulation { super.replaceOverride(next) } - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(gitRequest) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckProjectsCacheFlushEntries.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckProjectsCacheFlushEntries.scala index f1966b3b20..96943ce1b4 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckProjectsCacheFlushEntries.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CheckProjectsCacheFlushEntries.scala @@ -27,7 +27,7 @@ class CheckProjectsCacheFlushEntries extends CacheFlushSimulation { this.producer = Some(producer) } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { if (producer.nonEmpty) { @@ -36,7 +36,7 @@ class CheckProjectsCacheFlushEntries extends CacheFlushSimulation { session } }) - .exec(http(unique).get("${url}") + .exec(http(uniqueName).get("${url}") .check(regex("\"" + memKey + "\": (\\d+)") .is(session => session(entriesKey).as[String]))) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CloneUsingBothProtocols.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CloneUsingBothProtocols.scala index c252b0a584..e07a9e611e 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CloneUsingBothProtocols.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CloneUsingBothProtocols.scala @@ -29,7 +29,7 @@ class CloneUsingBothProtocols extends GitSimulation { replaceKeyWith("_project", default, in) } - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(gitRequest) 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 ce37777002..48bc04b790 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 @@ -33,7 +33,7 @@ class CreateChange extends ProjectSimulation { this.default = default } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(httpRequest .body(ElFileBody(body)).asJson diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateProject.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateProject.scala index f2b3d12f2a..515a39ff2b 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateProject.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/CreateProject.scala @@ -26,7 +26,7 @@ class CreateProject extends ProjectSimulation { this.default = default } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(httpRequest.body(ElFileBody(body)).asJson) 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 5a06ff7680..d832bde3d7 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 @@ -30,7 +30,7 @@ class DeleteChange extends GerritSimulation { this.createChange = Some(createChange) } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { if (createChange.nonEmpty) { @@ -39,7 +39,7 @@ class DeleteChange extends GerritSimulation { session } }) - .exec(http(unique).delete("${url}${number}")) + .exec(http(uniqueName).delete("${url}${number}")) setUp( test.inject( diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteProject.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteProject.scala index 2007eba902..39a74dd674 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteProject.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/DeleteProject.scala @@ -26,7 +26,7 @@ class DeleteProject extends ProjectSimulation { this.default = default } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(httpRequest) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCache.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCache.scala index f89200eb50..60145e54de 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCache.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCache.scala @@ -26,7 +26,7 @@ class FlushProjectsCache extends CacheFlushSimulation { override def relativeRuntimeWeight = 2 - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(httpRequest) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCacheThenRebuild.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCacheThenRebuild.scala index 7d732ea44b..07b0c0b86b 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCacheThenRebuild.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/FlushProjectsCacheThenRebuild.scala @@ -23,7 +23,7 @@ import scala.concurrent.duration._ class FlushProjectsCacheThenRebuild extends GerritSimulation { private val data: FeederBuilder = jsonFile(resource).convert(keys).queue - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(httpRequest) 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 38b2c4be6e..7b31b3d386 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 @@ -31,7 +31,7 @@ class GerritSimulation extends Simulation { protected val resource = s"$pathName.json" protected val body = s"$pathName-body.json" - protected val unique: String = className + "-" + hashCode() + protected val uniqueName: String = className + "-" + hashCode() protected val single = 1 val replicationDelay: Int = replaceProperty("replication_delay", 15).toInt @@ -55,7 +55,7 @@ class GerritSimulation extends Simulation { currentWaitTime } - protected val httpRequest: HttpRequestBuilder = http(unique).post("${url}") + protected val httpRequest: HttpRequestBuilder = http(uniqueName).post("${url}") protected val httpProtocol: HttpProtocolBuilder = http.basicAuth( conf.httpConfiguration.userName, conf.httpConfiguration.password) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetMasterBranchRevision.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetMasterBranchRevision.scala index 4ceba60e09..1137ad53b9 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetMasterBranchRevision.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetMasterBranchRevision.scala @@ -25,9 +25,9 @@ class GetMasterBranchRevision extends ProjectSimulation { val revisionKey = "revision" val revisionPattern: String = "\"" + revisionKey + "\": \"(.+)\"" - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) - .exec(http(unique).get("${url}") + .exec(http(uniqueName).get("${url}") .check(regex(revisionPattern).saveAs(revisionKey))) .exec(session => { revision = Some(session(revisionKey).as[String]) diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetProjectsCacheEntries.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetProjectsCacheEntries.scala index e73559e4ef..266c0b9da3 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetProjectsCacheEntries.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/GetProjectsCacheEntries.scala @@ -27,9 +27,9 @@ class GetProjectsCacheEntries extends CacheFlushSimulation { this.consumer = Some(consumer) } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) - .exec(http(unique).get("${url}") + .exec(http(uniqueName).get("${url}") .check(regex("\"" + memKey + "\": (\\d+)").saveAs(entriesKey))) .exec(session => { if (consumer.nonEmpty) { diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ListProjects.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ListProjects.scala index bfc97f405a..26aed0bad3 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ListProjects.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ListProjects.scala @@ -22,9 +22,9 @@ import io.gatling.http.Predef.http class ListProjects extends GerritSimulation { private val data: FeederBuilder = jsonFile(resource).convert(keys).queue - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) - .exec(http(unique).get("${url}")) + .exec(http(uniqueName).get("${url}")) setUp( test.inject( diff --git a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ReplayRecordsFromFeeder.scala b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ReplayRecordsFromFeeder.scala index 12fe47f6ff..613a73af2d 100644 --- a/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ReplayRecordsFromFeeder.scala +++ b/e2e-tests/src/test/scala/com/google/gerrit/scenarios/ReplayRecordsFromFeeder.scala @@ -30,7 +30,7 @@ class ReplayRecordsFromFeeder extends GitSimulation { replaceKeyWith("_project", default, in) } - private val test: ScenarioBuilder = scenario(unique) + private val test: ScenarioBuilder = scenario(uniqueName) .repeat(10) { feed(data) .exec(gitRequest) 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 a7aab67441..2d7a08e775 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 @@ -33,12 +33,12 @@ class SubmitChange extends GerritSimulation { this.createChange = createChange } - val test: ScenarioBuilder = scenario(unique) + val test: ScenarioBuilder = scenario(uniqueName) .feed(data) .exec(session => { session.set("number", createChange.number) }) - .exec(http(unique).post("${url}${number}/submit")) + .exec(http(uniqueName).post("${url}${number}/submit")) private val createProject = new CreateProject(default) private val approveChange = new ApproveChange(createChange) |