From 0b2d779a0168a28284677f5372b25ff82eb1797a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Eduard-Cristian=20Bolo=C8=99?= Date: Wed, 19 Jun 2024 13:07:34 +0200 Subject: [PATCH] Remove redundant endTimestamp field --- .../com/automattic/android/measure/lifecycle/BuildTaskService.kt | 1 - .../java/com/automattic/android/measure/models/MeasuredTask.kt | 1 - .../java/com/automattic/android/measure/BuildTimePluginTest.kt | 1 - 3 files changed, 3 deletions(-) diff --git a/measure-builds/src/main/java/com/automattic/android/measure/lifecycle/BuildTaskService.kt b/measure-builds/src/main/java/com/automattic/android/measure/lifecycle/BuildTaskService.kt index b8a5292..ba43a30 100644 --- a/measure-builds/src/main/java/com/automattic/android/measure/lifecycle/BuildTaskService.kt +++ b/measure-builds/src/main/java/com/automattic/android/measure/lifecycle/BuildTaskService.kt @@ -34,7 +34,6 @@ abstract class BuildTaskService : name = event.descriptor?.name.toString(), duration = (event.result.endTime - event.result.startTime).milliseconds, startTimestamp = event.result.startTime, - endTimestamp = event.result.endTime, state = when { result.isFromCache -> IS_FROM_CACHE result.isUpToDate -> UP_TO_DATE diff --git a/measure-builds/src/main/java/com/automattic/android/measure/models/MeasuredTask.kt b/measure-builds/src/main/java/com/automattic/android/measure/models/MeasuredTask.kt index b9e089f..23d1e17 100644 --- a/measure-builds/src/main/java/com/automattic/android/measure/models/MeasuredTask.kt +++ b/measure-builds/src/main/java/com/automattic/android/measure/models/MeasuredTask.kt @@ -8,7 +8,6 @@ data class MeasuredTask( val name: String, val duration: Duration, val startTimestamp: Long, - val endTimestamp: Long, val state: State ) { enum class State { diff --git a/measure-builds/src/test/java/com/automattic/android/measure/BuildTimePluginTest.kt b/measure-builds/src/test/java/com/automattic/android/measure/BuildTimePluginTest.kt index 3015136..b42111e 100644 --- a/measure-builds/src/test/java/com/automattic/android/measure/BuildTimePluginTest.kt +++ b/measure-builds/src/test/java/com/automattic/android/measure/BuildTimePluginTest.kt @@ -138,7 +138,6 @@ class BuildTimePluginTest { assertThat(executionData.executedTasks).hasSize(1).first().satisfies({ task -> assertThat(task.name).isEqualTo(":help") assertThat(task.state).isEqualTo(MeasuredTask.State.EXECUTED) - assertThat(task.endTimestamp - task.startTimestamp).isEqualTo(task.duration.inWholeMilliseconds) }) } }