Attachment #205212 for bug #357425

Lines 1827-1832 Link Here

(-)model/org/eclipse/jdt/internal/core/ClasspathEntry.java (+3 lines)
1827	* @return a java model status describing the problem related to this classpath entry if any, a status object with code <code>IStatus.OK</code> if the entry is fine	1827	* @return a java model status describing the problem related to this classpath entry if any, a status object with code <code>IStatus.OK</code> if the entry is fine
1828	*/	1828	*/
1829	public static IJavaModelStatus validateClasspathEntry(IJavaProject project, IClasspathEntry entry, boolean checkSourceAttachment, boolean referredByContainer){	1829	public static IJavaModelStatus validateClasspathEntry(IJavaProject project, IClasspathEntry entry, boolean checkSourceAttachment, boolean referredByContainer){
		1830	if (entry.getEntryKind() == IClasspathEntry.CPE_LIBRARY) {
		1831	JavaModelManager.getJavaModelManager().removeFromInvalidArchiveCache(entry.getPath());
		1832	}
1830	IJavaModelStatus status = validateClasspathEntry(project, entry, null, checkSourceAttachment, referredByContainer);	1833	IJavaModelStatus status = validateClasspathEntry(project, entry, null, checkSourceAttachment, referredByContainer);
1831	// https://bugs.eclipse.org/bugs/show_bug.cgi?id=171136 and https://bugs.eclipse.org/bugs/show_bug.cgi?id=300136	1834	// https://bugs.eclipse.org/bugs/show_bug.cgi?id=171136 and https://bugs.eclipse.org/bugs/show_bug.cgi?id=300136
1832	// Ignore class path errors from optional entries.	1835	// Ignore class path errors from optional entries.




				// project does not exist -> ignore
				continue;
			}
			boolean deltaContainsModifiedJar = false;
			for (int j = 0; j < entries.length; j++){
				if (entries[j].getEntryKind() == IClasspathEntry.CPE_LIBRARY) {
					IPath entryPath = entries[j].getPath();

								System.out.println("- External JAR ADDED, affecting root: "+root.getElementName()); //$NON-NLS-1$
							}
							elementAdded(root, null, null);
							deltaContainsModifiedJar = true;
							this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733
							hasDelta = true;
						} else if (status == EXTERNAL_JAR_CHANGED) {

								System.out.println("- External JAR CHANGED, affecting root: "+root.getElementName()); //$NON-NLS-1$
							}
							contentChanged(root);
							deltaContainsModifiedJar = true;
							hasDelta = true;
						} else if (status == EXTERNAL_JAR_REMOVED) {
							PackageFragmentRoot root = (PackageFragmentRoot) javaProject.getPackageFragmentRoot(entryPath.toString());

								System.out.println("- External JAR REMOVED, affecting root: "+root.getElementName()); //$NON-NLS-1$
							}
							elementRemoved(root, null, null);
							deltaContainsModifiedJar = true;
							this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733
							hasDelta = true;
						}

				}
			}
			
			if (deltaContainsModifiedJar) {
				javaProject.resetResolvedClasspath();
			}
		}

Lines 3086-3091 Link Here

(-)model/org/eclipse/jdt/internal/core/JavaModelManager.java (+6 lines)
3086	return this.invalidArchives != null && this.invalidArchives.contains(path);	3086	return this.invalidArchives != null && this.invalidArchives.contains(path);
3087	}	3087	}
3088		3088
		3089	public void removeFromInvalidArchiveCache(IPath path) {
		3090	if (this.invalidArchives != null) {
		3091	this.invalidArchives.remove(path);
		3092	}
		3093	}
		3094
3089	public void setClasspathBeingResolved(IJavaProject project, boolean classpathIsResolved) {	3095	public void setClasspathBeingResolved(IJavaProject project, boolean classpathIsResolved) {
3090	if (classpathIsResolved) {	3096	if (classpathIsResolved) {
3091	getClasspathBeingResolved().add(project);	3097	getClasspathBeingResolved().add(project);




	suite.addTest(new ClasspathTests("testInvalidClasspath2"));
	suite.addTest(new ClasspathTests("testInvalidExternalClassFolder"));
	suite.addTest(new ClasspathTests("testInvalidExternalJar"));
	suite.addTest(new ClasspathTests("testTransitionFromInvalidToValidJar"));
	suite.addTest(new ClasspathTests("testInvalidInternalJar1"));
	suite.addTest(new ClasspathTests("testInvalidInternalJar2"));
	suite.addTest(new ClasspathTests("testInvalidSourceFolder"));

	suite.addTest(new ClasspathTests("testBug321170"));
	suite.addTest(new ClasspathTests("testBug229042"));
	suite.addTest(new ClasspathTests("testBug274737"));
	suite.addTest(new ClasspathTests("testBug357425"));
	return suite;
}
public void setUpSuite() throws Exception {

	}
}
/*
 * Ensures that validateClasspathEntry() sees a transition from an invalid/missing jar to a valid jar.
 */
public void testTransitionFromInvalidToValidJar() throws CoreException, IOException {
	String transitioningJarName = "transitioningJar.jar";
	String transitioningJar = getExternalPath() + transitioningJarName;
	String nonExistingJar = getExternalPath() + "nonExisting.jar";
	IClasspathEntry transitioningEntry = JavaCore.newLibraryEntry(new Path(transitioningJar), null, null);
	IClasspathEntry nonExistingEntry = JavaCore.newLibraryEntry(new Path(nonExistingJar), null, null);

	try {
		IJavaProject proj = createJavaProject("P", new String[] {}, new String[] {transitioningJar, nonExistingJar}, "bin");
		
		IJavaModelStatus status1 = ClasspathEntry.validateClasspathEntry(proj, transitioningEntry, false, false);
		IJavaModelStatus status2 = ClasspathEntry.validateClasspathEntry(proj, nonExistingEntry, false, false);
		assertFalse("Non-existing jar should be invalid", status1.isOK());
		assertFalse("Non-existing jar should be invalid", status2.isOK());

		Util.createJar(	
			new String[0],
			new String[] {
				"META-INF/MANIFEST.MF",
				"Manifest-Version: 1.0\n"
			},
			transitioningJar,
			JavaCore.VERSION_1_4);
		status1 = ClasspathEntry.validateClasspathEntry(proj, transitioningEntry, false, false);
		status2 = ClasspathEntry.validateClasspathEntry(proj, nonExistingEntry, false, false);
		assertTrue("Existing jar should be valid", status1.isOK());
		assertFalse("Non-existing jar should be invalid", status2.isOK());
	} finally {
		deleteExternalResource(transitioningJarName);
		deleteProject("P");
	}
}
/*
 * Ensures that a non existing internal jar cannot be put on the classpath.
 */
public void testInvalidInternalJar1() throws CoreException {

	}
}

/*
 * Ensures that the correct delta is reported when changing the Class-Path: clause 
 * of an external jar from not containing a chained jar to containing a chained jar.
 * (regression test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=357425)
 */
public void testBug357425() throws Exception {
	try {
		IJavaProject p = createJavaProject("P");
		addExternalLibrary(p, getExternalResourcePath("lib357425_a.jar"), new String[0], 
			new String[] {
				"META-INF/MANIFEST.MF",
				"Manifest-Version: 1.0\n"
			},
			JavaCore.VERSION_1_4);
		refreshExternalArchives(p);		

		startDeltas();
		org.eclipse.jdt.core.tests.util.Util.createJar(new String[0],
			new String[] {
				"META-INF/MANIFEST.MF",
				"Manifest-Version: 1.0\n" +
				"Class-Path: lib357425_b.jar\n",
			},
			getExternalResourcePath("lib357425_a.jar"),
			JavaCore.VERSION_1_4);
		createExternalFile("lib357425_b.jar", "");

		refreshExternalArchives(p);
		assertDeltas(
			"Unexpected delta",
			"P[*]: {CHILDREN | RESOLVED CLASSPATH CHANGED}\n" + 
			"	"+ getExternalPath() + "lib357425_a.jar[*]: {CONTENT | REORDERED | ARCHIVE CONTENT CHANGED}\n" + 
			"	"+ getExternalPath() + "lib357425_b.jar[+]: {}"
				);
	} finally {
		stopDeltas();
		deleteProject("P");
		deleteExternalResource("lib357425_a.jar");
		deleteExternalResource("lib357425_b.jar");
	}
}

}

Return to bug 357425

Lines 954-960 Link Here

(-)model/org/eclipse/jdt/internal/core/DeltaProcessor.java (-5 / +5 lines)
954	// project does not exist -> ignore	954	// project does not exist -> ignore
955	continue;	955	continue;
956	}	956	}
957	boolean hasChainedJar = false;	957	boolean deltaContainsModifiedJar = false;
958	for (int j = 0; j < entries.length; j++){	958	for (int j = 0; j < entries.length; j++){
959	if (entries[j].getEntryKind() == IClasspathEntry.CPE_LIBRARY) {	959	if (entries[j].getEntryKind() == IClasspathEntry.CPE_LIBRARY) {
960	IPath entryPath = entries[j].getPath();	960	IPath entryPath = entries[j].getPath();
Lines 1024-1030 Link Here
1024	System.out.println("- External JAR ADDED, affecting root: "+root.getElementName()); //$NON-NLS-1$	1024	System.out.println("- External JAR ADDED, affecting root: "+root.getElementName()); //$NON-NLS-1$
1025	}	1025	}
1026	elementAdded(root, null, null);	1026	elementAdded(root, null, null);
1027	hasChainedJar \|= !this.manager.isNonChainingJar(entryPath);	1027	deltaContainsModifiedJar = true;
1028	this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733	1028	this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733
1029	hasDelta = true;	1029	hasDelta = true;
1030	} else if (status == EXTERNAL_JAR_CHANGED) {	1030	} else if (status == EXTERNAL_JAR_CHANGED) {
Lines 1033-1039 Link Here
1033	System.out.println("- External JAR CHANGED, affecting root: "+root.getElementName()); //$NON-NLS-1$	1033	System.out.println("- External JAR CHANGED, affecting root: "+root.getElementName()); //$NON-NLS-1$
1034	}	1034	}
1035	contentChanged(root);	1035	contentChanged(root);
1036	hasChainedJar \|= !this.manager.isNonChainingJar(entryPath);	1036	deltaContainsModifiedJar = true;
1037	hasDelta = true;	1037	hasDelta = true;
1038	} else if (status == EXTERNAL_JAR_REMOVED) {	1038	} else if (status == EXTERNAL_JAR_REMOVED) {
1039	PackageFragmentRoot root = (PackageFragmentRoot) javaProject.getPackageFragmentRoot(entryPath.toString());	1039	PackageFragmentRoot root = (PackageFragmentRoot) javaProject.getPackageFragmentRoot(entryPath.toString());
Lines 1041-1047 Link Here
1041	System.out.println("- External JAR REMOVED, affecting root: "+root.getElementName()); //$NON-NLS-1$	1041	System.out.println("- External JAR REMOVED, affecting root: "+root.getElementName()); //$NON-NLS-1$
1042	}	1042	}
1043	elementRemoved(root, null, null);	1043	elementRemoved(root, null, null);
1044	hasChainedJar \|= !this.manager.isNonChainingJar(entryPath);	1044	deltaContainsModifiedJar = true;
1045	this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733	1045	this.state.addClasspathValidation(javaProject); // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=185733
1046	hasDelta = true;	1046	hasDelta = true;
1047	}	1047	}
Lines 1049-1055 Link Here
1049	}	1049	}
1050	}	1050	}
1051		1051
1052	if (hasChainedJar) {	1052	if (deltaContainsModifiedJar) {
1053	javaProject.resetResolvedClasspath();	1053	javaProject.resetResolvedClasspath();
1054	}	1054	}
1055	}	1055	}

Lines 249-254 Link Here

(-)src/org/eclipse/jdt/core/tests/model/ClasspathTests.java (+79 lines)
249	suite.addTest(new ClasspathTests("testInvalidClasspath2"));	249	suite.addTest(new ClasspathTests("testInvalidClasspath2"));
250	suite.addTest(new ClasspathTests("testInvalidExternalClassFolder"));	250	suite.addTest(new ClasspathTests("testInvalidExternalClassFolder"));
251	suite.addTest(new ClasspathTests("testInvalidExternalJar"));	251	suite.addTest(new ClasspathTests("testInvalidExternalJar"));
		252	suite.addTest(new ClasspathTests("testTransitionFromInvalidToValidJar"));
252	suite.addTest(new ClasspathTests("testInvalidInternalJar1"));	253	suite.addTest(new ClasspathTests("testInvalidInternalJar1"));
253	suite.addTest(new ClasspathTests("testInvalidInternalJar2"));	254	suite.addTest(new ClasspathTests("testInvalidInternalJar2"));
254	suite.addTest(new ClasspathTests("testInvalidSourceFolder"));	255	suite.addTest(new ClasspathTests("testInvalidSourceFolder"));
Lines 334-339 Link Here
334	suite.addTest(new ClasspathTests("testBug321170"));	335	suite.addTest(new ClasspathTests("testBug321170"));
335	suite.addTest(new ClasspathTests("testBug229042"));	336	suite.addTest(new ClasspathTests("testBug229042"));
336	suite.addTest(new ClasspathTests("testBug274737"));	337	suite.addTest(new ClasspathTests("testBug274737"));
		338	suite.addTest(new ClasspathTests("testBug357425"));
337	return suite;	339	return suite;
338	}	340	}
339	public void setUpSuite() throws Exception {	341	public void setUpSuite() throws Exception {
Lines 4201-4206 Link Here
4201	}	4203	}
4202	}	4204	}
4203	/*	4205	/*
		4206	* Ensures that validateClasspathEntry() sees a transition from an invalid/missing jar to a valid jar.
		4207	*/
		4208	public void testTransitionFromInvalidToValidJar() throws CoreException, IOException {
		4209	String transitioningJarName = "transitioningJar.jar";
		4210	String transitioningJar = getExternalPath() + transitioningJarName;
		4211	String nonExistingJar = getExternalPath() + "nonExisting.jar";
		4212	IClasspathEntry transitioningEntry = JavaCore.newLibraryEntry(new Path(transitioningJar), null, null);
		4213	IClasspathEntry nonExistingEntry = JavaCore.newLibraryEntry(new Path(nonExistingJar), null, null);
		4214
		4215	try {
		4216	IJavaProject proj = createJavaProject("P", new String[] {}, new String[] {transitioningJar, nonExistingJar}, "bin");
		4217
		4218	IJavaModelStatus status1 = ClasspathEntry.validateClasspathEntry(proj, transitioningEntry, false, false);
		4219	IJavaModelStatus status2 = ClasspathEntry.validateClasspathEntry(proj, nonExistingEntry, false, false);
		4220	assertFalse("Non-existing jar should be invalid", status1.isOK());
		4221	assertFalse("Non-existing jar should be invalid", status2.isOK());
		4222
		4223	Util.createJar(
		4224	new String[0],
		4225	new String[] {
		4226	"META-INF/MANIFEST.MF",
		4227	"Manifest-Version: 1.0\n"
		4228	},
		4229	transitioningJar,
		4230	JavaCore.VERSION_1_4);
		4231	status1 = ClasspathEntry.validateClasspathEntry(proj, transitioningEntry, false, false);
		4232	status2 = ClasspathEntry.validateClasspathEntry(proj, nonExistingEntry, false, false);
		4233	assertTrue("Existing jar should be valid", status1.isOK());
		4234	assertFalse("Non-existing jar should be invalid", status2.isOK());
		4235	} finally {
		4236	deleteExternalResource(transitioningJarName);
		4237	deleteProject("P");
		4238	}
		4239	}
		4240	/*
4204	* Ensures that a non existing internal jar cannot be put on the classpath.	4241	* Ensures that a non existing internal jar cannot be put on the classpath.
4205	*/	4242	*/
4206	public void testInvalidInternalJar1() throws CoreException {	4243	public void testInvalidInternalJar1() throws CoreException {
Lines 7190-7193 Link Here
7190	}	7227	}
7191	}	7228	}
7192		7229
		7230	/*
		7231	* Ensures that the correct delta is reported when changing the Class-Path: clause
		7232	* of an external jar from not containing a chained jar to containing a chained jar.
		7233	* (regression test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=357425)
		7234	*/
		7235	public void testBug357425() throws Exception {
		7236	try {
		7237	IJavaProject p = createJavaProject("P");
		7238	addExternalLibrary(p, getExternalResourcePath("lib357425_a.jar"), new String[0],
		7239	new String[] {
		7240	"META-INF/MANIFEST.MF",
		7241	"Manifest-Version: 1.0\n"
		7242	},
		7243	JavaCore.VERSION_1_4);
		7244	refreshExternalArchives(p);
		7245
		7246	startDeltas();
		7247	org.eclipse.jdt.core.tests.util.Util.createJar(new String[0],
		7248	new String[] {
		7249	"META-INF/MANIFEST.MF",
		7250	"Manifest-Version: 1.0\n" +
		7251	"Class-Path: lib357425_b.jar\n",
		7252	},
		7253	getExternalResourcePath("lib357425_a.jar"),
		7254	JavaCore.VERSION_1_4);
		7255	createExternalFile("lib357425_b.jar", "");
		7256
		7257	refreshExternalArchives(p);
		7258	assertDeltas(
		7259	"Unexpected delta",
		7260	"P[*]: {CHILDREN \| RESOLVED CLASSPATH CHANGED}\n" +
		7261	" "+ getExternalPath() + "lib357425_a.jar[*]: {CONTENT \| REORDERED \| ARCHIVE CONTENT CHANGED}\n" +
		7262	" "+ getExternalPath() + "lib357425_b.jar[+]: {}"
		7263	);
		7264	} finally {
		7265	stopDeltas();
		7266	deleteProject("P");
		7267	deleteExternalResource("lib357425_a.jar");
		7268	deleteExternalResource("lib357425_b.jar");
		7269	}
		7270	}
		7271
7193	}	7272	}