Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 275020 - Reverting workspace only works for compatibility tests
Summary: Reverting workspace only works for compatibility tests
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: API Tools (show other bugs)
Version: 3.5   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6 M1   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 282537
  Show dependency tree
 
Reported: 2009-05-05 12:30 EDT by Michael Rennie CLA
Modified: 2009-07-27 14:23 EDT (History)
1 user (show)

See Also:
Olivier_Thomann: review+

Attachments
fix (285.13 KB, patch)
2009-07-16 15:54 EDT, Michael Rennie CLA 		no flags Details | Diff
update (294.55 KB, patch)
2009-07-17 10:50 EDT, Michael Rennie CLA 		no flags Details | Diff
Show Obsolete (1) 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 Michael Rennie CLA 2009-05-05 12:30:12 EDT 
FUP to bug 275005, the usage tests were attempting to use the revertible workspace, which was found to only work for compatibility tests (reliably). The reverting workspace should be updated to allow all tests to leverage it.
Comment 1 Michael Rennie CLA 2009-07-16 15:54:16 EDT 
Created attachment 141814 [details]
fix

While trying to figure out bug 282537, I fixed this one so that the tests did not take as long to run.
Comment 2 Michael Rennie CLA 2009-07-17 10:50:23 EDT 
Created attachment 141893 [details]
update

updated patch, apparently when creating the other patch, two test src dirs did not get included, so running with the other patch causes tonnes of "Resource not found" exceptions.
Comment 3 Michael Rennie CLA 2009-07-17 11:35:46 EDT 
apparently the patch wizard does not like folders with dummy text files in them...the second patch also is missing the test dirs, even though they are checked in the patch wizard...
Comment 4 Michael Rennie CLA 2009-07-17 12:17:15 EDT 
released to HEAD (including test dirs)
Comment 5 Michael Rennie CLA 2009-07-17 12:32:16 EDT 
please verify Olivier
Comment 6 Olivier Thomann CLA 2009-07-27 14:23:11 EDT 
+1. Verify.
Some failures still show up, but this change still looks good.