feat: update error handling to log more consistently (#39)

Author: bigboxer23

pom.xml

@@ -4,7 +4,7 @@
 
   <groupId>com.bigboxer23</groupId>
   <artifactId>switchbotapi-java</artifactId>
-  <version>1.1.4</version>
+  <version>1.1.5</version>
 
   <name>switchbotapi-java</name>
   <url>https://github.com/bigboxer23/switchbotapi-java</url>

src/main/java/com/bigboxer23/switch_bot/SwitchBotApi.java

@@ -11,6 +11,7 @@
 import javax.crypto.Mac;
 import javax.crypto.spec.SecretKeySpec;
 import lombok.Getter;
+import okhttp3.Response;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -78,17 +79,17 @@ protected RequestBuilderCallback addAuth() {
 	 * @param apiResponse the API response
 	 * @return true if error occurs
 	 */
-	protected boolean checkForError(IApiResponse apiResponse) {
-		return Optional.ofNullable(apiResponse)
-				.map(response -> {
-					if (response.getStatusCode() != 100) {
-						logger.error("error code: " + response.getStatusCode() + " : " + response.getMessage());
+	protected boolean checkForError(Response response, Optional<IApiResponse> apiResponse) {
+		return apiResponse
+				.map(api -> {
+					if (api.getStatusCode() != 100) {
+						logger.error("error code: " + api.getStatusCode() + " : " + api.getMessage());
 						return false;
 					}
 					return true;
 				})
 				.orElseGet(() -> {
-					logger.error("null api response");
+					logger.error("Error calling switchbot api: " + response.code() + " " + response.message());
 					return false;
 				});
 	}

src/main/java/com/bigboxer23/switch_bot/SwitchBotDeviceApi.java

@@ -7,7 +7,6 @@
 import java.nio.charset.StandardCharsets;
 import java.util.List;
 import java.util.Optional;
-
 import okhttp3.RequestBody;
 import okhttp3.Response;
 import org.slf4j.Logger;
@@ -78,7 +77,7 @@ public void sendDeviceControlCommands(String deviceId, DeviceCommand command) th
 	 */
 	private <T extends IApiResponse> T parseResponse(Response response, Class<T> clazz) throws IOException {
 		Optional<T> apiResponse = OkHttpUtil.getBody(response, clazz);
-		if (apiResponse.isEmpty() || !provider.checkForError(apiResponse.get())) {
+		if (!provider.checkForError(response, (Optional<IApiResponse>) apiResponse)) {
 			throw new IOException(response.code() + " " + response.message());
 		}
 		return apiResponse.get();