Attachment 53543 Details for Bug 162621 – Proposed patch

[patch] Proposed patch

v00.txt (text/plain), 4.57 KB, created by Frederic Fusier

on 2006-11-09 09:48:37 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Frederic Fusier

Created: 2006-11-09 09:48:37 EST

Size: 4.57 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core.tests.model
>Index: src/org/eclipse/jdt/core/tests/model/ReconcilerTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/ReconcilerTests.java,v
>retrieving revision 1.106
>diff -u -r1.106 ReconcilerTests.java
>--- src/org/eclipse/jdt/core/tests/model/ReconcilerTests.java	8 Nov 2006 15:18:37 -0000	1.106
>+++ src/org/eclipse/jdt/core/tests/model/ReconcilerTests.java	9 Nov 2006 14:52:28 -0000
>@@ -16,6 +16,7 @@
> 
> import junit.framework.Test;
> 
>+import org.eclipse.core.resources.IResource;
> import org.eclipse.core.resources.IWorkspaceRunnable;
> import org.eclipse.core.runtime.CoreException;
> import org.eclipse.core.runtime.ILogListener;
>@@ -870,6 +871,84 @@
> 	}
> }
> /**
>+ * @bug 162621: [model][delta] Validation errors do not clear after replacing jar file
>+ * @test Ensures that changing an internal jar and refreshing takes the change into account
>+ * @see "https://bugs.eclipse.org/bugs/show_bug.cgi?id=162621"
>+ */
>+public void testChangeInternalJar() throws CoreException, IOException {
>+	IJavaProject project = getJavaProject("Reconciler");
>+	String jarName = "b162621.jar";
>+	try {
>+		String[] pathAndContents = new String[] {
>+			"test/before/Foo.java",
>+			"package test.before;\n" + 
>+			"public class Foo {\n" + 
>+			"}\n"
>+		};
>+		addLibrary(project, jarName, "b162621_src.zip", pathAndContents, JavaCore.VERSION_1_4);
>+
>+		// Set working copy content with no error
>+		setUpWorkingCopy("/Reconciler/src/test/Test.java",
>+			"package test;\n" + 
>+			"import test.before.Foo;\n" + 
>+			"public class Test {\n" + 
>+			"	Foo f;\n" + 
>+			"}\n"
>+		);
>+		assertProblems(
>+			"Unexpected problems", 
>+			"----------\n" + 
>+			"----------\n"
>+		);
>+
>+		// Update working copy with Jar expected changes
>+		String contents = "package test;\n" + 
>+			"import test.after.Foo;\n" + 
>+			"public class Test {\n" + 
>+			"	Foo f;\n" + 
>+			"}\n";
>+		setWorkingCopyContents(contents);
>+		this.workingCopy.reconcile(ICompilationUnit.NO_AST, true, null, null);
>+		assertProblems(
>+			"Wrong expected problems", 
>+			"----------\n" + 
>+			"1. ERROR in /Reconciler/src/test/Test.java (at line 2)\n" + 
>+			"	import test.after.Foo;\n" + 
>+			"	       ^^^^^^^^^^\n" + 
>+			"The import test.after cannot be resolved\n" + 
>+			"----------\n" + 
>+			"2. ERROR in /Reconciler/src/test/Test.java (at line 4)\n" + 
>+			"	Foo f;\n" + 
>+			"	^^^\n" + 
>+			"Foo cannot be resolved to a type\n" + 
>+			"----------\n"
>+		);
>+		
>+		// change jar and refresh
>+		String projectLocation = project.getProject().getLocation().toOSString();
>+		String jarPath = projectLocation + File.separator + jarName;
>+		org.eclipse.jdt.core.tests.util.Util.createJar(new String[] {
>+			"test/after/Foo.java",
>+			"package test.after;\n" + 
>+			"public class Foo {\n" + 
>+			"}\n"
>+		}, jarPath, "1.4");
>+		project.getProject().refreshLocal(IResource.DEPTH_INFINITE, null);
>+
>+		// Verify that error is gone
>+		this.problemRequestor.initialize(contents.toCharArray());
>+		this.workingCopy.reconcile(ICompilationUnit.NO_AST, true, null, null);
>+		assertProblems(
>+			"Unexpected problems", 
>+			"----------\n" + 
>+			"----------\n"
>+		);
>+	} finally {
>+		removeLibraryEntry(project, new Path(jarName));
>+		deleteFile(new File(jarName));
>+	}
>+}
>+/**
>  * Ensures that the reconciler reconciles the new contents with the current
>  * contents,updating the structure of this reconciler's compilation
>  * unit, and fires the Java element deltas for the structural changes
>#P org.eclipse.jdt.core
>Index: model/org/eclipse/jdt/internal/core/DeltaProcessor.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/model/org/eclipse/jdt/internal/core/DeltaProcessor.java,v
>retrieving revision 1.291
>diff -u -r1.291 DeltaProcessor.java
>--- model/org/eclipse/jdt/internal/core/DeltaProcessor.java	6 Nov 2006 18:11:00 -0000	1.291
>+++ model/org/eclipse/jdt/internal/core/DeltaProcessor.java	9 Nov 2006 14:52:30 -0000
>@@ -624,6 +624,9 @@
> 			int flags = IJavaElementDelta.F_CONTENT;
> 			if (element instanceof JarPackageFragmentRoot){
> 				flags |= IJavaElementDelta.F_ARCHIVE_CONTENT_CHANGED;
>+				// need also to reset project cache otherwise it will be out-of-date
>+				// see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=162621
>+				this.projectCachesToReset.add(element.getJavaProject());
> 			}
> 			if (isPrimary) {
> 				flags |= IJavaElementDelta.F_PRIMARY_RESOURCE;

Actions: View | Diff

Attachments on bug 162621: 52885 | 53543 | 53548