Bug 301529 - ClasspathTests.testBug300136a fails on non-Windows machines
Summary: ClasspathTests.testBug300136a fails on non-Windows machines
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.6   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.6 M6   Edit
Assignee: Jay Arthanareeswaran CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-02 07:31 EST by Frederic Fusier CLA
Modified: 2010-03-08 07:16 EST (History)
2 users (show)

See Also:

Attachments
Test updated (3.03 KB, patch)
2010-02-03 01:24 EST, Jay Arthanareeswaran CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Frederic Fusier CLA 2010-02-02 07:31:48 EST 
Unit Test Results for N20100201-2000 show a failure in JDT/Core model test:
ClasspathTests.testBug300136a fails on all non-Windows machines...

The faulty line is:
	"Project \'P\' is missing required library: \'\\lib\\tmp.jar\'\n" + 
where the file separator is supposed to be the backslash '\' although it's only trur on Windows machine...

Looking closer at this test, it also relies on the fact that the TEST_LIB classpath variable has been set in a previous test. This should also be changed because we cannot assume that tests are always executed in the same order while running a test suite (it's VM dependent...)
Comment 1 Frederic Fusier CLA 2010-02-02 07:33:56 EST 
I've disabled the test to avoid these failures in today's integration build...
Comment 2 Jay Arthanareeswaran CLA 2010-02-03 01:24:21 EST 
Created attachment 158002 [details]
Test updated

Updated the test addressing the problems mentioned in comment 0.
Comment 3 Jay Arthanareeswaran CLA 2010-02-03 01:52:35 EST 
Released in HEAD for 3.6M6. 
To be verified by a code review.
Comment 4 Satyam Kandula CLA 2010-03-08 07:08:34 EST 
Reviewed the source code
Verified for 3.6M6 using build I20100305-101
Comment 5 Jay Arthanareeswaran CLA 2010-03-08 07:16:27 EST 
Verified.