Add an application migration to delete the old log files.
This commit is contained in:
parent
4b4acfca1e
commit
5b13a51b0c
6 changed files with 96 additions and 9 deletions
|
|
@ -0,0 +1,38 @@
|
|||
/*
|
||||
* Copyright (c) 2024 New Vector Ltd
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package io.element.android.features.migration.impl.migrations
|
||||
|
||||
import com.squareup.anvil.annotations.ContributesMultibinding
|
||||
import io.element.android.features.rageshake.api.logs.LogFilesRemover
|
||||
import io.element.android.libraries.di.AppScope
|
||||
import javax.inject.Inject
|
||||
|
||||
/**
|
||||
* Delete the previous log files.
|
||||
*/
|
||||
@ContributesMultibinding(AppScope::class)
|
||||
class AppMigration07 @Inject constructor(
|
||||
private val logFilesRemover: LogFilesRemover,
|
||||
) : AppMigration {
|
||||
override val order: Int = 7
|
||||
|
||||
override suspend fun migrate() {
|
||||
logFilesRemover.perform { file ->
|
||||
file.name.startsWith("logs-")
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -0,0 +1,39 @@
|
|||
/*
|
||||
* Copyright (c) 2024 New Vector Ltd
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
* You may obtain a copy of the License at
|
||||
*
|
||||
* http://www.apache.org/licenses/LICENSE-2.0
|
||||
*
|
||||
* Unless required by applicable law or agreed to in writing, software
|
||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
* See the License for the specific language governing permissions and
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
package io.element.android.features.migration.impl.migrations
|
||||
|
||||
import com.google.common.truth.Truth.assertThat
|
||||
import io.element.android.features.rageshake.test.logs.FakeLogFilesRemover
|
||||
import io.element.android.tests.testutils.lambda.lambdaRecorder
|
||||
import kotlinx.coroutines.test.runTest
|
||||
import org.junit.Test
|
||||
import java.io.File
|
||||
|
||||
class AppMigration07Test {
|
||||
@Test
|
||||
fun `test migration`() = runTest {
|
||||
val performLambda = lambdaRecorder<(File) -> Boolean, Unit> { predicate ->
|
||||
// Test the predicate
|
||||
assertThat(predicate(File("logs-0433.log.gz"))).isTrue()
|
||||
assertThat(predicate(File("logs.2024-08-01-20.log.gz"))).isFalse()
|
||||
}
|
||||
val logsFileRemover = FakeLogFilesRemover(performLambda = performLambda)
|
||||
val migration = AppMigration07(logsFileRemover)
|
||||
migration.migrate()
|
||||
logsFileRemover.performLambda.assertions().isCalledOnce()
|
||||
}
|
||||
}
|
||||
Loading…
Add table
Add a link
Reference in a new issue