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

Bug 388897

Summary:	Create test cases for downloading attachment content
Product:	z_Archived	Reporter:	Minoo Ziaei <minoo.ziaei>
Component:	Mylyn	Assignee:	Frank Becker <eclipse>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	P3	CC:	b.muskalla, eclipse, minoo.ziaei
Version:	3.7
Target Milestone:	3.9
Hardware:	All
OS:	All
Whiteboard:
Bug Depends on:	395010
Bug Blocks:

Description Minoo Ziaei

2012-09-05 21:28:42 EDT

Currently, there are no test cases available to test downloading attachments in Bugzilla.
There is a possibility that if we create few test cases that in each of them we both post and get contents, we will fail, receiving ConnectionPoolTimeoutException. But we need to verify that.

Comment 1 Steffen Pingel

2012-09-05 22:33:03 EDT

We need to add a few simple tests that retrieve attachments. We should also look into properly implementing the streaming API in BugzillaTaskAttachmentHandler instead of caching attachment data in memory.

Comment 2 Steffen Pingel

2012-11-24 19:21:12 EST

Frank identified the problem that was causing a connection leak. I have pushed a review here with a proposed fix: https://git.eclipse.org/r/8846.

Comment 3 Steffen Pingel

2013-05-07 07:50:32 EDT

Frank, there are some tests in BugzillaAttachmentHandlerTest. Do we need anything else here?

Comment 4 Frank Becker

2013-05-08 15:38:13 EDT

No, I do not have a plan to add some more tests.

It is fine to close this.