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

feat: Integrate flank.jar with corellium CLI #1945

Merged
merged 3 commits into from
May 18, 2021

Conversation

jan-goral
Copy link
Contributor

@jan-goral jan-goral commented May 18, 2021

Fixes #1944

Test Plan

How do we know the code works?

  • Unit tests pass
  • $ flank corellium test android run is executing flank.corellium.cli.RunTestCorelliumAndroidCommand

Checklist

  • Documented
  • Unit tested

@jan-goral jan-goral self-assigned this May 18, 2021
@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2021

Timestamp: 2021-05-18 15:30:38
Buildscan url for ubuntu-workflow run 853826209
https://gradle.com/s/aog3i3pgxqrm2

Base automatically changed from 1897-Corellium-fixes-3 to master May 18, 2021 13:07
@jan-goral jan-goral force-pushed the 1944_Integrate_flank-corellium_cli branch from d079394 to 17f9c7a Compare May 18, 2021 13:24
@jan-goral jan-goral marked this pull request as ready for review May 18, 2021 13:24
@bootstraponline bootstraponline force-pushed the 1944_Integrate_flank-corellium_cli branch from 17f9c7a to fd783c9 Compare May 18, 2021 13:38
@jan-goral jan-goral changed the title feat: Integrate flank.jar corellium cli feat: Integrate flank.jar with corellium CLI May 18, 2021
// TODO steps will be added in next PR
)
State() execute flowOf {
println("Not integrated yet, check following PR:")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Let's use logLn here

Copy link
Contributor Author

@jan-goral jan-goral May 18, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not accessible in this module.

@bootstraponline bootstraponline force-pushed the 1944_Integrate_flank-corellium_cli branch from fd783c9 to 25049a9 Compare May 18, 2021 15:24
@mergify mergify bot merged commit a9232d7 into master May 18, 2021
@mergify mergify bot deleted the 1944_Integrate_flank-corellium_cli branch May 18, 2021 16:07
@github-actions github-actions bot locked and limited conversation to collaborators May 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate flank.jar with corellium CLI
3 participants