Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

IE11/Edge throws "Permission denied" error for "should be synced with editorData property" unit test #72

Closed
f1ames opened this issue Nov 20, 2019 · 6 comments · Fixed by #215
Closed

IE11/Edge throws "Permission denied" error for "should be synced with editorData property" unit test #72

f1ames opened this issue Nov 20, 2019 · 6 comments · Fixed by #215
Assignees
@sculpt0r
Labels
browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. status:confirmed An issue confirmed by the development team. type:failingtest A failing test.
Milestone
2.3.0

Comments

@f1ames
Copy link
Contributor

f1ames commented Nov 20, 2019

Are you reporting a feature request or a bug?

Failing test

Provide detailed reproduction steps (if any)

See https://travis-ci.org/ckeditor/ckeditor4-angular/builds/614548847#L290.

Expected result

Test is passing without errors.

Actual result

Tests fails on fixture.detectChanges() line throwing Permission denied error.

Other details

  • Browser: IE11 / Edge
  • OS: Windows 10
  • Integration version: 1.0.0
  • CKEditor version: 4.13
  • Installed CKEditor plugins: -
f1ames added a commit that referenced this issue Nov 20, 2019
@f1ames
Tests: Ignore one failing test (see #72).
de9d036
@jacekbogdanski jacekbogdanski added status:confirmed An issue confirmed by the development team. type:failingtest A failing test. and removed confirmed labels Feb 7, 2020
ezintz added a commit to ezintz/ckeditor4-angular that referenced this issue Apr 11, 2020
@ezintz
Merge remote-tracking branch 'upstream/master'
cd45929 
* upstream/master: (124 commits)
  Bump acorn from 6.3.0 to 6.4.1
  Update Travis config to latest spec.
  Disable Travis caching.
  Remove install unnecessary after bumping dependency.
  Bump Protractor dependency which contains newer webdriver-manager package.
  Add CHANGELOG.md to build process.
  Changelog updated.
  Started 1.1.1 development.
  Updated package-lock version to 1.1.0.
  Updated package version to 1.1.0.
  Updated default CDN CKEditor 4 dependency to 4.14.0.
  Updated deps license entry.
  Updated license header to 2020.
  Started 1.0.2 development.
  Updated default CDN CKEditor 4 dependency to 4.13.1.
  Started 1.0.1 developemnt.
  Changelog adjustments.
  NPM package files updated.
  Tests: Removed 'debugger' leftover. [skip ci]
  Tests: Ignore one failing test (see ckeditor#72).
  ...
@Dumluregn Dumluregn added this to the 1.3.0 milestone Jul 22, 2020
@Dumluregn Dumluregn added the browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. label Jul 22, 2020
@f1ames
Copy link
Contributor Author

f1ames commented Jul 31, 2020

Let's check quickly if something can be done and if not ignore failing test on IE11.

@Dumluregn Dumluregn removed this from the 1.2.3 milestone Aug 17, 2020
@Dumluregn Dumluregn mentioned this issue Jan 19, 2021
Merged
@Dumluregn Dumluregn mentioned this issue Feb 5, 2021
Merged
@Dumluregn Dumluregn mentioned this issue Mar 18, 2021
Merged
@jacekbogdanski
Copy link
Member

Looks like this issue breaks CI. Maybe it's a good idea to disable Windows-based browsers until it's resolved?

@Dumluregn
Copy link
Collaborator

Dumluregn commented Mar 19, 2021 •
edited
Loading

BTW fixing tests for IE11 (#72 and #74) is pointless until we fix the built version for IE11 (#169) (perhaps also #181?).

@f1ames f1ames added the status:blocked An issue which development is blocked by another issue (internal or external one). label Mar 29, 2021
@Dumluregn
Copy link
Collaborator

We have #169 fixed, so this issue is no longer blocked.

@Dumluregn Dumluregn removed the status:blocked An issue which development is blocked by another issue (internal or external one). label May 14, 2021
@sculpt0r sculpt0r self-assigned this Aug 13, 2021
@sculpt0r
Copy link
Contributor

🤔 no errors after removing IE prevention in test: https://app.travis-ci.com/github/ckeditor/ckeditor4-angular/builds/235396480

@jacekbogdanski jacekbogdanski mentioned this issue Aug 25, 2021
Merged
@CKEditorBot
Copy link
Collaborator

Closed in #215

@jacekbogdanski jacekbogdanski added this to the Next milestone Aug 25, 2021
@jacekbogdanski jacekbogdanski removed this from the Next milestone Nov 22, 2021
@jacekbogdanski jacekbogdanski added this to the 2.3.0 milestone Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sculpt0r sculpt0r

Labels
browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. status:confirmed An issue confirmed by the development team. type:failingtest A failing test.
Projects
None yet
Milestone
2.3.0
Development

Successfully merging a pull request may close this issue.

Removed IE test prevention ckeditor/ckeditor4-angular
5 participants
@f1ames @sculpt0r @jacekbogdanski @Dumluregn @CKEditorBot