Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345133 - [client] [Chrome] Test failures in textMateStyler
Summary: [client] [Chrome] Test failures in textMateStyler
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.2   Edit
Assignee: Mark Macdonald CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-09 09:02 EDT by Mark Macdonald CLA
Modified: 2011-09-01 11:42 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Macdonald CLA 2011-05-09 09:02:41 EDT 
Using Chrome:

 [failed] test TextMateStyler style one line (0.018s)
  AssertionError: 4 regions - expected: [4], actual: [5].
 Stack Trace:
Error
    at new AssertionError (http://localhost:8080/js/assert.js:25:13)
    at Object.equal (http://localhost:8080/js/assert.js:73:10)
    at http://localhost:8080/js-tests/textMateStyler/testcase.js:105:10
    at http://localhost:8080/js-tests/textMateStyler/testcase.js:40:5
    at _run (http://localhost:8080/js/test.js:67:25)
    at Object.run (http://localhost:8080/js/test.js:219:11)
    at http://localhost:8080/js-tests/textMateStyler/test.html:22:14
test.js:260 

[failed] test TextMateStyler style multiple lines (0.02s)
  AssertionError: line0 has 1 region - expected: [1], actual: [2].
 Stack Trace:
Error
    at new AssertionError (http://localhost:8080/js/assert.js:25:13)
    at Object.equal (http://localhost:8080/js/assert.js:73:10)
    at http://localhost:8080/js-tests/textMateStyler/testcase.js:124:10
    at http://localhost:8080/js-tests/textMateStyler/testcase.js:40:5
    at _run (http://localhost:8080/js/test.js:67:25)
    at Object.run (http://localhost:8080/js/test.js:219:11)
    at http://localhost:8080/js-tests/textMateStyler/test.html:22:14
Comment 1 Mark Macdonald CLA 2011-05-09 11:48:09 EDT 
The styler test case was poking at the editor DOM nodes. It broke due to a recent editor fix (Bug 334910 or Bug 342973 ?) that added more DOM nodes. Changed to not rely on the editor internals, so it should be less brittle now.

http://git.eclipse.org/c/e4/org.eclipse.orion.client.git/commit/?id=6326e642f594fea53c2af6155a3e8652bfe7ae5b