TrackStatistics startTime/stopTime is null on creation.

src/androidTest/java/de/dennisguse/opentracks/io/file/importer/ExportImportTest.java

@@ -47,7 +47,6 @@ import static org.junit.Assert.assertNull;
 /**
  * Export a track to {@link TrackFileFormat} and verify that the import is identical.
  * <p>
- * TODO: test ignores {@link TrackStatistics} for now.
  */
 @RunWith(JUnit4.class)
 public class ExportImportTest {
@@ -431,7 +430,7 @@ public class ExportImportTest {
     }
 
     private static TrackPoint createTrackPoint(long time, double latitude, double longitude, float accuracy, long speed, long altitude, float elevationGain, float heartRate, float cyclingCadence, float power) {
-        Location location = new Location("gps");
+        Location location = new Location("");
         location.setTime(time);
         location.setLongitude(longitude);
         location.setLatitude(latitude);

src/main/java/de/dennisguse/opentracks/content/provider/ContentProviderUtils.java

@@ -264,7 +264,6 @@ public class ContentProviderUtils {
         ContentValues values = new ContentValues();
         TrackStatistics trackStatistics = track.getTrackStatistics();
 
-        // Value < 0 indicates no id is available
         if (track.getId() != null) {
             values.put(TracksColumns._ID, track.getId().getId());
         }
@@ -272,8 +271,12 @@ public class ContentProviderUtils {
         values.put(TracksColumns.NAME, track.getName());
         values.put(TracksColumns.DESCRIPTION, track.getDescription());
         values.put(TracksColumns.CATEGORY, track.getCategory());
-        values.put(TracksColumns.STARTTIME, trackStatistics.getStartTime().toEpochMilli());
-        values.put(TracksColumns.STOPTIME, trackStatistics.getStopTime().toEpochMilli());
+        if (trackStatistics.getStartTime() != null) {
+            values.put(TracksColumns.STARTTIME, trackStatistics.getStartTime().toEpochMilli());
+        }
+        if (trackStatistics.getStopTime() != null) {
+            values.put(TracksColumns.STOPTIME, trackStatistics.getStopTime().toEpochMilli());
+        }
         values.put(TracksColumns.TOTALDISTANCE, trackStatistics.getTotalDistance());
         values.put(TracksColumns.TOTALTIME, trackStatistics.getTotalTime().toMillis());
         values.put(TracksColumns.MOVINGTIME, trackStatistics.getMovingTime().toMillis());

src/main/java/de/dennisguse/opentracks/content/provider/CustomContentProvider.java

@@ -341,10 +341,6 @@ public class CustomContentProvider extends ContentProvider {
     }
 
     private Uri insertTrack(Uri url, ContentValues contentValues) {
-        boolean hasStartTime = contentValues.containsKey(TracksColumns.STARTTIME);
-        if (!hasStartTime) {
-            throw new IllegalArgumentException("Both start time and start id values are required.");
-        }
         long rowId = db.insert(TracksColumns.TABLE_NAME, TracksColumns._ID, contentValues);
         if (rowId >= 0) {
             return ContentUris.appendId(TracksColumns.CONTENT_URI.buildUpon(), rowId).build();

src/main/java/de/dennisguse/opentracks/stats/TrackStatistics.java

@@ -112,7 +112,8 @@ public class TrackStatistics {
     }
 
     public void reset() {
-        setStartTime(Instant.ofEpochMilli(0)); //TODO should probably initialized with null
+        startTime = null;
+        stopTime = null;
 
         setTotalDistance(0);
         setTotalTime(Duration.ofSeconds(0));