Addressing comments from a475cd4d33.

This commit is contained in:
Rodrigo Damazio
2010-11-21 00:44:34 -08:00
parent 6a29531467
commit cb97847392
2 changed files with 3 additions and 2 deletions
@@ -43,6 +43,7 @@ public class StatusAnnouncerTask implements PeriodicTask {
/**
* The rate at which announcements are spoken.
*/
// @VisibleForTesting
static final float TTS_SPEECH_RATE = 0.9f;
/**
@@ -99,7 +100,7 @@ public class StatusAnnouncerTask implements PeriodicTask {
*/
private void onTtsInit(int status) {
Log.i(MyTracksConstants.TAG, "TrackRecordingService.TTS init: " + status);
this.ready = (status == TextToSpeech.SUCCESS);
this.ready = status == TextToSpeech.SUCCESS;
if (ready) {
// Force the language to be the same as the string we will be speaking,
@@ -97,7 +97,7 @@ public class StatusAnnouncerTaskTest extends AndroidTestCase {
* like this, but basically the issue is that TextToSpeech cannot be mocked
* without running its constructor, its constructor runs async operations
* which call other methods (and then if the methods are part of a mock we'd
* have to set a behaviour, but we can't 'cause the object hasn't been fully
* have to set a behavior, but we can't 'cause the object hasn't been fully
* built yet).
* The logic is that calls made during the constructor (when tts is not yet
* set) will go up to the original class, but after tts is set we'll forward