fixup!

Author: jfreden

server/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/restore/RestoreSnapshotRequest.java

@@ -132,7 +132,6 @@ public ActionRequestValidationException validate() {
         if (indicesOptions == null) {
             validationException = addValidationError("indicesOptions is missing", validationException);
         }
-        // TODO should additional validation be done here to make sure views are not attempted to be restored?
         // This action does not use the IndexNameExpressionResolver to resolve concrete indices, this is why we check here for selectors
         if (indicesOptions.allowSelectors() == false) {
             for (String index : indices) {

server/src/test/java/org/elasticsearch/action/admin/cluster/snapshots/restore/RestoreSnapshotRequestTests.java

@@ -30,8 +30,6 @@
 import java.util.Map;
 
 import static org.hamcrest.Matchers.containsString;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
 
 public class RestoreSnapshotRequestTests extends AbstractWireSerializingTestCase<RestoreSnapshotRequest> {
     private RestoreSnapshotRequest randomState(RestoreSnapshotRequest instance) {
@@ -90,7 +88,7 @@ private RestoreSnapshotRequest randomState(RestoreSnapshotRequest instance) {
                             randomBoolean(),
                             instance.indicesOptions().ignoreAliases() == false,
                             randomBoolean(),
-                            false // Restoring views is not supported
+                            false // Specifying views in the restore snapshot request is not supported
                         )
                     )
                     .gatekeeperOptions(IndicesOptions.GatekeeperOptions.builder().allowSelectors(false).includeFailureIndices(true).build())