Some cleanup on unused methods

Author: breskeby

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/test/rest/CopyRestTestsTask.java

@@ -56,7 +56,7 @@ public abstract class CopyRestTestsTask extends DefaultTask {
     private FileCollection coreConfig;
     private FileCollection xpackConfig;
     private FileCollection additionalConfig;
-    private SerializableFunction<FileCollection, FileTree> coreConfigToFileTree = FileCollection::getAsFileTree;
+    private SerializableCopyRestTestsTaskFunction<FileCollection, FileTree> coreConfigToFileTree = FileCollection::getAsFileTree;
     private SerializableFunction<FileCollection, FileTree> xpackConfigToFileTree = FileCollection::getAsFileTree;
     private SerializableFunction<FileCollection, FileTree> additionalConfigToFileTree = FileCollection::getAsFileTree;
 
@@ -65,9 +65,6 @@ public abstract class CopyRestTestsTask extends DefaultTask {
     private final ProjectLayout projectLayout;
     private final FileSystemOperations fileSystemOperations;
 
-    @Inject
-    public abstract FileOperations getFileOperations();
-
     @Inject
     public CopyRestTestsTask(
         ProjectLayout projectLayout,
@@ -85,16 +82,7 @@ public CopyRestTestsTask(
     }
 
     @Inject
-    protected abstract ProjectLayout getProjectLayout();
-
-    @Inject
-    protected abstract FileSystemOperations getFileSystemOperations();
-
-    @Inject
-    protected abstract ObjectFactory getObjectFactory();
-
-    @Inject
-    protected abstract PatternSetFactory getPatternSetFactory();
+    public abstract FileOperations getFileOperations();
 
     @Input
     public ListProperty<String> getIncludeCore() {

build-tools/src/main/java/org/elasticsearch/gradle/test/SystemPropertyCommandLineArgumentProvider.java

@@ -8,7 +8,6 @@
  */
 package org.elasticsearch.gradle.test;
 
-import org.gradle.api.provider.Provider;
 import org.gradle.api.tasks.Input;
 import org.gradle.process.CommandLineArgumentProvider;