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

Bug 314711

Summary: Invalid characters allowed as name of test assets
Product: z_Archived Reporter: Ben Chen <chenbs>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED WORKSFORME QA Contact: Kathy Chan <kathy>
Severity: normal    
Priority: P2 CC: paulslau
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Ben Chen CLA 2010-05-27 11:55:37 EDT
Build Identifier: TPTP-4.7.0-201005251601

related to Test.Execution.JUnitRunner
case: Bugzilla_286952

Reproducible: Always

Steps to Reproduce:
1.In the overview section of a test asset, change its name to contain any of the following: "!", "*", "(", ")"
2.Notice that the action is granted and the test asset is saved with the invalid name string
Comment 1 Ben Chen CLA 2010-05-27 11:57:44 EDT
Paul, I recall seeing a test case covering this scenario in the Navigator test suite but cannot remember it on top of my head. If no such navigator test exists, shouldn't this test step be written as an individual navigator test instead of within the JUnit test suite?
Comment 2 Paul Slauenwhite CLA 2010-05-27 15:27:26 EDT
(In reply to comment #1)
> Paul, I recall seeing a test case covering this scenario in the Navigator test
> suite but cannot remember it on top of my head. If no such navigator test
> exists, shouldn't this test step be written as an individual navigator test
> instead of within the JUnit test suite?

The only test suite that was removed was /org.eclipse.hyades.test.ui.navigator.tests/manual/Test.UI.TestNavigator_bugzilla_160861.testsuite:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=308686#c2
Comment 3 Paul Slauenwhite CLA 2010-05-27 15:49:13 EDT
This is working as designed since name in the editor is the logical name (not the resource name), which is limited to the conditions of org.eclipse.hyades.test.ui.util.TestUIUtil.isLogicalNameValid(String), of which org.eclipse.hyades.ui.util.LocationValidator.validateEMFSegment(String) does not limit the !, *, (, and ) characters.  Note, the wizard validates the resource and logical names since the same name is used to create the file/logical name.
Comment 4 Ben Chen CLA 2010-06-01 15:02:21 EDT
(In reply to comment #3)
> This is working as designed since name in the editor is the logical name (not
> the resource name), which is limited to the conditions of
> org.eclipse.hyades.test.ui.util.TestUIUtil.isLogicalNameValid(String), of which
> org.eclipse.hyades.ui.util.LocationValidator.validateEMFSegment(String) does
> not limit the !, *, (, and ) characters.  Note, the wizard validates the
> resource and logical names since the same name is used to create the
> file/logical name.

Closing