[Experimental]Android support #258
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Partialy done still failing tests and inner static classes break reflection process
…y can access super constructors and set their visibility for reflection "raphw/byte-buddy#199 (comment)"
Enable method handle after constructor call
(failing due to missing DexOptions still)
…bytebuddy Adapted the Jgiven rule so a cache dir from the device is fetched for saving the test results problem left: pulling the results back on the build machine, and creating html reports automagically unknown: ansiUtils proper usage?
# Conflicts: # gradle.properties took over version number, but kept the org.gradle.jvmargs=-Xmx1536M
- JSON reports are now stored in the public Download folder of the Android device. That way the reports are still available after the application has been uninstalled, which cannot currently prevented with the Gradle connectedAndroidTest task - This requires that the permission WRITE_EXTERNAL_STORAGE - After the test, the JSON files are pulled from the device and the HTML report can be generated
- Also grant read access at runtime, which seems to be necessary - Improved the Gradle tasks to properly clean up - Made the Activity and the JGiven test more interesting
- This refactoring should be largely backwards compatible as long as no internal JGiven classes have been used - Created a new interface StageCreator to make it easier to change the stage creation strategy. This simplifies the Spring and the Android integrations - Changed the way the StepInterceptor is used - Created new classes JGivenMethodRule and JGivenClassRule to replace the old rule classes. The old classes are deprecated now - Simplified the implementation of the Spring and Android integrations
Enable downloading of JGiven JSON reports from Android Device
Closed
|
Great, thanks! I will merge as soon as I have configured Travis to build the Android projects. |
|
For now, I have just excluded the modules from the Travis CI build. This can be fixed later. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
lets merge it to master and polish it with additional commits so it does not get stale (hopefully it could also attract attention from other android devs)