Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 349607 - [Test] ExtractSchemaTextFieldModifyListenerTest is failing under Linux
Summary: [Test] ExtractSchemaTextFieldModifyListenerTest is failing under Linux
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: WTP Incubator (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Dimitar Donchev CLA
QA Contact: Dimitar Tenev CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-16 13:59 EDT by Stanislav Nichev CLA
Modified: 2021-11-18 16:19 EST (History)
1 user (show)

See Also:


Attachments
patch fixing the test (2.12 KB, patch)
2011-06-16 14:05 EDT, Stanislav Nichev CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stanislav Nichev CLA 2011-06-16 13:59:48 EDT
Build Identifier: 

Hi Colleagues,

Test ExtractSchemaTextFieldModifyListenerTest is failing when executed under linux. 
There is a problem with one of the file separators.

Regards,
Stanislav

Reproducible: Always
Comment 1 Stanislav Nichev CLA 2011-06-16 14:05:02 EDT
Created attachment 198139 [details]
patch fixing the test

Hi Colleagues,

I'm attaching a patch fixing the test.

Regards,
Stanislav
Comment 2 Dimitar Donchev CLA 2011-06-17 08:14:45 EDT
pls use File.separator in stead of "/" or "\\".
Tathc applied, all tests are passing.
Comment 3 Stanislav Nichev CLA 2011-06-17 08:19:35 EDT
Hi Dimitar,

Actually I was thinking whether or not I should open a "task" for this - to check all the tests and code and refactor it to use the File.separator.

This probably will be the best thing to do in order to avoid further issues with separators and such.

That's the reason I didn't do it here. The second reason was that I couldn't find the constant :)

Regards,
Stanislav
Comment 4 Dimitar Donchev CLA 2011-06-17 09:23:09 EDT
Yep , maybe a good idea, but as Dimitar Teneve mentioned , there might be some slashes related to xml schema namespace construction, so it will be a big task to differentiate all cases. 

Since the editor almost doesn't work with files, why don't we check only the code related to file operations for such constants? :)
Or if not, since you are running under linux, and I at home am too, we should catch all the bugs related to this issue anyway..

Regards,
Dido