Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

step.testExecutionStep.testTiming must not be null #561

Closed
bootstraponline opened this issue Jun 3, 2019 · 4 comments

Comments

Projects
None yet
2 participants
@bootstraponline
Copy link
Contributor

commented Jun 3, 2019

Exception in thread "main" picocli.CommandLine$ExecutionException: Error while running command (ftl.cli.firebase.test.android.AndroidRunCommand@3b582111): java.lang.IllegalStateException: step.testExecutionStep.testTiming must not be null
    at picocli.CommandLine.execute(CommandLine.java:1051)
    at picocli.CommandLine.access$900(CommandLine.java:142)
    at picocli.CommandLine$RunLast.handle(CommandLine.java:1246)
    at picocli.CommandLine$RunLast.handle(CommandLine.java:1214)
    at picocli.CommandLine$AbstractParseResultHandler.handleParseResult(CommandLine.java:1122)
    at picocli.CommandLine.parseWithHandlers(CommandLine.java:1405)
    at picocli.CommandLine.run(CommandLine.java:1864)
    at picocli.CommandLine.run(CommandLine.java:1810)
    at ftl.Main$Companion.main(Main.kt:42)
    at ftl.Main.main(Main.kt)
Caused by: java.lang.IllegalStateException: step.testExecutionStep.testTiming must not be null
    at ftl.json.SavedMatrix.finished(SavedMatrix.kt:81)
    at ftl.json.SavedMatrix.update(SavedMatrix.kt:57)
    at ftl.run.TestRunner.pollMatrices(TestRunner.kt:249)
    at ftl.run.TestRunner.newRun(TestRunner.kt:348)
    at ftl.run.TestRunner$newRun$1.invokeSuspend(TestRunner.kt)
    at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:32)
    at kotlinx.coroutines.ResumeModeKt.resumeUninterceptedMode(ResumeMode.kt:45)
    at kotlinx.coroutines.internal.ScopeCoroutine.onCompletionInternal$kotlinx_coroutines_core(Scopes.kt:25)
    at kotlinx.coroutines.JobSupport.completeStateFinalization(JobSupport.kt:299)
    at kotlinx.coroutines.JobSupport.tryFinalizeFinishingState(JobSupport.kt:225)
    at kotlinx.coroutines.JobSupport.tryMakeCompleting(JobSupport.kt:792)
    at kotlinx.coroutines.JobSupport.makeCompletingOnce$kotlinx_coroutines_core(JobSupport.kt:735)
    at kotlinx.coroutines.AbstractCoroutine.resumeWith(AbstractCoroutine.kt:117)
    at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:45)
    at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:236)
    at kotlinx.coroutines.EventLoopBase.processNextEvent(EventLoop.kt:123)
    at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:69)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:45)
    at kotlinx.coroutines.BuildersKt.runBlocking(Unknown Source)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(Builders.kt:35)
    at kotlinx.coroutines.BuildersKt.runBlocking$default(Unknown Source)
    at ftl.cli.firebase.test.android.AndroidRunCommand.run(AndroidRunCommand.kt:34)
    at picocli.CommandLine.execute(CommandLine.java:1043)
    ... 9 more
``
@niranjani23

This comment has been minimized.

Copy link

commented Jun 3, 2019

Thanks for creating the bug @bootstraponline !

@bootstraponline bootstraponline self-assigned this Jun 3, 2019

@bootstraponline

This comment has been minimized.

Copy link
Contributor Author

commented Jun 4, 2019

This should be already fixed in the latest stable version
https://github.com/TestArmada/flank/releases/tag/v6.2.0

Relevant line of code:

val testTimeSeconds = step.testExecutionStep?.testTiming?.testProcessDuration?.seconds ?: return 

https://github.com/TestArmada/flank/blob/master/test_runner/src/main/kotlin/ftl/json/SavedMatrix.kt#L83-L84

@niranjani23

This comment has been minimized.

Copy link

commented Jun 4, 2019

Will update my flank version and check. Am currently using 4.1.1

@bootstraponline

This comment has been minimized.

Copy link
Contributor Author

commented Jun 6, 2019

I think the bug has been resolved in the latest version. If not, let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.