Skip to content

Commit

Permalink
#100 ControlMeasure integTest not needed, sufficiently covered by Con…
Browse files Browse the repository at this point in the history
…trolMeasureBuilderTest (+adjusted the to check the usage of `cmb.withSoftware()`)
  • Loading branch information
dk1844 committed Jul 28, 2021
1 parent 8e879c3 commit 9f8e998
Showing 1 changed file with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ class ControlMeasureBuilderTest extends AnyFlatSpec with ControlMeasureBaseTestS
.withSourceType("SourceType1")
.withInitialCheckpointName("InitCheckpoint1")
.withWorkflowName("Workflow1")
.withSoftware("MyAwesomeSw", "v1.2.3-beta.4")
.build

val expectedCustomControlMeasure: ControlMeasure = ControlMeasure(
Expand All @@ -95,10 +96,10 @@ class ControlMeasureBuilderTest extends AnyFlatSpec with ControlMeasureBaseTestS
)
)

// prior to stabilization, let's check the actual by-default generated software fields:
customCm.checkpoints.map(_.software).foreach { swName =>
swName shouldBe defined
swName.get should fullyMatch regex("""^atum_(2\.11|2\.12)$""")
// prior to stabilization, let's check the custom software/version fields:
customCm.checkpoints.foreach { cp =>
cp.software shouldBe Some("MyAwesomeSw")
cp.version shouldBe Some("v1.2.3-beta.4")
}

customCm.stabilizeTestingControlMeasure shouldBe expectedCustomControlMeasure
Expand Down

0 comments on commit 9f8e998

Please sign in to comment.