Skip to content

Commit 92065f6

Browse files
committed
#188: optimization / refactoring
1 parent 464752a commit 92065f6

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

server/src/test/scala/za/co/absa/atum/server/api/repository/PartitioningRepositoryUnitTests.scala

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,9 @@
1717
package za.co.absa.atum.server.api.repository
1818

1919
import org.mockito.Mockito.{mock, when}
20-
import za.co.absa.atum.model.dto.{AdditionalDataDTO, MeasureDTO}
21-
import za.co.absa.atum.server.api.database.runs.functions.{
22-
CreateOrUpdateAdditionalData,
23-
CreatePartitioningIfNotExists,
24-
GetPartitioningAdditionalData,
25-
GetPartitioningCheckpoints,
26-
GetPartitioningMeasures}
27-
import za.co.absa.atum.server.api.exception.DatabaseError
2820
import za.co.absa.atum.server.api.TestData
21+
import za.co.absa.atum.server.api.database.runs.functions._
22+
import za.co.absa.atum.server.api.exception.DatabaseError
2923
import za.co.absa.atum.server.model.CheckpointFromDB
3024
import za.co.absa.fadb.exceptions.ErrorInDataException
3125
import za.co.absa.fadb.status.FunctionStatus
@@ -126,7 +120,7 @@ object PartitioningRepositoryUnitTests extends ZIOSpecDefault with TestData {
126120
test("Returns expected Seq") {
127121
for {
128122
result <- PartitioningRepository.getPartitioningMeasures(partitioningDTO1)
129-
} yield assertTrue(result.isInstanceOf[Seq[MeasureDTO]] && result == Seq(measureDTO1, measureDTO2))
123+
} yield assertTrue(result == Seq(measureDTO1, measureDTO2))
130124
},
131125
test("Returns expected Exception") {
132126
assertZIO(PartitioningRepository.getPartitioningMeasures(partitioningDTO2).exit)(
@@ -139,7 +133,7 @@ object PartitioningRepositoryUnitTests extends ZIOSpecDefault with TestData {
139133
test("Returns expected Right with empty Map") {
140134
for {
141135
result <- PartitioningRepository.getPartitioningAdditionalData(partitioningDTO1)
142-
} yield assertTrue(result.isInstanceOf[AdditionalDataDTO] && result == additionalDataDTO1)
136+
} yield assertTrue(result == additionalDataDTO1)
143137
},
144138
test("Returns expected Left with DatabaseError") {
145139
assertZIO(PartitioningRepository.getPartitioningAdditionalData(partitioningDTO2).exit)(
@@ -152,7 +146,7 @@ object PartitioningRepositoryUnitTests extends ZIOSpecDefault with TestData {
152146
test("Returns expected Seq") {
153147
for {
154148
result <- PartitioningRepository.getPartitioningCheckpoints(checkpointQueryDTO1)
155-
} yield assertTrue(result.isInstanceOf[Seq[CheckpointFromDB]] && result == Seq(checkpointFromDB1))
149+
} yield assertTrue(result == Seq(checkpointFromDB1))
156150
},
157151
test("Returns expected DatabaseError") {
158152
assertZIO(PartitioningRepository.getPartitioningCheckpoints(checkpointQueryDTO2).exit)(

0 commit comments

Comments
 (0)