Signal/Android
1
0
Fork 0
mirror of https://github.com/signalapp/Signal-Android.git synced 2025-12-27 14:40:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Ignore expensive AttachmentCipherTests.

Browse Source 
They can take up to 15 seconds to run, and they've already proven
their usefulness. We can un-ignore them temporarily if we ever need
to make changes.
This commit is contained in:
Greyson Parrelli
2025-08-22 09:55:28 -04:00
committed by Michelle Tang
parent a234896438
commit f052b1fd90
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
libsignal-service/src/test/java/org/whispersystems/signalservice/api/crypto/AttachmentCipherTest.kt
View File

@@ -6,6 +6,7 @@ import assertk.assertions.isTrue
import assertk.fail
import org.conscrypt.Conscrypt
import org.junit.Assert
import org.junit.Ignore
import org.junit.Test
import org.signal.core.util.StreamUtil
import org.signal.core.util.readFully
@@ -62,6 +63,7 @@ class AttachmentCipherTest {
attachment_encryptDecrypt(incremental = true, fileSize = MEBIBYTE, integrityCheckMode = IntegrityCheckMode.BOTH)
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun attachment_encryptDecrypt_nonIncremental_manyFileSizes() {
for (i in 0..99) {
@@ -69,6 +71,7 @@ class AttachmentCipherTest {
}
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun attachment_encryptDecrypt_incremental_manyFileSizes() {
// Designed to stress the various boundary conditions of reading the final mac
@@ -101,6 +104,7 @@ class AttachmentCipherTest {
cipherFile.delete()
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun attachment_encryptDecrypt_skipAll_manyFileSizes() {
for (i in 0..99) {
@@ -416,6 +420,7 @@ class AttachmentCipherTest {
archive_encryptDecrypt(incremental = true, fileSize = MEBIBYTE)
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun archive_encryptDecrypt_nonIncremental_manyFileSizes() {
for (i in 0..99) {
@@ -423,6 +428,7 @@ class AttachmentCipherTest {
}
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun archive_encryptDecrypt_incremental_manyFileSizes() {
// Designed to stress the various boundary conditions of reading the final mac
@@ -458,6 +464,7 @@ class AttachmentCipherTest {
cipherFile.delete()
}
@Ignore("Useful when making changes, otherwise a bit slow.")
@Test
fun archiveThumbnail_encryptDecrypt_manyFileSizes() {
for (i in 0..99) {