Bug 248508 - [logview] unit tests
Summary: [logview] unit tests
Status: RESOLVED INVALID
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Jacek Pospychala CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted
Depends on:
Blocks:
 
Reported: 2008-09-24 18:25 EDT by Chris Aniszczyk CLA
Modified: 2013-01-28 11:56 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Aniszczyk CLA 2008-09-24 18:25:07 EDT
The log view is one of the most widely used views in Eclipse... from people using it in their daily use or just ripping it apart for inclusion in RCP applications. It's about time we have some unit tests for the view so we don't break things unexpectedly.
Comment 1 Chris Aniszczyk CLA 2008-09-24 18:28:31 EDT
Jacek, I will leave this in your bucket.

I don't know if you have any good ideas on this one, but one of the themes for 3.5 is robustness so having unit tests here is a good thing.
Comment 2 Jacek Pospychala CLA 2008-09-25 08:55:14 EDT
oh this is good idea, I think I had some unit tests somewhere here long time ago, when rewriting the log parser stuff a bit. I'll look around for that.
I think that having log files parser tests is a must, test cases for listening events from framework would be good too. Another fragile part, worth testing, is events grouping (by date/plugin/etc...). That's for start, I guess there may be more things to test, when I start writing junits...
Comment 3 Chris Aniszczyk CLA 2008-09-25 09:26:51 EDT
The first cut doesn't have to be comprehensive.

It's important that we have some though so it can easily be added to once we come across bugs or other issues.
Comment 4 Curtis Windatt CLA 2013-01-28 11:56:40 EST
No work has been done on this in 4 years.  Closing as INVALID.