Opened 10 years ago

Closed 9 years ago

#16928 closed Task (fixed)

Bleeding tests

Reported by: Tomasz Jakut Owned by: Tomasz Jakut
Priority: Normal Milestone: CKEditor 4.7.0
Component: General Version: 4.7.0
Keywords: Cc:

Description

During running unit tests in sequence there are some bleeding, but still passing ones:

  • resourcemanager.js:211 Uncaught Error: [CKEDITOR.resourceManager.load] Resource name "default" was not found at "http://tests.ckeditor.dev:1030/apps/ckeditor/styles.js"
  • focusmanager.js:164 Uncaught TypeError: Cannot read property '$' of undefined
  • lang.js:54 Uncaught TypeError: Cannot set property 'dir' of undefined

These errors appear randomly, in different tests and don't show up if unit tests are run separately. Probably the issue is connected with the way in which Bender runs tests and causes that DOM is not always ready when the test runs.

Change History (6)

comment:1 Changed 10 years ago by Tomasz Jakut

Owner: set to Tomasz Jakut
Status: new → assigned

comment:2 Changed 10 years ago by Tomasz Jakut

focusmanager.js:164 Uncaught TypeError: Cannot read property '$' of undefined is caused by #16935.

comment:3 Changed 10 years ago by Tomasz Jakut

Status: assigned → review

The other errors were easily resolved using some async magic (I've added wait in 3 places). It's also worth noting that these tests were bleeding only when run from Bender Dashboard, run directly behaved correcly.

Pushed ​branch:t/16928.

comment:4 Changed 9 years ago by kkrzton

Status: review → review_passed

Looks good.

comment:5 Changed 9 years ago by kkrzton

Milestone: → CKEditor 4.7.0

comment:6 Changed 9 years ago by kkrzton

Resolution: → fixed
Status: review_passed → closed

Fixed with ​cf4eaff.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy