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.

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

webUIMarkdownEditor/markdownFile.feature:26 test is flaky #5456

Closed
saw-jan opened this issue Jul 6, 2021 · 4 comments · Fixed by #5664
Closed

webUIMarkdownEditor/markdownFile.feature:26 test is flaky #5456

saw-jan opened this issue Jul 6, 2021 · 4 comments · Fixed by #5664
Labels

Comments

@saw-jan
Copy link
Member

saw-jan commented Jul 6, 2021

test: webUIMarkdownEditor/markdownFile.feature:26
scenario: append new content in a markdown file

CI log https://drone.owncloud.com/owncloud/web/17015/10/14
(including the CI log as older CI logs will be archived)

Then as "Alice" the file "simple.md" should have the content "simple markdown file new content added"
--
1614 | ✖ failed
1615 | AssertionError [ERR_ASSERTION]: File should have content "simple markdown file new content added", but found "simple markdown file"
1616 | + expected - actual
1617 |  
1618 | -simple markdown file
1619 | +simple markdown file new content added

Possible cause: TBD

@saw-jan saw-jan added the QA:team label Jul 6, 2021
@SwikritiT
Copy link
Contributor

This issue still exists

CI LOG: https://drone.owncloud.com/owncloud/web/18037/12/14

https://drone.owncloud.com/owncloud/web/17972/12/14

✔ And the user closes the markdown editor using the webUI # tests/acceptance/stepDefinitions/markdownEditorContext.js:32
--
3433 | ✖ Then as "Alice" the file "simple.md" should have the content "simple markdown file new content added" # tests/acceptance/stepDefinitions/webdavContext.js:227
3434 | AssertionError [ERR_ASSERTION]: File should have content "simple markdown file new content added", but found "simple markdown file"
3435 | + expected - actual
3436 |  
3437 | -simple markdown file
3438 | +simple markdown file new content added

@sakshamgurung
Copy link
Contributor

sakshamgurung commented Aug 6, 2021

After running the scenario is a loop, this might be the possible reason for failure:

  • After saving the .md file this error pops up
  • env: local with oc10 and web

Screenshot from 2021-08-06 17-15-15

@saw-jan
Copy link
Member Author

saw-jan commented Aug 12, 2021

@disablePreviews tag can be a possible solution
#5652 (comment)

@sakshamgurung
Copy link
Contributor

@saw-jan Yes, It is work for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants