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

T/ckeditor5 engine/404: Handle spaces as output by new DOM > View conversion. #81

Merged
merged 10 commits into from
Sep 18, 2019

Conversation

jodator
Copy link
Contributor

@jodator jodator commented Aug 14, 2019

Suggested merge commit message (convention)

Tests: Update test fixtures to reflect changes in DOM to View conversion.


Additional information

@coveralls
Copy link

coveralls commented Aug 14, 2019

Pull Request Test Coverage Report for Build 318

  • 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 312: 0.0%
Covered Lines: 208
Relevant Lines: 208

💛 - Coveralls

@jodator
Copy link
Contributor Author

jodator commented Sep 6, 2019

@Reinmar So the PFO tests are fixed and the solution shold be OK now.

@Reinmar
Copy link
Member

Reinmar commented Sep 17, 2019

You forgot to push something here, because those tests still use the standard assertions.

Copy link
Member

@Reinmar Reinmar left a comment

Choose a reason for hiding this comment

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

Expecting new assertion util to be used here.

@jodator
Copy link
Contributor Author

jodator commented Sep 17, 2019

This might have gone missing during the fight. I've readded this to the tests.

@Reinmar Reinmar merged commit 051743e into master Sep 18, 2019
@Reinmar Reinmar deleted the t/ckeditor5-engine/404 branch September 18, 2019 06:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants