Fixed problem with last action list deserialization when data returned

from the game server doesn't contain any valid action.

Author: ocraft

ocraft-s2client-protocol/src/main/java/com/github/ocraft/s2client/protocol/response/ResponseObservation.java

@@ -37,6 +37,7 @@
 
 import java.util.Collections;
 import java.util.List;
+import java.util.function.Predicate;
 
 import static com.github.ocraft.s2client.protocol.DataExtractor.tryGet;
 import static com.github.ocraft.s2client.protocol.Errors.required;
@@ -58,7 +59,7 @@ private ResponseObservation(Sc2Api.ResponseObservation sc2ApiResponseObservation
         super(ResponseType.OBSERVATION, GameStatus.from(status));
 
         this.actions = sc2ApiResponseObservation.getActionsList().stream()
-                .filter(a -> a.getSerializedSize() > 0).map(Action::from)
+                .filter(actionIsValid()).map(Action::from)
                 .collect(collectingAndThen(toList(), Collections::unmodifiableList));
 
         this.actionErrors = sc2ApiResponseObservation.getActionErrorsList().stream()
@@ -74,6 +75,15 @@ private ResponseObservation(Sc2Api.ResponseObservation sc2ApiResponseObservation
                 .collect(collectingAndThen(toList(), Collections::unmodifiableList));
     }
 
+    private Predicate<Sc2Api.Action> actionIsValid() {
+        return a -> a.getSerializedSize() > 0 && (
+                a.hasActionRaw() ||
+                        a.hasActionChat() ||
+                        a.hasActionFeatureLayer() ||
+                        a.hasActionRender() ||
+                        a.hasActionUi());
+    }
+
     public static ResponseObservation from(Sc2Api.Response sc2ApiResponse) {
         if (!hasObservationResponse(sc2ApiResponse)) {
             throw new IllegalArgumentException("provided argument doesn't have observation response");