Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 3024

Summary:

[resources] Ensure removing sync info deletes resource (1GD43Y0)

Product:

[Eclipse Project] Platform

Reporter:

DJ Houghton <dj.houghton>

Component:

Resources

Assignee:

Rafael Chaves <eclipse>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Keywords:

helpwanted

Version:

2.0

Target Milestone:

2.1 M4

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Test case for checking expected behaviour	none

Description DJ Houghton

2001-10-10 22:48:03 EDT

Make sure that when we remove sync info for a resource
	that we are deleting the resource (and all its children?) if
	there is no other info.

NOTES:

DJH (5/3/01 8:35:02 AM)
	Should we be deleting children automatically?
	Consider the following:
	- have project /proj
	- have phantoms /proj/a, /proj/a/b, /proj/a/c (note: all have sync info)
	- set the sync info for /proj/a to be null
	- /proj/a is a phantom and is no longer synchronizing so
	  we should delete it
	- but its children still have sync info
	- should we go by the assumption that if the parent is no longer
	  synchronizing, then the children aren't either?
	- can you synchronize subtrees?
	- I think targets can!

DJH (5/4/01 7:14:26 PM)
	We will not allow the synchronization of subtrees. This is just
	too expensive w.r.t. the deletions and garbage collection.

	Currently VCM forces the user to synchronize at the project
	level.

	With targets we will add to the API that if a user wants
	to synchronize a subtree, then they must put sync info
	on the parents of the tree.

Comment 1 Rafael Chaves

2002-12-03 11:52:17 EST

Created attachment 2627 [details]
Test case for checking expected behaviour

Adds a test case to ISynchronizerTest that removes a resource, sets its sync
info to null and ensures the phantom does not exist any more. Tests when the
removed resource is a file and a folder containing a file.

Please verify and release.

Comment 2 DJ Houghton

2002-12-03 13:29:27 EST

Test case released into HEAD.