Removing deprecated FileTreeWatcher (#6688)

eclipse-che · Oct 12, 2017 · c35d27e · c35d27e
1 parent 757dd97
commit c35d27e
Show file tree

Hide file tree

Showing 14 changed files with 14 additions and 1,209 deletions.
diff --git a/...debugger-server/src/test/java/org/eclipse/che/plugin/jdb/server/util/ProjectApiUtils.java b/...debugger-server/src/test/java/org/eclipse/che/plugin/jdb/server/util/ProjectApiUtils.java
@@ -31,7 +31,6 @@
 import org.eclipse.che.api.project.server.importer.ProjectImporterRegistry;
 import org.eclipse.che.api.project.server.type.ProjectTypeRegistry;
 import org.eclipse.che.api.vfs.impl.file.DefaultFileWatcherNotificationHandler;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.LocalVirtualFileSystemProvider;
 import org.eclipse.che.api.vfs.search.impl.FSLuceneSearcherProvider;
@@ -91,8 +90,6 @@ private static void init() throws Exception {
     ProjectImporterRegistry importerRegistry = new ProjectImporterRegistry(new HashSet<>());
     FileWatcherNotificationHandler fileWatcherNotificationHandler =
         new DefaultFileWatcherNotificationHandler(vfsProvider);
-    FileTreeWatcher fileTreeWatcher =
-        new FileTreeWatcher(root, new HashSet<>(), fileWatcherNotificationHandler);
     ProjectManager projectManager =
         new ProjectManager(
             vfsProvider,
@@ -102,7 +99,6 @@ private static void init() throws Exception {
             projectHandlerRegistry,
             importerRegistry,
             fileWatcherNotificationHandler,
-            fileTreeWatcher,
             workspaceHolder,
             mock(FileWatcherManager.class));
 
@@ -121,6 +117,7 @@ private static void init() throws Exception {
   }
 
   private static class TestWorkspaceHolder extends WorkspaceProjectsSyncer {
+
     private List<ProjectConfigDto> projects;
 
     TestWorkspaceHolder(List<ProjectConfigDto> projects) {

diff --git a/...erver/src/test/java/org/eclipse/che/plugin/java/server/rest/JavaFormatterServiceTest.java b/...erver/src/test/java/org/eclipse/che/plugin/java/server/rest/JavaFormatterServiceTest.java
@@ -39,7 +39,6 @@
 import org.eclipse.che.api.project.server.importer.ProjectImporterRegistry;
 import org.eclipse.che.api.project.server.type.ProjectTypeRegistry;
 import org.eclipse.che.api.vfs.impl.file.DefaultFileWatcherNotificationHandler;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.LocalVirtualFileSystemProvider;
 import org.eclipse.che.api.vfs.search.impl.FSLuceneSearcherProvider;
@@ -60,6 +59,7 @@
 /** Test for the java formatter service. */
 @Listeners(value = {MockitoTestNGListener.class})
 public class JavaFormatterServiceTest {
+
   private static final String FORMATTER_CONTENT =
       "<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"no\"?>\n"
           + "<profiles version=\"13\">\n"
@@ -109,8 +109,6 @@ protected void initProjectApi() throws Exception {
     ProjectImporterRegistry importerRegistry = new ProjectImporterRegistry(new HashSet<>());
     FileWatcherNotificationHandler fileWatcherNotificationHandler =
         new DefaultFileWatcherNotificationHandler(vfsProvider);
-    FileTreeWatcher fileTreeWatcher =
-        new FileTreeWatcher(root, new HashSet<>(), fileWatcherNotificationHandler);
     projectManager =
         new ProjectManager(
             vfsProvider,
@@ -120,7 +118,6 @@ protected void initProjectApi() throws Exception {
             projectHandlerRegistry,
             importerRegistry,
             fileWatcherNotificationHandler,
-            fileTreeWatcher,
             workspaceHolder,
             mock(FileWatcherManager.class));
 
@@ -243,6 +240,7 @@ private void checkProjectFormatterFile() throws IOException {
   }
 
   private static class TestWorkspaceHolder extends WorkspaceProjectsSyncer {
+
     private List<ProjectConfigDto> projects;
 
     TestWorkspaceHolder() {

diff --git a/...in-java-plain-server/src/test/java/org/eclipse/che/plugin/java/plain/server/BaseTest.java b/...in-java-plain-server/src/test/java/org/eclipse/che/plugin/java/plain/server/BaseTest.java
@@ -34,7 +34,6 @@
 import org.eclipse.che.api.project.server.importer.ProjectImporterRegistry;
 import org.eclipse.che.api.project.server.type.ProjectTypeRegistry;
 import org.eclipse.che.api.vfs.impl.file.DefaultFileWatcherNotificationHandler;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.LocalVirtualFileSystemProvider;
 import org.eclipse.che.api.vfs.search.impl.FSLuceneSearcherProvider;
@@ -55,6 +54,7 @@
 
 /** @author Valeriy Svydenko */
 public abstract class BaseTest {
+
   private static final String wsPath = BaseTest.class.getResource("/projects").getFile();
   private static final String INDEX_PATH = "target/fs_index";
 
@@ -70,7 +70,9 @@ protected void initProjectApi() throws Exception {
 
     TestWorkspaceHolder workspaceHolder = new TestWorkspaceHolder();
 
-    if (root == null) root = new File(wsPath);
+    if (root == null) {
+      root = new File(wsPath);
+    }
 
     if (root.exists()) {
       IoUtil.deleteRecursive(root);
@@ -109,9 +111,6 @@ protected void initProjectApi() throws Exception {
     ProjectImporterRegistry importerRegistry = new ProjectImporterRegistry(new HashSet<>());
     FileWatcherNotificationHandler fileWatcherNotificationHandler =
         new DefaultFileWatcherNotificationHandler(vfsProvider);
-    FileTreeWatcher fileTreeWatcher =
-        new FileTreeWatcher(root, new HashSet<>(), fileWatcherNotificationHandler);
-
     projectManager =
         new ProjectManager(
             vfsProvider,
@@ -121,7 +120,6 @@ protected void initProjectApi() throws Exception {
             projectHandlerRegistry,
             importerRegistry,
             fileWatcherNotificationHandler,
-            fileTreeWatcher,
             new TestWorkspaceHolder(new ArrayList<>()),
             mock(FileWatcherManager.class));
 

diff --git a/...n/che-plugin-maven-server/src/test/java/org/eclipse/che/plugin/maven/server/BaseTest.java b/...n/che-plugin-maven-server/src/test/java/org/eclipse/che/plugin/maven/server/BaseTest.java
@@ -40,7 +40,6 @@
 import org.eclipse.che.api.project.server.type.ProjectTypeDef;
 import org.eclipse.che.api.project.server.type.ProjectTypeRegistry;
 import org.eclipse.che.api.vfs.impl.file.DefaultFileWatcherNotificationHandler;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.LocalVirtualFileSystemProvider;
 import org.eclipse.che.api.vfs.search.impl.FSLuceneSearcherProvider;
@@ -83,7 +82,6 @@ public abstract class BaseTest {
   protected LocalVirtualFileSystemProvider vfsProvider;
   protected ProjectRegistry projectRegistry;
   protected FileWatcherNotificationHandler fileWatcherNotificationHandler;
-  protected FileTreeWatcher fileTreeWatcher;
   protected ProjectTypeRegistry projectTypeRegistry;
   protected ProjectHandlerRegistry projectHandlerRegistry;
   protected ProjectImporterRegistry importerRegistry;
@@ -111,7 +109,9 @@ protected void initProjectApi() throws Exception {
     mavenServerManager = new MavenServerManager(mavenServerPath);
     workspaceHolder = new TestWorkspaceHolder();
 
-    if (root == null) root = new File(wsPath);
+    if (root == null) {
+      root = new File(wsPath);
+    }
 
     if (root.exists()) {
       IoUtil.deleteRecursive(root);
@@ -149,7 +149,6 @@ protected void initProjectApi() throws Exception {
     importerRegistry = new ProjectImporterRegistry(new HashSet<>());
 
     fileWatcherNotificationHandler = new DefaultFileWatcherNotificationHandler(vfsProvider);
-    fileTreeWatcher = new FileTreeWatcher(root, new HashSet<>(), fileWatcherNotificationHandler);
 
     pm =
         new ProjectManager(
@@ -160,7 +159,6 @@ protected void initProjectApi() throws Exception {
             projectHandlerRegistry,
             importerRegistry,
             fileWatcherNotificationHandler,
-            fileTreeWatcher,
             new TestWorkspaceHolder(new ArrayList<>()),
             mock(FileWatcherManager.class));
 

diff --git a/...ing-testng-server/src/test/java/org/ecipse/che/plugin/testing/testng/server/BaseTest.java b/...ing-testng-server/src/test/java/org/ecipse/che/plugin/testing/testng/server/BaseTest.java
@@ -32,7 +32,6 @@
 import org.eclipse.che.api.project.server.type.ProjectTypeDef;
 import org.eclipse.che.api.project.server.type.ProjectTypeRegistry;
 import org.eclipse.che.api.vfs.impl.file.DefaultFileWatcherNotificationHandler;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.LocalVirtualFileSystemProvider;
 import org.eclipse.che.api.vfs.search.impl.FSLuceneSearcherProvider;
@@ -69,7 +68,6 @@ public abstract class BaseTest {
   protected LocalVirtualFileSystemProvider vfsProvider;
   protected ProjectRegistry projectRegistry;
   protected FileWatcherNotificationHandler fileWatcherNotificationHandler;
-  protected FileTreeWatcher fileTreeWatcher;
   protected ProjectTypeRegistry projectTypeRegistry;
   protected ProjectHandlerRegistry projectHandlerRegistry;
   protected ProjectImporterRegistry importerRegistry;
@@ -131,7 +129,6 @@ protected void initProjectApi() throws Exception {
     importerRegistry = new ProjectImporterRegistry(new HashSet<>());
 
     fileWatcherNotificationHandler = new DefaultFileWatcherNotificationHandler(vfsProvider);
-    fileTreeWatcher = new FileTreeWatcher(root, new HashSet<>(), fileWatcherNotificationHandler);
 
     pm =
         new ProjectManager(
@@ -142,7 +139,6 @@ protected void initProjectApi() throws Exception {
             projectHandlerRegistry,
             importerRegistry,
             fileWatcherNotificationHandler,
-            fileTreeWatcher,
             new TestWorkspaceHolder(new ArrayList<>()),
             Mockito.mock(FileWatcherManager.class));
 

diff --git a/...che-core-api-project/src/main/java/org/eclipse/che/api/project/server/ProjectManager.java b/...che-core-api-project/src/main/java/org/eclipse/che/api/project/server/ProjectManager.java
@@ -55,7 +55,6 @@
 import org.eclipse.che.api.vfs.VirtualFile;
 import org.eclipse.che.api.vfs.VirtualFileSystem;
 import org.eclipse.che.api.vfs.VirtualFileSystemProvider;
-import org.eclipse.che.api.vfs.impl.file.FileTreeWatcher;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationHandler;
 import org.eclipse.che.api.vfs.impl.file.FileWatcherNotificationListener;
 import org.eclipse.che.api.vfs.search.Searcher;
@@ -81,7 +80,6 @@ public class ProjectManager {
   private final ProjectRegistry projectRegistry;
   private final ProjectHandlerRegistry handlers;
   private final ProjectImporterRegistry importers;
-  private final FileTreeWatcher fileWatcher;
   private final FileWatcherNotificationHandler fileWatchNotifier;
   private final ExecutorService executor;
   private final WorkspaceProjectsSyncer workspaceProjectsHolder;
@@ -98,7 +96,6 @@ public ProjectManager(
       ProjectHandlerRegistry handlers,
       ProjectImporterRegistry importers,
       FileWatcherNotificationHandler fileWatcherNotificationHandler,
-      FileTreeWatcher fileTreeWatcher,
       WorkspaceProjectsSyncer workspaceProjectsHolder,
       FileWatcherManager fileWatcherManager)
       throws ServerException {
@@ -109,7 +106,6 @@ public ProjectManager(
     this.handlers = handlers;
     this.importers = importers;
     this.fileWatchNotifier = fileWatcherNotificationHandler;
-    this.fileWatcher = fileTreeWatcher;
     this.workspaceProjectsHolder = workspaceProjectsHolder;
     this.fileWatcherManager = fileWatcherManager;
 
@@ -164,12 +160,6 @@ public void onFileWatcherEvent(VirtualFile virtualFile, FileWatcherEventType eve
           }
         };
     fileWatchNotifier.addNotificationListener(defaultListener);
-    try {
-      fileWatcher.startup();
-    } catch (IOException e) {
-      LOG.error(e.getMessage(), e);
-      fileWatchNotifier.removeNotificationListener(defaultListener);
-    }
   }
 
   @PreDestroy
@@ -198,13 +188,9 @@ public void removeWatchListener(FileWatcherNotificationListener listener) {
     fileWatchNotifier.removeNotificationListener(listener);
   }
 
-  public void addWatchExcludeMatcher(PathMatcher matcher) {
-    fileWatcher.addExcludeMatcher(matcher);
-  }
+  public void addWatchExcludeMatcher(PathMatcher matcher) {}
 
-  public void removeWatchExcludeMatcher(PathMatcher matcher) {
-    fileWatcher.removeExcludeMatcher(matcher);
-  }
+  public void removeWatchExcludeMatcher(PathMatcher matcher) {}
 
   /**
    * @return all the projects

diff --git a/...rc/main/java/org/eclipse/che/api/vfs/impl/file/DefaultFileWatcherNotificationHandler.java b/...rc/main/java/org/eclipse/che/api/vfs/impl/file/DefaultFileWatcherNotificationHandler.java
@@ -27,6 +27,7 @@
 
 @Singleton
 public class DefaultFileWatcherNotificationHandler implements FileWatcherNotificationHandler {
+
   private static final Logger LOG =
       LoggerFactory.getLogger(DefaultFileWatcherNotificationHandler.class);
 
@@ -93,6 +94,7 @@ private VirtualFile convertToVirtualFile(File root, String subPath, boolean isDi
   }
 
   private static class DeletedLocalVirtualFile extends LocalVirtualFile {
+
     private final boolean isDir;
 
     DeletedLocalVirtualFile(