Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323922 - Use smaller repository for SynchronizeViewTest
Summary: Use smaller repository for SynchronizeViewTest
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-29 17:56 EDT by Benjamin Muskalla CLA
Modified: 2010-10-05 18:09 EDT (History)
3 users (show)

See Also:

Attachments
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 Benjamin Muskalla CLA 2010-08-29 17:56:14 EDT 
Currently, the SynchronizeViewTest uses the egit repo as test repository. This adds a lot of waiting time for the test to finish due to cloning and synchronizing (test takes 2m 40secs to finish on my machine). We should really use a smaller and more deterministic repository for these tests.
Comment 1 Stefan Lay CLA 2010-08-30 04:22:09 EDT 
We should use the same local repository which is used for the clone test. In commit e29cc2a4a20e94eda3ea902da3e1894846860813 Matthias introduced a repo which "is now
generated, has a much smaller size and its daemon is run as part
of the test harness to speedup the UI tests and make them
independent from external resources."
Comment 2 Matthias Sohn CLA 2010-09-28 06:01:55 EDT 
Take a look at GitCloneWizardTest and SampleTestRepository how I did that
for the clone tests. Since changing them to use locally generated repository 
we never had any problems with these tests anymore.

Couldn't you instead of cloning generate a test repo programmatically ? Have a look at
GitRepositoriesViewTestBase and its abstract base class LocalRepositoryTestCase
Comment 3 Dariusz Luksza CLA 2010-09-29 20:06:06 EDT 
I've rewrite SynchronizeViewTest, now it is based on LocalRepositoryTestCase, here is patch for this:

http://egit.eclipse.org/r/#change,1677
Comment 4 Matthias Sohn CLA 2010-10-05 18:09:58 EDT 
merged as 57b2c2f4d67f307ff32af541ca7a5ca1b4b97e22