Benoit Marty
2 weeks ago
committed by
GitHub
14 changed files with 259 additions and 18 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,59 @@ |
|||||||
|
/* |
||||||
|
* 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.libraries.matrix.test.A_SESSION_ID |
||||||
|
import io.element.android.libraries.sessionstorage.impl.memory.InMemorySessionStore |
||||||
|
import io.element.android.libraries.sessionstorage.test.aSessionData |
||||||
|
import kotlinx.coroutines.test.runTest |
||||||
|
import org.junit.Test |
||||||
|
import java.io.File |
||||||
|
|
||||||
|
class AppMigration05Test { |
||||||
|
@Test |
||||||
|
fun `empty session path should be set to an expected path`() = runTest { |
||||||
|
val sessionStore = InMemorySessionStore().apply { |
||||||
|
updateData( |
||||||
|
aSessionData( |
||||||
|
sessionId = A_SESSION_ID, |
||||||
|
sessionPath = "", |
||||||
|
) |
||||||
|
) |
||||||
|
} |
||||||
|
val migration = AppMigration05(sessionStore = sessionStore, baseDirectory = File("/a/path")) |
||||||
|
migration.migrate() |
||||||
|
val storedData = sessionStore.getSession(A_SESSION_ID.value)!! |
||||||
|
assertThat(storedData.sessionPath).isEqualTo("/a/path/${A_SESSION_ID.value.replace(':', '_')}") |
||||||
|
} |
||||||
|
|
||||||
|
@Test |
||||||
|
fun `non empty session path should not be impacted by the migration`() = runTest { |
||||||
|
val sessionStore = InMemorySessionStore().apply { |
||||||
|
updateData( |
||||||
|
aSessionData( |
||||||
|
sessionId = A_SESSION_ID, |
||||||
|
sessionPath = "/a/path/existing", |
||||||
|
) |
||||||
|
) |
||||||
|
} |
||||||
|
val migration = AppMigration05(sessionStore = sessionStore, baseDirectory = File("/a/path")) |
||||||
|
migration.migrate() |
||||||
|
val storedData = sessionStore.getSession(A_SESSION_ID.value)!! |
||||||
|
assertThat(storedData.sessionPath).isEqualTo("/a/path/existing") |
||||||
|
} |
||||||
|
} |
@ -0,0 +1,60 @@ |
|||||||
|
/* |
||||||
|
* 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.libraries.matrix.test.A_SESSION_ID |
||||||
|
import io.element.android.libraries.sessionstorage.impl.memory.InMemorySessionStore |
||||||
|
import io.element.android.libraries.sessionstorage.test.aSessionData |
||||||
|
import kotlinx.coroutines.test.runTest |
||||||
|
import org.junit.Test |
||||||
|
import java.io.File |
||||||
|
|
||||||
|
class AppMigration06Test { |
||||||
|
@Test |
||||||
|
fun `empty cache path should be set to an expected path`() = runTest { |
||||||
|
val sessionStore = InMemorySessionStore().apply { |
||||||
|
updateData( |
||||||
|
aSessionData( |
||||||
|
sessionId = A_SESSION_ID, |
||||||
|
sessionPath = "/a/path/to/a/session/AN_ID", |
||||||
|
cachePath = "", |
||||||
|
) |
||||||
|
) |
||||||
|
} |
||||||
|
val migration = AppMigration06(sessionStore = sessionStore, cacheDirectory = File("/a/path/cache")) |
||||||
|
migration.migrate() |
||||||
|
val storedData = sessionStore.getSession(A_SESSION_ID.value)!! |
||||||
|
assertThat(storedData.cachePath).isEqualTo("/a/path/cache/AN_ID") |
||||||
|
} |
||||||
|
|
||||||
|
@Test |
||||||
|
fun `non empty cache path should not be impacted by the migration`() = runTest { |
||||||
|
val sessionStore = InMemorySessionStore().apply { |
||||||
|
updateData( |
||||||
|
aSessionData( |
||||||
|
sessionId = A_SESSION_ID, |
||||||
|
cachePath = "/a/path/existing", |
||||||
|
) |
||||||
|
) |
||||||
|
} |
||||||
|
val migration = AppMigration05(sessionStore = sessionStore, baseDirectory = File("/a/path/cache")) |
||||||
|
migration.migrate() |
||||||
|
val storedData = sessionStore.getSession(A_SESSION_ID.value)!! |
||||||
|
assertThat(storedData.cachePath).isEqualTo("/a/path/existing") |
||||||
|
} |
||||||
|
} |
@ -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() |
||||||
|
performLambda.assertions().isCalledOnce() |
||||||
|
} |
||||||
|
} |
Loading…
Reference in new issue