diff --git a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTest.java b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTest.java index 814f01232..92bbb6337 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTest.java +++ b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTest.java @@ -211,6 +211,7 @@ public class TrackRecordingServiceTest { trackPointCreator.setClock(Clock.fixed(Instant.parse("2020-02-02T02:02:02Z"), ZoneId.of("CET"))); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); trackPointCreator.stopGPS(); trackPointCreator.setAltitudeSumManager(altitudeSumManager); @@ -241,6 +242,7 @@ public class TrackRecordingServiceTest { trackPointCreator.setClock(Clock.fixed(Instant.parse("2020-02-02T02:02:02Z"), ZoneId.of("CET"))); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); trackPointCreator.stopGPS(); trackPointCreator.setAltitudeSumManager(altitudeSumManager); @@ -280,6 +282,7 @@ public class TrackRecordingServiceTest { trackPointCreator.setClock(Clock.fixed(Instant.parse("2020-02-02T02:02:02Z"), ZoneId.of("CET"))); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); trackPointCreator.stopGPS(); trackPointCreator.setAltitudeSumManager(altitudeSumManager); @@ -322,6 +325,7 @@ public class TrackRecordingServiceTest { trackPointCreator.setClock(Clock.fixed(Instant.parse("2020-02-02T02:02:02Z"), ZoneId.of("CET"))); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); trackPointCreator.stopGPS(); trackPointCreator.setAltitudeSumManager(altitudeSumManager); @@ -352,10 +356,13 @@ public class TrackRecordingServiceTest { TrackRecordingService service = ((TrackRecordingService.Binder) mServiceRule.bindService(createStartIntent(context))) .getService(); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); + assertTrue(service.isRecording()); // when Track.Id newTrackId = service.startNewTrack(); + service.stopUpdateRecordingData(); // then assertNotNull(trackId); @@ -405,6 +412,7 @@ public class TrackRecordingServiceTest { trackPointCreator.setClock(Clock.fixed(Instant.parse("2020-02-02T02:02:02Z"), ZoneId.of("CET"))); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); assertTrue(service.isRecording()); trackPointCreator.onNewTrackPoint( diff --git a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLocation.java b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLocation.java index 177b437b5..fe84ec121 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLocation.java +++ b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLocation.java @@ -106,6 +106,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_movingAccurate() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); // when @@ -179,6 +180,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_slowMovingAccurate() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); // when @@ -224,6 +226,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_idle() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); // when @@ -269,6 +272,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_idle_withMovement() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); service.getTrackPointCreator().stopGPS(); @@ -329,6 +333,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_idle_withSensorData() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); service.getTrackPointCreator().setRemoteSensorManager(new BluetoothRemoteSensorManager(context) { @@ -403,6 +408,7 @@ public class TrackRecordingServiceTestLocation { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setRemoteSensorManager(remoteSensorManager); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); altitudeSumManager.stop(service); @@ -474,6 +480,7 @@ public class TrackRecordingServiceTestLocation { public void testOnLocationChangedAsync_segment() { // given Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); // when diff --git a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLooper.java b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLooper.java index 008fb48b7..5cac4f870 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLooper.java +++ b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestLooper.java @@ -1,5 +1,10 @@ package de.dennisguse.opentracks.services; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertTrue; + import android.content.ContentProvider; import android.content.Context; import android.content.SharedPreferences; @@ -31,13 +36,8 @@ import de.dennisguse.opentracks.content.data.Track; import de.dennisguse.opentracks.content.data.TrackPoint; import de.dennisguse.opentracks.content.provider.ContentProviderUtils; import de.dennisguse.opentracks.content.provider.CustomContentProvider; -import de.dennisguse.opentracks.stats.TrackStatistics; import de.dennisguse.opentracks.settings.PreferencesUtils; - -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertTrue; +import de.dennisguse.opentracks.stats.TrackStatistics; /** * Tests for the track recording service, which require a {@link Looper}. @@ -195,6 +195,7 @@ public class TrackRecordingServiceTestLooper { // Start a track. Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); assertNotNull(trackId); assertTrue(service.isRecording()); Track track = contentProviderUtils.getTrack(trackId); diff --git a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestStatistics.java b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestStatistics.java index b85564f76..2326f7c37 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestStatistics.java +++ b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceTestStatistics.java @@ -118,6 +118,7 @@ public class TrackRecordingServiceTestStatistics { // given service.getTrackPointCreator().setClock(Clock.fixed(Instant.ofEpochMilli(0), ZoneId.systemDefault())); Track.Id trackId = service.startNewTrack(); + service.stopUpdateRecordingData(); service.getTrackPointCreator().setAltitudeSumManager(altitudeSumManager); Function assertMovingTime = expected -> {