Do not throw RuntimeException from IFile.getContents(false) #2290

Author: basilevs

resources/bundles/org.eclipse.core.resources/src/org/eclipse/core/internal/localstore/FileSystemResourceManager.java

@@ -935,6 +935,10 @@ private IFileStore getFileStore(IFile target, boolean force) throws ResourceExce
 			final IFileInfo fileInfo = store.fetchInfo();
 			Resource resource = (Resource) target;
 			ResourceInfo info = resource.getResourceInfo(true, false);
+			if (info == null) {
+				String message = NLS.bind(Messages.resources_mustExist, target.getFullPath());
+				throw new ResourceException(IResourceStatus.RESOURCE_NOT_FOUND, target.getFullPath(), message, null);
+			}
 			if (fileInfo.getLastModified() != info.getLocalSyncInfo()) {
 				asyncRefresh(target);
 				String message = NLS.bind(Messages.localstore_resourceIsOutOfSync, target.getFullPath());

resources/tests/org.eclipse.core.tests.resources/src/org/eclipse/core/tests/resources/regression/IFileTest.java

@@ -13,6 +13,7 @@
  *******************************************************************************/
 package org.eclipse.core.tests.resources.regression;
 
+import static java.lang.System.currentTimeMillis;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.eclipse.core.resources.ResourcesPlugin.getWorkspace;
 import static org.eclipse.core.tests.resources.ResourceTestUtil.createInWorkspace;
@@ -24,22 +25,27 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.OutputStream;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IFolder;
 import org.eclipse.core.resources.IProject;
 import org.eclipse.core.resources.IProjectDescription;
 import org.eclipse.core.resources.IResource;
 import org.eclipse.core.resources.IResourceStatus;
 import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Platform.OS;
+import org.eclipse.core.runtime.Status;
+import org.eclipse.core.runtime.jobs.Job;
 import org.eclipse.core.tests.resources.util.WorkspaceResetExtension;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 
 @ExtendWith(WorkspaceResetExtension.class)
 public class IFileTest {
-
 	/**
 	 * Bug states that the error code in the CoreException which is thrown when
 	 * you try to create a file in a read-only folder on Linux should be
@@ -125,4 +131,72 @@ public void testBug43936() throws CoreException {
 		project.setDescription(desc, createTestMonitor());
 	}
 
+	/**
+	 * Do not throw RuntimeException when accessing a deleted file
+	 */
+	@Test
+	public void testIssue2290() throws CoreException, InterruptedException {
+		IProject project = getWorkspace().getRoot().getProject("MyProject");
+		IFile subject = project.getFile("subject.txt");
+		Job noise = Job.create("Create/delete", monitor -> {
+			try {
+				while (!monitor.isCanceled()) {
+					createInWorkspace(subject);
+					subject.delete(true, null);
+				}
+			} catch (CoreException e) {
+				return e.getStatus();
+			}
+			return Status.OK_STATUS;
+		});
+		noise.setPriority(Job.INTERACTIVE);
+
+		long stop = currentTimeMillis() + 1000;
+		try {
+			noise.schedule();
+			while (currentTimeMillis() < stop) {
+				assertContentAccessibleOrNotFound(subject); // should not throw
+			}
+		} finally {
+			noise.cancel();
+			noise.join();
+			IStatus result = noise.getResult();
+			if (!result.isOK()) {
+				throw new CoreException(result);
+			}
+		}
+	}
+
+	private void assertContentAccessibleOrNotFound(IFile file) {
+		try (InputStream contents = file.getContents(false)) {
+			contents.transferTo(OutputStream.nullOutputStream());
+		} catch (IOException e) {
+			throw new AssertionError(e);
+		} catch (CoreException e) {
+			switch (e.getStatus().getCode()) {
+			case IResourceStatus.RESOURCE_NOT_LOCAL:
+			case IResourceStatus.RESOURCE_NOT_FOUND:
+			case IResourceStatus.FAILED_READ_LOCAL:
+			case IResourceStatus.OUT_OF_SYNC_LOCAL:
+				break;
+			default:
+				throw new AssertionError(e);
+			}
+		}
+		try (InputStream contents = file.getContents(true)) {
+			contents.transferTo(OutputStream.nullOutputStream());
+		} catch (IOException e) {
+			throw new AssertionError(e);
+		} catch (CoreException e) {
+			switch (e.getStatus().getCode()) {
+			case IResourceStatus.RESOURCE_NOT_LOCAL:
+			case IResourceStatus.RESOURCE_NOT_FOUND:
+			case IResourceStatus.FAILED_READ_LOCAL:
+				break;
+			default:
+				throw new AssertionError(e);
+			}
+		}
+	}
+
 }