Last modified: 2014-03-01 02:26:09 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T55721, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53721 - VisualEditor: Inserting some content in a QA script doesn't round-trip (null diff) - failure to inserting CE content into DM somehow?
VisualEditor: Inserting some content in a QA script doesn't round-trip (null ...
Status: ASSIGNED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Editing team bugs – take if you're interested!
: browser-test-bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-03 20:49 UTC by rachelqa99
Modified: 2014-03-01 02:26 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
shows error message (72.17 KB, image/png)
2013-09-03 20:49 UTC, rachelqa99
Details
feature script (757 bytes, application/octet-stream)
2013-09-03 20:52 UTC, rachelqa99
Details

Description rachelqa99 2013-09-03 20:49:50 UTC
Created attachment 13227 [details]
shows error message

I got this message when running a test automation script, that puts in a line of code and tries to save it. It doesn't recognize the new line of code I put in, so it gives this error message upon save. Not sure if you are aware of this particular scenario, which is why I am submitting it.
Comment 1 rachelqa99 2013-09-03 20:52:23 UTC
Created attachment 13228 [details]
feature script
Comment 2 rachelqa99 2013-09-03 20:53:14 UTC
this is running off of test2
Comment 3 rachelqa99 2013-09-03 21:03:27 UTC
Possibly a situation caused by bug 53360 which is marked as resolved.
Comment 4 James Forrester 2013-09-03 22:20:05 UTC
(In reply to comment #0)
> Created attachment 13227 [details]
> shows error message
> 
> I got this message when running a test automation script,

Yes, this is the normal (you tried to get the diff but as far as we're concerned, no changes were made to the page).

> that puts in a line of code and tries to save it. It doesn't recognize the new
> line of code I put in, so it gives this error message upon save.

What "line of code" does it insert? Where does it insert it? On what page? On what wiki? Can you reproduce manually or does this only happen with the script?

(In reply to comment #3)
> Possibly a situation caused by bug 53360 which is marked as resolved.

The fix for bug 53360 is running on test2 right now, so almost certainly not related to that bug.
Comment 5 rachelqa99 2013-09-04 13:55:46 UTC
I am adding the line of text  This is a test  at the beginning of the page into the Selenium user's homepage on test2.  There is a bunch of already existing text below it, I am just trying to add a new line at the begining of the page, "This is a test" and hit Return key for a new line.  I can't seem to reproduce it manually.
Comment 6 Chris McMahon 2013-09-04 15:45:00 UTC
I think this might be a real bug, but I think it might take some particular piece of hardware to trigger it. 

WebDriver/Selenium has two ways of sending a newline to the browser

:enter
:return

":return" is intended to emulate the "Return" key on a normal keyboard; ":enter" is intended to emulate the "Enter" key on a numeric keypad.

In this test, when we type a string plus :return in VE, the diff view is correct, and the test would pass.  But when the test sends a string plus :enter in VE, VE does not recognize that the contents of the edit page has changed. 

I tried to reproduce this issue manually using a Macintosh USB keyboard on a Macbook Pro laptop but both the Return key and Enter key on that particular piece of hardware worked correctly with VE. 

I suggest trying out Return (on the keyboard) and Enter (on the numeric keypad) on a few keyboards to make sure we don't see this in the wild.
Comment 7 Chris McMahon 2013-09-04 16:37:53 UTC
A clarification about each case we tried in the automated test: 

* type a string and the string is shown correctly in CE and DM
* type a string and :return and both the string and the empty line are shown correctly in CE and DM

* but type a string and :enter and CE correctly shows *both* the string *and* the empty line but DM shows *neither* the string nor the empty line.  That is, the :enter command is recognized in CE but seems to have the effect of making all the changes in CE invisible to DM.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links