Revert "@Ignore all tests currently failing in CI"

This reverts commit 8d234b4b65a7a899abdeed9c5167a8c09c9af3e0.
This commit is contained in:
Michael Kaye 2022-05-13 16:07:10 +01:00
parent 70682b4931
commit fa26e2a23e
11 changed files with 0 additions and 45 deletions

View file

@ -23,7 +23,6 @@ import org.amshove.kluent.fail
import org.amshove.kluent.internal.assertEquals import org.amshove.kluent.internal.assertEquals
import org.junit.Assert import org.junit.Assert
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.JUnit4 import org.junit.runners.JUnit4
@ -78,7 +77,6 @@ class E2eeSanityTests : InstrumentedTest {
* Alice sends a new message, then check that the new one can be decrypted * Alice sends a new message, then check that the new one can be decrypted
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun testSendingE2EEMessages() { fun testSendingE2EEMessages() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -350,7 +348,6 @@ class E2eeSanityTests : InstrumentedTest {
* get them from an older one. * get them from an older one.
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun testSimpleGossip() { fun testSimpleGossip() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -456,7 +453,6 @@ class E2eeSanityTests : InstrumentedTest {
* Test that if a better key is forwarded (lower index, it is then used) * Test that if a better key is forwarded (lower index, it is then used)
*/ */
@Test @Test
@Ignore("This test is failing in CI")
fun testForwardBetterKey() { fun testForwardBetterKey() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)

View file

@ -21,7 +21,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4
import org.junit.Assert.assertEquals import org.junit.Assert.assertEquals
import org.junit.Assert.assertNotNull import org.junit.Assert.assertNotNull
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -42,7 +41,6 @@ class PreShareKeysTest : InstrumentedTest {
private val cryptoTestHelper = CryptoTestHelper(testHelper) private val cryptoTestHelper = CryptoTestHelper(testHelper)
@Test @Test
@Ignore("This test is failing on CI")
fun ensure_outbound_session_happy_path() { fun ensure_outbound_session_happy_path() {
val testData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom(true) val testData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom(true)
val e2eRoomID = testData.roomId val e2eRoomID = testData.roomId

View file

@ -21,7 +21,6 @@ import org.amshove.kluent.shouldBe
import org.junit.Assert import org.junit.Assert
import org.junit.Before import org.junit.Before
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -86,7 +85,6 @@ class UnwedgingTest : InstrumentedTest {
* -> This is automatically fixed after SDKs restarted the olm session * -> This is automatically fixed after SDKs restarted the olm session
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun testUnwedging() { fun testUnwedging() {
val cryptoTestData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom() val cryptoTestData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom()

View file

@ -54,7 +54,6 @@ class XSigningTest : InstrumentedTest {
private val cryptoTestHelper = CryptoTestHelper(testHelper) private val cryptoTestHelper = CryptoTestHelper(testHelper)
@Test @Test
@Ignore("This test is failing in CI")
fun test_InitializeAndStoreKeys() { fun test_InitializeAndStoreKeys() {
val aliceSession = testHelper.createAccount(TestConstants.USER_ALICE, SessionTestParams(true)) val aliceSession = testHelper.createAccount(TestConstants.USER_ALICE, SessionTestParams(true))
@ -89,7 +88,6 @@ class XSigningTest : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing in CI")
fun test_CrossSigningCheckBobSeesTheKeys() { fun test_CrossSigningCheckBobSeesTheKeys() {
val cryptoTestData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom() val cryptoTestData = cryptoTestHelper.doE2ETestWithAliceAndBobInARoom()

View file

@ -20,7 +20,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4
import kotlinx.coroutines.runBlocking import kotlinx.coroutines.runBlocking
import org.amshove.kluent.shouldBe import org.amshove.kluent.shouldBe
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -47,7 +46,6 @@ class EncryptionTest : InstrumentedTest {
private val cryptoTestHelper = CryptoTestHelper(testHelper) private val cryptoTestHelper = CryptoTestHelper(testHelper)
@Test @Test
@Ignore("This test is failing in CI")
fun test_EncryptionEvent() { fun test_EncryptionEvent() {
performTest(roomShouldBeEncrypted = false) { room -> performTest(roomShouldBeEncrypted = false) { room ->
// Send an encryption Event as an Event (and not as a state event) // Send an encryption Event as an Event (and not as a state event)
@ -59,7 +57,6 @@ class EncryptionTest : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing in CI")
fun test_EncryptionStateEvent() { fun test_EncryptionStateEvent() {
performTest(roomShouldBeEncrypted = true) { room -> performTest(roomShouldBeEncrypted = true) { room ->
runBlocking { runBlocking {

View file

@ -26,7 +26,6 @@ import org.amshove.kluent.internal.assertEquals
import org.junit.Assert import org.junit.Assert
import org.junit.Assert.assertNull import org.junit.Assert.assertNull
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -53,7 +52,6 @@ import org.matrix.android.sdk.common.TestConstants
class KeyShareTests : InstrumentedTest { class KeyShareTests : InstrumentedTest {
@Test @Test
@Ignore("This test is failing on CI")
fun test_DoNotSelfShareIfNotTrusted() { fun test_DoNotSelfShareIfNotTrusted() {
val commonTestHelper = CommonTestHelper(context()) val commonTestHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(commonTestHelper) val cryptoTestHelper = CryptoTestHelper(commonTestHelper)
@ -196,7 +194,6 @@ class KeyShareTests : InstrumentedTest {
* if the key was originally shared with him * if the key was originally shared with him
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun test_reShareIfWasIntendedToBeShared() { fun test_reShareIfWasIntendedToBeShared() {
val commonTestHelper = CommonTestHelper(context()) val commonTestHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(commonTestHelper) val cryptoTestHelper = CryptoTestHelper(commonTestHelper)
@ -230,7 +227,6 @@ class KeyShareTests : InstrumentedTest {
* if the key was originally shared with him * if the key was originally shared with him
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun test_reShareToUnverifiedIfWasIntendedToBeShared() { fun test_reShareToUnverifiedIfWasIntendedToBeShared() {
val commonTestHelper = CommonTestHelper(context()) val commonTestHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(commonTestHelper) val cryptoTestHelper = CryptoTestHelper(commonTestHelper)
@ -270,7 +266,6 @@ class KeyShareTests : InstrumentedTest {
* Tests that keys reshared with own verified session are done from the earliest known index * Tests that keys reshared with own verified session are done from the earliest known index
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun test_reShareFromTheEarliestKnownIndexWithOwnVerifiedSession() { fun test_reShareFromTheEarliestKnownIndexWithOwnVerifiedSession() {
val commonTestHelper = CommonTestHelper(context()) val commonTestHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(commonTestHelper) val cryptoTestHelper = CryptoTestHelper(commonTestHelper)
@ -393,7 +388,6 @@ class KeyShareTests : InstrumentedTest {
* Tests that we don't cancel a request to early on first forward if the index is not good enough * Tests that we don't cancel a request to early on first forward if the index is not good enough
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun test_dontCancelToEarly() { fun test_dontCancelToEarly() {
val commonTestHelper = CommonTestHelper(context()) val commonTestHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(commonTestHelper) val cryptoTestHelper = CryptoTestHelper(commonTestHelper)

View file

@ -21,7 +21,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4
import androidx.test.filters.LargeTest import androidx.test.filters.LargeTest
import org.junit.Assert import org.junit.Assert
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -48,7 +47,6 @@ import org.matrix.android.sdk.common.TestConstants
class WithHeldTests : InstrumentedTest { class WithHeldTests : InstrumentedTest {
@Test @Test
@Ignore("This test is failing on CI")
fun test_WithHeldUnverifiedReason() { fun test_WithHeldUnverifiedReason() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -154,7 +152,6 @@ class WithHeldTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_WithHeldNoOlm() { fun test_WithHeldNoOlm() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -238,7 +235,6 @@ class WithHeldTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_WithHeldKeyRequest() { fun test_WithHeldKeyRequest() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)

View file

@ -102,7 +102,6 @@ class KeysBackupTest : InstrumentedTest {
* Check that prepareKeysBackupVersionWithPassword returns valid data * Check that prepareKeysBackupVersionWithPassword returns valid data
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun prepareKeysBackupVersionTest() { fun prepareKeysBackupVersionTest() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
@ -958,7 +957,6 @@ class KeysBackupTest : InstrumentedTest {
* -> That must fail and her backup state must be WrongBackUpVersion * -> That must fail and her backup state must be WrongBackUpVersion
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun testBackupWhenAnotherBackupWasCreated() { fun testBackupWhenAnotherBackupWasCreated() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -1136,7 +1134,6 @@ class KeysBackupTest : InstrumentedTest {
* - Delete the backup * - Delete the backup
*/ */
@Test @Test
@Ignore("This test is failing on CI")
fun deleteKeysBackupTest() { fun deleteKeysBackupTest() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)

View file

@ -55,7 +55,6 @@ class QuadSTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_Generate4SKey() { fun test_Generate4SKey() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
@ -109,7 +108,6 @@ class QuadSTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_StoreSecret() { fun test_StoreSecret() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
@ -155,7 +153,6 @@ class QuadSTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_SetDefaultLocalEcho() { fun test_SetDefaultLocalEcho() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
@ -178,7 +175,6 @@ class QuadSTests : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_StoreSecretWithMultipleKey() { fun test_StoreSecretWithMultipleKey() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())

View file

@ -56,7 +56,6 @@ import java.util.concurrent.CountDownLatch
class SASTest : InstrumentedTest { class SASTest : InstrumentedTest {
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceStartThenAliceCancel() { fun test_aliceStartThenAliceCancel() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -315,7 +314,6 @@ class SASTest : InstrumentedTest {
// any two devices may only have at most one key verification in flight at a time. // any two devices may only have at most one key verification in flight at a time.
// If a device has two verifications in progress with the same device, then it should cancel both verifications. // If a device has two verifications in progress with the same device, then it should cancel both verifications.
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceStartTwoRequests() { fun test_aliceStartTwoRequests() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -420,7 +418,6 @@ class SASTest : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBobSASCode() { fun test_aliceAndBobSASCode() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -481,7 +478,6 @@ class SASTest : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_happyPath() { fun test_happyPath() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)
@ -561,7 +557,6 @@ class SASTest : InstrumentedTest {
} }
@Test @Test
@Ignore("This test is failing on CI")
fun test_ConcurrentStart() { fun test_ConcurrentStart() {
val testHelper = CommonTestHelper(context()) val testHelper = CommonTestHelper(context())
val cryptoTestHelper = CryptoTestHelper(testHelper) val cryptoTestHelper = CryptoTestHelper(testHelper)

View file

@ -19,7 +19,6 @@ package org.matrix.android.sdk.internal.crypto.verification.qrcode
import androidx.test.ext.junit.runners.AndroidJUnit4 import androidx.test.ext.junit.runners.AndroidJUnit4
import org.amshove.kluent.shouldBe import org.amshove.kluent.shouldBe
import org.junit.FixMethodOrder import org.junit.FixMethodOrder
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import org.junit.runner.RunWith import org.junit.runner.RunWith
import org.junit.runners.MethodSorters import org.junit.runners.MethodSorters
@ -69,7 +68,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_sas_sas() = doTest( fun test_aliceAndBob_sas_sas() = doTest(
sas, sas,
sas, sas,
@ -78,7 +76,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_sas_show() = doTest( fun test_aliceAndBob_sas_show() = doTest(
sas, sas,
sasShow, sasShow,
@ -95,7 +92,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_sas_scan() = doTest( fun test_aliceAndBob_sas_scan() = doTest(
sas, sas,
sasScan, sasScan,
@ -104,7 +100,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_scan_sas() = doTest( fun test_aliceAndBob_scan_sas() = doTest(
sasScan, sasScan,
sas, sas,
@ -113,7 +108,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_scan_scan() = doTest( fun test_aliceAndBob_scan_scan() = doTest(
sasScan, sasScan,
sasScan, sasScan,
@ -122,7 +116,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_show_show() = doTest( fun test_aliceAndBob_show_show() = doTest(
sasShow, sasShow,
sasShow, sasShow,
@ -131,7 +124,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_show_scan() = doTest( fun test_aliceAndBob_show_scan() = doTest(
sasShow, sasShow,
sasScan, sasScan,
@ -140,7 +132,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_scan_show() = doTest( fun test_aliceAndBob_scan_show() = doTest(
sasScan, sasScan,
sasShow, sasShow,
@ -149,7 +140,6 @@ class VerificationTest : InstrumentedTest {
) )
@Test @Test
@Ignore("This test is failing on CI")
fun test_aliceAndBob_all_all() = doTest( fun test_aliceAndBob_all_all() = doTest(
sasShowScan, sasShowScan,
sasShowScan, sasShowScan,