futurice / freesound-android

Unofficial Android client for the Freesound Project
Apache License 2.0
85 stars 16 forks source link

Add tests for WaveformExtractor #13

Open peter-tackage opened 8 years ago

peter-tackage commented 8 years ago

The WaveformExtractor class (and its concrete implementations) have logic in them that would be very useful to test. The classes contain Android dependencies, such as Bitmap, so either use Android instrumentation tests or possibly add a testable Bitmap interface.

peter-tackage commented 7 years ago

This this a case where Robolectric should be used? Could possibly pull out some of the logic into a separate class and androidTest it too.