Skip to content

Commit

Permalink
Rely on FTL to set the exti status correctly
Browse files Browse the repository at this point in the history
  • Loading branch information
bootstraponline committed Oct 8, 2019
1 parent c97b19a commit 8b61acc
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions test_runner/src/test/kotlin/ftl/reports/utils/ReportManagerTest.kt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package ftl.reports.utils

import com.google.common.truth.Truth.assertThat
import ftl.args.AndroidArgs
import ftl.args.IosArgs
import ftl.reports.util.ReportManager
import ftl.reports.xml.model.JUnitTestCase
import ftl.reports.xml.model.JUnitTestResult
Expand Down Expand Up @@ -45,18 +44,6 @@ class ReportManagerTest {
ReportManager.generate(matrix, mockArgs, emptyList())
}

@Test
fun `generate correct exit code from multi-suite ios result`() {
// TODO: NPE on Windows
val matrix = TestRunner.matrixPathToObj("./src/test/kotlin/ftl/fixtures/ios_exit_code", IosArgs.default())
val mockArgs = mock(IosArgs::class.java)
`when`(mockArgs.smartFlankGcsPath).thenReturn("")
// Must set flaky test attempts > 0 for exit code to be based on JUnit XML results.
`when`(mockArgs.flakyTestAttempts).thenReturn(1)
val exitCode = ReportManager.generate(matrix, mockArgs, emptyList())
assertThat(exitCode).isEqualTo(0)
}

@Test
fun createShardEfficiencyListTest() {
val oldRunTestCases = mutableListOf(
Expand Down

0 comments on commit 8b61acc

Please sign in to comment.