Skip to content

Added functional tests for editor: tag editor, image uploads, post settings#2395

Merged
ErisDS merged 1 commit intoTryGhost:masterfrom
m-tse:mattse#2273
Mar 14, 2014
Merged

Added functional tests for editor: tag editor, image uploads, post settings#2395
ErisDS merged 1 commit intoTryGhost:masterfrom
m-tse:mattse#2273

Conversation

@m-tse
Copy link
Contributor

@m-tse m-tse commented Mar 14, 2014

Added functional tests for editor: tag editor, image uploads, post settings

references #2273

  • test tag creation and tag deletion
  • tests image uploader appears after typing ![]() in editor
  • tests image URL matches url inside ![](url)
  • tests all input elements of post settings menu

…ttings

references #2273
- test tag creation and tag deletion
- tests image uploader appears after typing `![]()` in editor
- tests image URL matches url inside `![](url)`
- tests all input elements of post settings menu
@ErisDS
Copy link
Member

ErisDS commented Mar 14, 2014

Love these PRs 👍

ErisDS added a commit that referenced this pull request Mar 14, 2014
Added functional tests for editor: tag editor, image uploads, post settings
@ErisDS ErisDS merged commit caa729e into TryGhost:master Mar 14, 2014
@halfdan
Copy link
Contributor

halfdan commented Mar 16, 2014

This PR seems to have introduced an intermittent functional test error: https://travis-ci.org/halfdan/Ghost/jobs/20900525

@m-tse
Copy link
Contributor Author

m-tse commented Mar 17, 2014

@halfdan for https://travis-ci.org/halfdan/Ghost/jobs/20900525, I see a race condition that I will fix. I'll also look through the rest of my code to see if I can find any more similar race conditions.

In #2409, you mentioned errors(plural). Did the functional test fail on any more of the tests besides the test in:

In /home/travis/build/halfdan/Ghost/core/test/functional/admin/editor_test.js:274
  Post settings menu
    assertVisible: delete post button should be visible for saved drafts

?

tigefa4u pushed a commit to tigefa4u/Ghost that referenced this pull request Aug 3, 2022
Co-authored-by: Renovate Bot <bot@renovateapp.com>
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.

3 participants