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

Bug 341887

Summary: Tests for new functionality in Equinox console
Product: [Eclipse Project] Equinox Reporter: Lazar Kirchev <l.kirchev>
Component: FrameworkAssignee: equinox.framework-inbox <equinox.framework-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: tjwatson
Version: 3.7   
Target Milestone: 3.7 M7   
Hardware: PC   
OS: Windows Vista   
Whiteboard:
Attachments:
Description Flags
Patch providing the necessary tests tjwatson: iplog+

Description Lazar Kirchev CLA 2011-04-05 06:57:38 EDT
Tests are needed for:
- the new functionality for providing help for an individual command
- the property which disables the console
- the restriction of the telnet access on a particular address of the host
Comment 1 Lazar Kirchev CLA 2011-04-05 07:05:02 EDT
Created attachment 192540 [details]
Patch providing the necessary tests

The test for the functionality for providing help for individual command is add to the TestCommandExecution class. The test is to execute the help command for one command from the FrameworkCommandProvider and for one command from a custom command provider. The test checks the output of the command.

The test for the property for disabling the console is in TestEquinoxStartWithoutConsole. The test starts a framework with the property for disabling the console set to true and checks that the FrameworkConsoleSession and FrameworkCommandProvider are not registered.

The test for restricting telnet access to a particular network interface is in TestRestricted TelnetHost. It starts a framework with telnet open on localhost and checks that a connection cannot be opened using the ip address of the host.
Comment 2 Thomas Watson CLA 2011-04-05 09:05:29 EDT
Comment on attachment 192540 [details]
Patch providing the necessary tests

Thanks Lazar
Comment 3 Thomas Watson CLA 2011-04-05 09:06:36 EDT
Thanks for the test contribution Lazar!  I released the tests with a small change to call Framework.stop() in a couple places where the test framework was left running.
Comment 4 Lazar Kirchev CLA 2011-04-05 09:36:58 EDT
Thanks for the correction Tom! Somehow I have missed this.