Skip to content

Conversation

@Mati365
Copy link
Member

@Mati365 Mati365 commented May 16, 2025

Suggested merge commit message (convention)

Tests: Stabilize license check related test. Closes #486


Additional information

Fixes https://app.circleci.com/pipelines/github/ckeditor/ckeditor5-angular/845/workflows/4c9e77d9-5f91-40d9-9fe9-f2af9b466a1a/jobs/1708

Chrome 136.0.0.0 (Linux x86_64) ERROR
  An error was thrown in afterAll
  CKEditorError: invalid-license-key
  Read more: https://ckeditor.com/docs/ckeditor5/latest/support/error-codes.html#error-invalid-license-key
  error properties: Object({ context: undefined, data: undefined })
  CKEditorError: invalid-license-key
  Read more: https://ckeditor.com/docs/ckeditor5/latest/support/error-codes.html#error-invalid-license-key
      at apply (http://localhost:9876/_karma_webpack_/webpack:/node_modules/@ckeditor/ckeditor5-core/dist/index.js:2489:15)
      at apply (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:2367:41)
      at _ZoneDelegate.invokeTask (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:402:31)
      at ProxyZoneSpec.onInvokeTask (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone-testing.js:304:39)
      at _ZoneDelegate.invokeTask (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:401:60)
      at Zone.runTask (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:173:47)
      at call (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:483:34)
      at apply (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:472:48)
      at data.args.<computed> (http://localhost:9876/_karma_webpack_/webpack:/node_modules/zone.js/fesm2015/zone.js:2347:32)

@Mati365 Mati365 requested a review from arkflpc May 16, 2025 08:55
@coveralls
Copy link

Pull Request Test Coverage Report for Build 88b83e7f-dce2-4bb8-9d74-807d9290e20e

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 397adb2e-85be-4c97-a61b-55f5e77f1f51: 0.0%
Covered Lines: 163
Relevant Lines: 163

💛 - Coveralls

Copy link

@arkflpc arkflpc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create a ticket before merge.

@Mati365 Mati365 merged commit 2767269 into master May 16, 2025
5 checks passed
@Mati365 Mati365 deleted the ck/fix-license-check branch May 16, 2025 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky license check test

4 participants