Switched data type of EPOCH timestamp method parameters from String to long in TheTVDBApi interface.

Author: m0nk3y2k4

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Included _org.jacoco:jacoco-maven-plugin_ `0.8.5` in order to generate a test coverage report at the end of the build process.
 
 ### Changed
+- Changed the data types of `TheTVDBApi.queryLastUpdated()` shortcut methods parameters from _String_ to _long_.
 - The remote API version used by this connector is now publicly available via the `TheTVDBApi.Version.API_VERSION` constant.
 - Specific handling of HTTP-405 responses. Enhance error message with additional information.
 - All code now has to have a JUnit test line coverage of at least 80% and no missed class counts.

src/main/java/com/github/m0nk3y2k4/thetvdb/api/TheTVDBApi.java

@@ -750,7 +750,7 @@ private Version() {}
      * of the returned map represent a <i>TheTVDB.com</i> series ID (key) and when it was updated the last time (value) as Epoch time. This is a shortcut-method for
      * {@link #queryLastUpdated(QueryParameters) queryLastUpdated(queryParameters)} with a single "fromTime" query parameter.
      *
-     * @see #queryLastUpdated(String, String) queryLastUpdated(fromTime, toTime)
+     * @see #queryLastUpdated(long, long) queryLastUpdated(fromTime, toTime)
      *
      * @param fromTime Epoch time to start your date range
      *
@@ -759,7 +759,7 @@ private Version() {}
      * @throws APIException If an exception with the remote API occurs, e.g. authentication failure, IO error, resource not found, etc. or
      *                      no records exist for the given timespan.
      */
-    Map<Long, Long> queryLastUpdated(@Nonnull String fromTime) throws APIException;
+    Map<Long, Long> queryLastUpdated(long fromTime) throws APIException;
 
     /**
      * Returns a map of series that have changed in between the given timeframe, but with a maximum of one week, starting at the provided <em>{@code fromTime}</em>
@@ -768,7 +768,7 @@ private Version() {}
      * time (value) as Epoch time. This is a shortcut-method for {@link #queryLastUpdated(QueryParameters) queryLastUpdated(queryParameters)} with a "fromTime"
      * and a "toTime" query parameter.
      *
-     * @see #queryLastUpdated(String) queryLastUpdated(fromTime)
+     * @see #queryLastUpdated(long) queryLastUpdated(fromTime)
      *
      * @param fromTime Epoch time to start your date range
      * @param toTime Epoch time to end your date range. Must not be greater than one week from <em>{@code fromTime}</em>.
@@ -778,7 +778,7 @@ private Version() {}
      * @throws APIException If an exception with the remote API occurs, e.g. authentication failure, IO error, resource not found, etc. or
      *                      no records exist for the given timespan.
      */
-    Map<Long, Long> queryLastUpdated(@Nonnull String fromTime, @Nonnull String toTime) throws APIException;
+    Map<Long, Long> queryLastUpdated(long fromTime, long toTime) throws APIException;
 
     /**
      * Returns a list of valid parameters for querying series which have been updated lately, as plain Strings. These keys are permitted to be used in

src/main/java/com/github/m0nk3y2k4/thetvdb/internal/api/impl/TheTVDBApiImpl.java

@@ -335,15 +335,13 @@ public Map<Long, Long> queryLastUpdated(QueryParameters queryParameters) throws
     }
 
     @Override
-    public Map<Long, Long> queryLastUpdated(@Nonnull String fromTime) throws APIException {
-        validateNotEmpty(fromTime);
-        return queryLastUpdated(query(Map.of(Query.Updates.FROMTIME, fromTime)));
+    public Map<Long, Long> queryLastUpdated(long fromTime) throws APIException {
+        return queryLastUpdated(query(Map.of(Query.Updates.FROMTIME, String.valueOf(fromTime))));
     }
 
     @Override
-    public Map<Long, Long> queryLastUpdated(@Nonnull String fromTime, @Nonnull String toTime) throws APIException {
-        validateNotEmpty(fromTime, toTime);
-        return queryLastUpdated(query(Map.of(Query.Updates.FROMTIME, fromTime, Query.Updates.TOTIME, toTime)));
+    public Map<Long, Long> queryLastUpdated(long fromTime, long toTime) throws APIException {
+        return queryLastUpdated(query(Map.of(Query.Updates.FROMTIME, String.valueOf(fromTime), Query.Updates.TOTIME, String.valueOf(toTime))));
     }
 
     @Override

src/test/java/com/github/m0nk3y2k4/thetvdb/internal/api/impl/TheTVDBApiImplTest.java

@@ -180,10 +180,6 @@ private Stream<Arguments> withInvalidParameters() {
                     of(route(() -> basicAPI.queryImagesByResolution(3645, "  "), "queryImagesByResolution() with empty parameter")),
                     of(route(() -> basicAPI.queryImagesBySubKey(6311, null), "queryImagesBySubKey() with NULL parameter")),
                     of(route(() -> basicAPI.queryImagesBySubKey(9447, "   "), "queryImagesBySubKey() with empty parameter")),
-                    of(route(() -> basicAPI.queryLastUpdated((String) null), "queryLastUpdated() with NULL from time parameter")),
-                    of(route(() -> basicAPI.queryLastUpdated(" "), "queryLastUpdated() with empty from time parameter")),
-                    of(route(() -> basicAPI.queryLastUpdated(null, null), "queryLastUpdated() with NULL from and to time parameter")),
-                    of(route(() -> basicAPI.queryLastUpdated("  ", ""), "queryLastUpdated() with empty from and to time parameter")),
                     of(route(() -> basicAPI.getUser(), "getUser() without authentication")),
                     of(route(() -> basicAPI.getFavorites(), "getFavorites() without authentication")),
                     of(route(() -> basicAPI.deleteFromFavorites(4874), "deleteFromFavorites() without authentication")),
@@ -245,8 +241,8 @@ private Stream<Arguments> withValidParameters() {
                     of(route(() -> basicAPI.queryImagesBySubKey(34964, "Key3"), "queryImagesBySubKey()"), IMAGES),
                     of(route(() -> basicAPI.getAvailableImageQueryParameters(88861), "getAvailableImageQueryParameters()"), IMAGEQUERYPARAMETERS),
                     of(route(() -> basicAPI.queryLastUpdated(params(FROMTIME, "7")), "queryLastUpdated() with query parameters"), UPDATES),
-                    of(route(() -> basicAPI.queryLastUpdated("54"), "queryLastUpdated() with from time"), UPDATES),
-                    of(route(() -> basicAPI.queryLastUpdated("76", "99"), "queryLastUpdated() with from and to time"), UPDATES),
+                    of(route(() -> basicAPI.queryLastUpdated(54), "queryLastUpdated() with from time"), UPDATES),
+                    of(route(() -> basicAPI.queryLastUpdated(76, 99), "queryLastUpdated() with from and to time"), UPDATES),
                     of(route(() -> basicAPI.getAvailableLastUpdatedQueryParameters(), "getAvailableLastUpdatedQueryParameters()"), QUERYPARAMETERS),
                     of(route(() -> userAuthApi.getUser(), "getUser()"), USER),
                     of(route(() -> userAuthApi.getFavorites(), "getFavorites()"), FAVORITES),