AndroidMidiTest moved to AndroidMidi module, project restructured to new Android conventions #8
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.
I've done some more restructuring to conform to the new Android project structure conventions and also allow building all of the projects contained in this repository with the new gradle build system. These changes include:
./gradlew assembleRelease
will now build all of them, not only AndroidMidi.It is now possible to run the tests by running
./gradlew test
from the command line or in Android Studio (For that note that you should click the 'Build Variants' tab and there select 'Unit Tests' as the test artifact).While running the tests, some tests in 'SystemMessageEncoderTest.java' failed. I'm not familiar with the code itself, so I'm not sure if this is a 'real' failure or perhaps a misconfiguration of the mock instance. I also don't know when was the last time that these tests passed, this is the first time today that I tried to run them. See the attached result:
TestResults.pdf