Catrope has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/405994 )
Change subject: [WIP] Failing test case for losing annotations
......................................................................
[WIP] Failing test case for losing annotations
Change-Id: I5ca3d475760e14f3ece788107f3a002a17729ec9
---
M tests/dm/ve.dm.RebaseServer.test.js
1 file changed, 115 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor
refs/changes/94/405994/1
diff --git a/tests/dm/ve.dm.RebaseServer.test.js
b/tests/dm/ve.dm.RebaseServer.test.js
index aa6b9ec..51b1510 100644
--- a/tests/dm/ve.dm.RebaseServer.test.js
+++ b/tests/dm/ve.dm.RebaseServer.test.js
@@ -282,9 +282,124 @@
[ '1', 'deliver' ],
[ 'server', 'assertHist', 'abXYZ' ]
]
+ },
+ {
+ name: 'Double client-side rebase with
annotation and other preceding transaction',
+ initialData: [
+ { type: 'paragraph' },
+ { type: '/paragraph' },
+ { type: 'internalList' },
+ { type: '/internalList' }
+ ],
+ clients: [ '1', '2' ],
+ ops: [
+ // Client 1 applies a local change that
inserts 'Q'
+ [ '1', 'apply', [
+ [ 'insert', 1, [ 'Q' ], 3 ]
+ ] ],
+
+ // Client 2 submits two changes
+ [ '2', 'apply', [
+ [ 'insert', 1, [ 'a' ], 3 ]
+ ] ],
+ [ '2', 'submit' ],
+ [ '2', 'apply', [
+ [ 'insert', 2, [ 'b' ], 3 ]
+ ] ],
+ [ '2', 'submit' ],
+
+ // Client 1 rebases its local change
over client 2's first change
+ [ '2', 'deliver' ],
+ [ 'server', 'assertHist', 'a' ],
+ [ '1', 'receive' ],
+ [ '1', 'assertHist', 'a/Q!' ],
+ // Client 1 submits its local change
+ [ '1', 'submit' ],
+
+ // Client 1 applies a local change that
introduces an annotation
+ [ '1', 'apply', {
+ start: 1,
+ transactions: [
+ {
+ operations: [
+ { type:
'retain', length: 2 },
+ { type:
'replace', remove: [], insert: [
+
[ 'X', [ 'h123' ] ],
+
[ 'Y', [ 'h123' ] ],
+
[ 'Z', [ 'h123' ] ]
+ ] },
+ { type:
'retain', length: 3 }
+ ],
+ authorId: '1'
+ }
+ ],
+ stores: [
+ {
+ hashes: [
'h123' ],
+ hashStore: {
+ h123: {
+
type: 'annotation',
+
value: {
+
type: 'textStyle/bold'
+
}
+ }
+ }
+ }
+ ],
+ selections: {
+ 1: {
+ type: 'linear',
+ range: {
+ type:
'range',
+ from: 4,
+ to: 4
+ }
+ }
+ }
+ } ],
+ [ '1', 'assert', function ( assert,
client ) {
+ var unsubmitted =
client.getChangeSince( client.sentLength, false );
+ assert.deepEqual(
unsubmitted.stores[ 0 ].hashes, [ 'h123' ], 'h123 is in the store' );
+ } ],
+
+ // Client 1 rebases its local changes
over client 2's second change
+ [ '2', 'deliver' ],
+ [ 'server', 'assertHist', 'ab' ],
+ [ '1', 'receive' ],
+ [ '1', 'assertHist', 'ab/Q?/XYZ!' ],
+ [ '1', 'assert', function ( assert,
client ) {
+ var unsubmitted =
client.getChangeSince( client.sentLength, false );
+ // FIXME this fails. If
uncommitted = client.getChangeSince( client.commitLength, false );
+ // then we expect
uncommitted.stores[1] to contain 'h123', but instead uncommitted.stores[0] does.
+ assert.deepEqual(
unsubmitted.stores[ 0 ].hashes, [ 'h123' ], 'h123 is still in the store after
receiving a foreign change' );
+ } ],
+
+ // Client 1 receives its first change
+ [ '1', 'deliver' ],
+ [ 'server', 'assertHist', 'abQ' ],
+ [ '1', 'receive' ],
+ [ '1', 'assertHist', 'abQ/XYZ!' ],
+ [ '1', 'assert', function ( assert,
client ) {
+ var unsubmitted =
client.getChangeSince( client.sentLength, false );
+ assert.deepEqual(
unsubmitted.stores[ 0 ].hashes, [ 'h123' ], 'h123 is still in the store after
receiving our own change' );
+ } ],
+
+ // Client 1 submits its second change
+ [ '1', 'submit' ],
+ [ '1', 'deliver' ],
+ [ 'server', 'assertHist', 'abQXYZ' ],
+ [ '2', 'assert', function ( assert,
client ) {
+ var lastChange =
client.getChangeSince( client.commitLength - 1, false );
+ assert.deepEqual(
lastChange.stores[ 0 ].hashes, [ 'h123' ], 'h123 is in the store on the other
side' );
+ } ],
+ ]
}
];
+ /*
+
+ */
+
function makeTransaction( doc, data ) {
var i, method,
builder = new ve.dm.TransactionBuilder();
--
To view, visit https://gerrit.wikimedia.org/r/405994
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5ca3d475760e14f3ece788107f3a002a17729ec9
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
