Fix: acceleration history task should be optional

The category of this task was broken by an earlier change.

Author: misolt

dumper/app/src/main/java/com/google/edwmigration/dumper/application/dumper/connector/snowflake/SnowflakeLogsConnector.java

@@ -430,7 +430,8 @@ enum TimeSeriesView {
         QueryAccelerationHistoryFormat.Header.class,
         QueryAccelerationHistoryFormat.ZIP_ENTRY_PREFIX,
         TimeSeriesColumn.END_TIME,
-        SnowflakeLogsConnectorProperty.QUERY_ACCELERATION_HISTORY_OVERRIDE_QUERY),
+        SnowflakeLogsConnectorProperty.QUERY_ACCELERATION_HISTORY_OVERRIDE_QUERY,
+        TaskCategory.OPTIONAL),
     REPLICATION_GROUP_USAGE_HISTORY(
         ReplicationGroupUsageHistoryFormat.Header.class,
         QueryAccelerationHistoryFormat.ZIP_ENTRY_PREFIX,
@@ -461,20 +462,31 @@ enum TimeSeriesView {
     final ConnectorProperty property;
     final String queryPrefix;
     final String zipPrefix;
+    final TaskCategory category;
     final TimeSeriesColumn column;
 
     TimeSeriesView(
         Class<? extends Enum<?>> headerClass,
         String zipPrefix,
         TimeSeriesColumn column,
-        SnowflakeLogsConnectorProperty property) {
+        SnowflakeLogsConnectorProperty property,
+        TaskCategory category) {
       this.headerClass = headerClass;
       this.property = property;
       this.queryPrefix = formatPrefix(headerClass, name());
       this.zipPrefix = zipPrefix;
+      this.category = category;
       this.column = column;
     }
 
+    TimeSeriesView(
+        Class<? extends Enum<?>> headerClass,
+        String zipPrefix,
+        TimeSeriesColumn column,
+        SnowflakeLogsConnectorProperty property) {
+      this(headerClass, zipPrefix, column, property, TaskCategory.REQUIRED);
+    }
+
     static final ImmutableList<TimeSeriesView> valuesInOrder =
         ImmutableList.of(
             WAREHOUSE_EVENTS_HISTORY,

dumper/app/src/test/java/com/google/edwmigration/dumper/application/dumper/connector/snowflake/SnowflakeLogsConnectorTest.java

@@ -26,11 +26,13 @@
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assume.assumeFalse;
 
 import com.google.common.collect.ImmutableList;
 import com.google.edwmigration.dumper.application.dumper.ConnectorArguments;
 import com.google.edwmigration.dumper.application.dumper.MetadataDumperUsageException;
 import com.google.edwmigration.dumper.application.dumper.connector.snowflake.SnowflakeLogsConnector.TimeSeriesView;
+import com.google.edwmigration.dumper.application.dumper.task.TaskCategory;
 import com.google.edwmigration.dumper.test.TestUtils;
 import java.io.File;
 import org.junit.Test;
@@ -111,6 +113,20 @@ public void addOverridesToQuery_whereOverride_success() {
     assertTrue(result, result.contains(override));
   }
 
+  @Test
+  public void category_accelerationHistory_isOptional() {
+    TimeSeriesView view = TimeSeriesView.QUERY_ACCELERATION_HISTORY;
+
+    assertEquals(view.category, TaskCategory.OPTIONAL);
+  }
+
+  @Theory
+  public void category_notAccelerationHistory_isRequired(TimeSeriesView view) {
+    assumeFalse(view.equals(TimeSeriesView.QUERY_ACCELERATION_HISTORY));
+
+    assertEquals(view.category, TaskCategory.REQUIRED);
+  }
+
   @Test
   public void earliestTimestamp_notProvided_emptyResult() {
     ConnectorArguments arguments =