ckeditor4 icon indicating copy to clipboard operation
ckeditor4 copied to clipboard

Unstable PFW test related to filters loading

Open f1ames opened this issue 6 years ago • 7 comments

Type of report

Failing tests

Provide detailed reproduction steps (if any)

Some PFW tests fails randomly, for example see ~https://travis-ci.org/ckeditor/ckeditor4/jobs/585973103~ https://github.com/ckeditor/ckeditor4/issues/3424#issuecomment-937711373. It looks like there might be some inconsistency / race condition when loading PFW filters and so sometimes the filters are not ready when test are executed.

f1ames avatar Sep 17 '19 10:09 f1ames

I think this is the same issue which was mentioned during #835 review (see https://github.com/ckeditor/ckeditor4/pull/3257#pullrequestreview-280849505) regarding the same tests failing on Edge:

image

f1ames avatar Sep 17 '19 15:09 f1ames

Fixed in #3539.

f1ames avatar Nov 21 '19 13:11 f1ames

Reopening since it it still an issue as we seen during recent testing phase.

f1ames avatar Feb 19 '20 11:02 f1ames

They seem to fail on the first test run and only in Firefox. After that, they run smoothly. Unfortunately there is only the first run on CI, so it fails there more regularly, e.g. https://app.travis-ci.com/github/ckeditor/ckeditor4/jobs/541909640 or https://app.travis-ci.com/github/ckeditor/ckeditor4/jobs/541657054.

Comandeer avatar Oct 07 '21 11:10 Comandeer

It seems that Firefox does something weird during script loading and load event does not mean that script is loaded and executed. Now I'm experimenting with using afterscriptexecute event in the script loader and the initial results are promising.

Comandeer avatar Oct 12 '21 16:10 Comandeer

It seems that the issue can be reproduced in Chrome after all. So probably the issue is deeper and connected with our script loader.

Comandeer avatar Nov 08 '21 10:11 Comandeer

That should be fixed as a part of a bigger solution, e.g. #4948

I was able to reproduce the issue (this time on Brave - chrome) only thanks to the browser plugin, which automatically refreshes the tab and stops if certain text is on the page...

All scripts are loaded, the only 404 is for favicon. Some screens, might eventually help: Screenshot 2022-03-30 at 22 39 36 Screenshot 2022-03-30 at 22 40 55 Screenshot 2022-03-30 at 22 40 40

sculpt0r avatar Mar 31 '22 06:03 sculpt0r