Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Tests: Adjusted tests to changes in upload plugin #258

Merged
merged 4 commits into from
Jan 7, 2019

Conversation

f1ames
Copy link
Contributor

@f1ames f1ames commented Dec 7, 2018

Suggested merge commit message (convention)

Tests: Adjusted tests to changes in upload plugin.


Additional information

Tests adjusted to changes from https://github.com/ckeditor/ckeditor5-upload/issues/87. Should be closed after or together with ckeditor/ckeditor5-upload#88.

The CI will fail untill ckeditor/ckeditor5-upload#88 will be merged to master so it will be good to merge ckeditor/ckeditor5-upload#88 first and then rerun the CI job for this PR to check if it is green.

@coveralls
Copy link

coveralls commented Dec 17, 2018

Coverage Status

Coverage decreased (-3.9%) to 96.087% when pulling fa309d5 on t/ckeditor5-upload/87 into 54059a1 on master.

@f1ames f1ames reopened this Dec 17, 2018
@f1ames
Copy link
Contributor Author

f1ames commented Dec 17, 2018

Sorry, I messed up by pushing some changes to this branch which should be in another one, but the PR is now back to its previous, correct state.

Copy link
Contributor

@jodator jodator left a comment

Choose a reason for hiding this comment

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

Similar request as in main PR: ckeditor/ckeditor5-upload#88 (review).

tests/imageupload/imageuploadediting.js Show resolved Hide resolved
@jodator jodator assigned jodator and unassigned jodator Jan 7, 2019
@jodator jodator merged commit fe1a7a5 into master Jan 7, 2019
@jodator jodator deleted the t/ckeditor5-upload/87 branch January 7, 2019 10:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants