2012-07-19 00:11:26 +00:00
|
|
|
/**
|
|
|
|
* VisualEditor data model TransactionProcessor tests.
|
2012-07-19 21:25:16 +00:00
|
|
|
*
|
2012-07-19 00:11:26 +00:00
|
|
|
* @copyright 2011-2012 VisualEditor Team and others; see AUTHORS.txt
|
|
|
|
* @license The MIT License (MIT); see LICENSE.txt
|
|
|
|
*/
|
|
|
|
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
QUnit.module( 've.dm.TransactionProcessor' );
|
2012-05-01 02:06:20 +00:00
|
|
|
|
|
|
|
/* Tests */
|
|
|
|
|
2012-10-23 00:28:37 +00:00
|
|
|
QUnit.test( 'protection against double application', 3, function ( assert ) {
|
|
|
|
var tx,
|
|
|
|
testDocument = new ve.dm.Document( ve.dm.example.data );
|
|
|
|
tx = new ve.dm.Transaction();
|
|
|
|
tx.pushRetain( 1 );
|
|
|
|
tx.pushReplace( [], ['H', 'e', 'l', 'l', 'o' ] );
|
|
|
|
assert.throws(
|
|
|
|
function () {
|
|
|
|
ve.dm.TransactionProcessor.rollback( testDocument, tx );
|
|
|
|
},
|
|
|
|
Error,
|
|
|
|
'exception thrown when trying to rollback an uncommitted transaction'
|
|
|
|
);
|
|
|
|
ve.dm.TransactionProcessor.commit( testDocument, tx );
|
|
|
|
assert.throws(
|
|
|
|
function () {
|
|
|
|
ve.dm.TransactionProcessor.commit( testDocument, tx );
|
|
|
|
},
|
|
|
|
Error,
|
|
|
|
'exception thrown when trying to commit an already-committed transaction'
|
|
|
|
);
|
|
|
|
ve.dm.TransactionProcessor.rollback( testDocument, tx );
|
|
|
|
assert.throws(
|
|
|
|
function () {
|
|
|
|
ve.dm.TransactionProcessor.rollback( testDocument, tx );
|
|
|
|
},
|
|
|
|
Error,
|
|
|
|
'exception thrown when trying to roll back a transaction that has already been rolled back'
|
|
|
|
);
|
|
|
|
} );
|
|
|
|
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
QUnit.test( 'commit/rollback', function ( assert ) {
|
2012-08-24 02:06:36 +00:00
|
|
|
var i, key, originalData, originalDoc, msg, testDocument, tx, expectedData, expectedDocument,
|
2012-10-06 00:34:12 +00:00
|
|
|
bold = ve.dm.example.createAnnotation( ve.dm.example.bold ),
|
|
|
|
italic = ve.dm.example.createAnnotation( ve.dm.example.italic ),
|
|
|
|
underline = ve.dm.example.createAnnotation( ve.dm.example.underline ),
|
2012-08-02 18:46:13 +00:00
|
|
|
cases = {
|
2012-05-14 21:31:32 +00:00
|
|
|
'no operations': {
|
|
|
|
'calls': [],
|
Make use of new jshint options
* Restricting "camelcase":
No changes, we were passing all of these already
* Explicitly unrestricting "forin" and "plusplus"
These are off by default in node-jshint, but some distro of jshint
and editors that use their own wrapper around jshint instead of
node-jshint (Eclipse?) may have different defaults. Therefor
setting them to false explicitly. This also serves as a reminder
for the future so we'll always know we don't pass that, in case
we would want to change that.
* Fix order ("quotemark" before "regexp")
* Restricting "unused"
We're not passing all of this, which is why I've set it to false
for now. But I did put it in .jshintrc as placeholder.
I've fixed most of them, there's some left where there is no clean
solution.
* While at it fix a few issues:
- Unused variables ($target, $window)
- Bad practices (using jQuery context for find instead of creation)
- Redundant /*global */ comments
- Parameters that are not used and don't have documentation either
- Lines longer than 100 chars @ 4 spaces/tab
* Note:
- ve.ce.Surface.prototype.onChange takes two arguments but never
uses the former. And even the second one can be null/undefined.
Aside from that, the .change() function emits
another event for the transaction already. Looks like this
should be refactored a bit, two more separated events probably
or one that is actually used better.
- Also cleaned up a lot of comments, some of which were missing,
others were incorrect
- Reworked the contentChange event so we are no longer using the
word new as an object key; expanded a complex object into multiple
arguments being passed through the event to make it easier to work
with and document
Change-Id: I8490815a508c6c379d5f9a743bb4aefd14576aa6
2012-08-07 06:02:18 +00:00
|
|
|
'expected': function () {}
|
2012-05-14 21:31:32 +00:00
|
|
|
},
|
|
|
|
'retaining': {
|
|
|
|
'calls': [['pushRetain', 38]],
|
Make use of new jshint options
* Restricting "camelcase":
No changes, we were passing all of these already
* Explicitly unrestricting "forin" and "plusplus"
These are off by default in node-jshint, but some distro of jshint
and editors that use their own wrapper around jshint instead of
node-jshint (Eclipse?) may have different defaults. Therefor
setting them to false explicitly. This also serves as a reminder
for the future so we'll always know we don't pass that, in case
we would want to change that.
* Fix order ("quotemark" before "regexp")
* Restricting "unused"
We're not passing all of this, which is why I've set it to false
for now. But I did put it in .jshintrc as placeholder.
I've fixed most of them, there's some left where there is no clean
solution.
* While at it fix a few issues:
- Unused variables ($target, $window)
- Bad practices (using jQuery context for find instead of creation)
- Redundant /*global */ comments
- Parameters that are not used and don't have documentation either
- Lines longer than 100 chars @ 4 spaces/tab
* Note:
- ve.ce.Surface.prototype.onChange takes two arguments but never
uses the former. And even the second one can be null/undefined.
Aside from that, the .change() function emits
another event for the transaction already. Looks like this
should be refactored a bit, two more separated events probably
or one that is actually used better.
- Also cleaned up a lot of comments, some of which were missing,
others were incorrect
- Reworked the contentChange event so we are no longer using the
word new as an object key; expanded a complex object into multiple
arguments being passed through the event to make it easier to work
with and document
Change-Id: I8490815a508c6c379d5f9a743bb4aefd14576aa6
2012-08-07 06:02:18 +00:00
|
|
|
'expected': function () {}
|
2012-05-14 21:31:32 +00:00
|
|
|
},
|
|
|
|
'annotating content': {
|
|
|
|
'calls': [
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'set', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'set', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'clear', italic],
|
|
|
|
['pushStartAnnotating', 'set', bold],
|
|
|
|
['pushStartAnnotating', 'set', underline],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'clear', italic],
|
|
|
|
['pushStopAnnotating', 'set', bold],
|
|
|
|
['pushStopAnnotating', 'set', underline]
|
2012-05-14 21:31:32 +00:00
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-10-06 00:34:12 +00:00
|
|
|
data[1] = ['a', new ve.AnnotationSet( [ bold ] )];
|
|
|
|
data[2] = ['b', new ve.AnnotationSet( [ bold ] )];
|
|
|
|
data[3] = ['c', new ve.AnnotationSet( [ bold, underline ] )];
|
2012-05-14 21:31:32 +00:00
|
|
|
}
|
|
|
|
},
|
2012-05-15 00:16:18 +00:00
|
|
|
'annotating content and leaf elements': {
|
|
|
|
'calls': [
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 38],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'set', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 2],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'set', bold]
|
2012-05-15 00:16:18 +00:00
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-10-06 00:34:12 +00:00
|
|
|
data[38] = ['h', new ve.AnnotationSet( [ bold ] )];
|
|
|
|
data[39].annotations = new ve.AnnotationSet( [ bold ] );
|
2012-05-15 00:16:18 +00:00
|
|
|
}
|
|
|
|
},
|
|
|
|
'using an annotation method other than set or clear throws an exception': {
|
|
|
|
'calls': [
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'invalid-method', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'invalid-method', bold]
|
2012-05-15 00:16:18 +00:00
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-05-15 00:16:18 +00:00
|
|
|
},
|
2012-06-08 05:08:49 +00:00
|
|
|
'annotating branch opening element throws an exception': {
|
2012-05-15 00:16:18 +00:00
|
|
|
'calls': [
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'set', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'set', bold]
|
2012-05-15 00:16:18 +00:00
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-06-08 05:08:49 +00:00
|
|
|
},
|
|
|
|
'annotating branch closing element throws an exception': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 4],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'set', bold],
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'set', bold]
|
2012-06-08 05:08:49 +00:00
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-05-15 00:16:18 +00:00
|
|
|
},
|
|
|
|
'setting duplicate annotations throws an exception': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 2],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'set', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'set', bold]
|
2012-05-15 00:16:18 +00:00
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-05-15 00:16:18 +00:00
|
|
|
},
|
|
|
|
'removing non-existent annotations throws an exception': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStartAnnotating', 'clear', bold],
|
2012-05-15 00:16:18 +00:00
|
|
|
['pushRetain', 1],
|
2012-10-06 00:34:12 +00:00
|
|
|
['pushStopAnnotating', 'clear', bold]
|
2012-05-15 00:16:18 +00:00
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-05-15 00:16:18 +00:00
|
|
|
},
|
2012-05-14 21:31:32 +00:00
|
|
|
'changing, removing and adding attributes': {
|
|
|
|
'calls': [
|
|
|
|
['pushReplaceElementAttribute', 'level', 1, 2],
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 12],
|
2012-05-14 21:31:32 +00:00
|
|
|
['pushReplaceElementAttribute', 'style', 'bullet', 'number'],
|
|
|
|
['pushReplaceElementAttribute', 'test', undefined, 'abcd'],
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 27],
|
2012-05-14 21:31:32 +00:00
|
|
|
['pushReplaceElementAttribute', 'html/src', 'image.png', undefined]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-14 21:31:32 +00:00
|
|
|
data[0].attributes.level = 2;
|
2012-06-08 05:08:49 +00:00
|
|
|
data[12].attributes.style = 'number';
|
|
|
|
data[12].attributes.test = 'abcd';
|
|
|
|
delete data[39].attributes['html/src'];
|
2012-05-14 21:31:32 +00:00
|
|
|
}
|
|
|
|
},
|
|
|
|
'changing attributes on non-element data throws an exception': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
|
|
|
['pushReplaceElementAttribute', 'foo', 23, 42]
|
|
|
|
],
|
2012-08-08 17:48:53 +00:00
|
|
|
'exception': Error
|
2012-05-14 21:31:32 +00:00
|
|
|
},
|
2012-05-31 13:23:44 +00:00
|
|
|
'inserting text': {
|
2012-05-16 20:33:27 +00:00
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
|
|
|
['pushReplace', [], ['F', 'O', 'O']]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-16 20:33:27 +00:00
|
|
|
data.splice( 1, 0, 'F', 'O', 'O' );
|
|
|
|
}
|
|
|
|
},
|
2012-05-31 13:23:44 +00:00
|
|
|
'removing text': {
|
2012-05-16 20:33:27 +00:00
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
|
|
|
['pushReplace', ['a'], []]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-16 20:33:27 +00:00
|
|
|
data.splice( 1, 1 );
|
|
|
|
}
|
|
|
|
},
|
2012-05-31 13:23:44 +00:00
|
|
|
'replacing text': {
|
2012-05-14 21:31:32 +00:00
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
|
|
|
['pushReplace', ['a'], ['F', 'O', 'O']]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-14 21:31:32 +00:00
|
|
|
data.splice( 1, 1, 'F', 'O', 'O' );
|
|
|
|
}
|
2012-05-15 00:16:18 +00:00
|
|
|
},
|
2012-05-31 13:23:44 +00:00
|
|
|
'inserting mixed content': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 1],
|
|
|
|
['pushReplace', ['a'], ['F', 'O', 'O', {'type':'image'}, {'type':'/image'}, 'B', 'A', 'R']]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-31 13:23:44 +00:00
|
|
|
data.splice( 1, 1, 'F', 'O', 'O', {'type':'image'}, {'type':'/image'}, 'B', 'A', 'R' );
|
|
|
|
}
|
|
|
|
},
|
2012-05-15 00:16:18 +00:00
|
|
|
'converting an element': {
|
|
|
|
'calls': [
|
|
|
|
[
|
|
|
|
'pushReplace',
|
|
|
|
[{ 'type': 'heading', 'attributes': { 'level': 1 } }],
|
|
|
|
[{ 'type': 'paragraph' }]
|
|
|
|
],
|
|
|
|
['pushRetain', 3],
|
|
|
|
['pushReplace', [{ 'type': '/heading' }], [{ 'type': '/paragraph' }]]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-15 00:16:18 +00:00
|
|
|
data[0].type = 'paragraph';
|
|
|
|
delete data[0].attributes;
|
|
|
|
data[4].type = '/paragraph';
|
|
|
|
}
|
2012-05-15 04:50:33 +00:00
|
|
|
},
|
|
|
|
'splitting an element': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 2],
|
|
|
|
[
|
|
|
|
'pushReplace',
|
|
|
|
[],
|
|
|
|
[{ 'type': '/heading' }, { 'type': 'heading', 'attributes': { 'level': 1 } }]
|
|
|
|
]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-05-15 04:50:33 +00:00
|
|
|
data.splice(
|
|
|
|
2,
|
|
|
|
0,
|
|
|
|
{ 'type': '/heading' },
|
|
|
|
{ 'type': 'heading', 'attributes': { 'level': 1 } }
|
|
|
|
);
|
|
|
|
}
|
2012-05-16 20:33:27 +00:00
|
|
|
},
|
|
|
|
'merging an element': {
|
|
|
|
'calls': [
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 57],
|
2012-05-16 20:33:27 +00:00
|
|
|
[
|
|
|
|
'pushReplace',
|
|
|
|
[{ 'type': '/paragraph' }, { 'type': 'paragraph' }],
|
|
|
|
[]
|
|
|
|
]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-06-08 05:08:49 +00:00
|
|
|
data.splice( 57, 2 );
|
2012-05-16 20:33:27 +00:00
|
|
|
}
|
2012-06-06 22:34:55 +00:00
|
|
|
},
|
|
|
|
'stripping elements': {
|
|
|
|
'calls': [
|
|
|
|
['pushRetain', 3],
|
|
|
|
[
|
|
|
|
'pushReplace',
|
2012-10-06 00:34:12 +00:00
|
|
|
[['c', [ ve.dm.example.italic ]]],
|
2012-06-06 22:34:55 +00:00
|
|
|
[]
|
|
|
|
],
|
2012-06-08 05:08:49 +00:00
|
|
|
['pushRetain', 6],
|
2012-06-06 22:34:55 +00:00
|
|
|
[
|
|
|
|
'pushReplace',
|
|
|
|
['d'],
|
|
|
|
[]
|
|
|
|
]
|
|
|
|
],
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
'expected': function ( data ) {
|
2012-06-08 05:08:49 +00:00
|
|
|
data.splice( 10, 1 );
|
2012-06-06 22:34:55 +00:00
|
|
|
data.splice( 3, 1 );
|
|
|
|
}
|
2012-05-14 21:31:32 +00:00
|
|
|
}
|
|
|
|
};
|
2012-08-24 02:06:36 +00:00
|
|
|
for ( key in cases ) {
|
|
|
|
for ( i = 0; i < cases[key].calls.length; i++ ) {
|
|
|
|
if ( cases[key].calls[i][0] === 'pushReplace' ) {
|
|
|
|
ve.dm.example.preprocessAnnotations( cases[key].calls[i][1] );
|
|
|
|
ve.dm.example.preprocessAnnotations( cases[key].calls[i][2] );
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-05-15 07:25:27 +00:00
|
|
|
// Generate original document
|
2012-08-02 18:46:13 +00:00
|
|
|
originalData = ve.dm.example.data;
|
|
|
|
originalDoc = new ve.dm.Document( originalData );
|
2012-05-14 21:31:32 +00:00
|
|
|
// Run tests
|
2012-08-02 18:46:13 +00:00
|
|
|
for ( msg in cases ) {
|
|
|
|
testDocument = new ve.dm.Document( ve.copyArray( originalData ) );
|
|
|
|
tx = new ve.dm.Transaction();
|
|
|
|
for ( i = 0; i < cases[msg].calls.length; i++ ) {
|
2012-05-14 21:31:32 +00:00
|
|
|
tx[cases[msg].calls[i][0]].apply( tx, cases[msg].calls[i].slice( 1 ) );
|
|
|
|
}
|
2012-05-15 00:16:18 +00:00
|
|
|
if ( 'expected' in cases[msg] ) {
|
2012-05-15 07:25:27 +00:00
|
|
|
// Generate expected document
|
2012-08-02 18:46:13 +00:00
|
|
|
expectedData = ve.copyArray( originalData );
|
2012-05-15 07:25:27 +00:00
|
|
|
cases[msg].expected( expectedData );
|
2012-08-02 18:46:13 +00:00
|
|
|
expectedDocument = new ve.dm.Document( expectedData );
|
2012-05-15 07:25:27 +00:00
|
|
|
// Commit
|
|
|
|
ve.dm.TransactionProcessor.commit( testDocument, tx );
|
2012-07-10 19:46:08 +00:00
|
|
|
assert.deepEqual( testDocument.getData(), expectedData, 'commit (data): ' + msg );
|
|
|
|
assert.equalNodeTree(
|
|
|
|
testDocument.getDocumentNode(),
|
|
|
|
expectedDocument.getDocumentNode(),
|
2012-05-15 05:36:06 +00:00
|
|
|
'commit (tree): ' + msg
|
|
|
|
);
|
2012-05-15 07:25:27 +00:00
|
|
|
// Rollback
|
|
|
|
ve.dm.TransactionProcessor.rollback( testDocument, tx );
|
2012-07-10 19:46:08 +00:00
|
|
|
assert.deepEqual( testDocument.getData(), ve.dm.example.data, 'rollback (data): ' + msg );
|
|
|
|
assert.equalNodeTree(
|
|
|
|
testDocument.getDocumentNode(),
|
|
|
|
originalDoc.getDocumentNode(),
|
2012-05-15 05:36:06 +00:00
|
|
|
'rollback (tree): ' + msg
|
|
|
|
);
|
2012-05-14 21:31:32 +00:00
|
|
|
} else if ( 'exception' in cases[msg] ) {
|
|
|
|
/*jshint loopfunc:true */
|
2012-07-10 19:46:08 +00:00
|
|
|
assert.throws(
|
Kranitor #1: On-boarding
'''Kranitor commits''' are commits by Krinkle with his janitor hat on.
Must never contain functional changes mixed with miscellaneous changes.
.gitignore:
* Add .DS_Store to the ignore list so that browsing the directories
on Mac OS X, will not add these files to the list of untracked
files.
* Fix missing newline at end of file
.jshintrc
* raises -> throws
* +module (QUnit.module)
* remove 'Node' (as of node-jshint 1.7.2 this is now part of
'browser:true', as it should be)
Authors:
* Adding myself
MWExtension/VisualEditor.php
* Fix default value of wgVisualEditorParsoidURL to not
point to the experimental instance in WMF Labs.
Issues:
* ve.ce.TextNode:
- Fix TODO: Don't perform a useless clone of an already-jQuerified object.
- Use .html() to set html content instead of encapsulating between
two strings. This is slightly faster but more importantly safer,
and prevents situations where the resulting jQuery collection
actually contains 2 elements instead of 1, thus messing up
what .contents() is iterating over.
* ve.ce.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Document.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.Transaction.test.js
- Fix: ReferenceError: assert is not defined
* ve.dm.TransactionProcessor.test.js
- Fix: ReferenceError: assert is not defined
* ext.visualEditor.viewPageTarget
- Missing dependency on 'mediawiki.Title'
Code conventions / Misc cleanup
* Various JSHint warnings.
* Whitespace
* jQuery(): Use '<tag>' for element creation,
use '<valid><xml/></valid>' for parsing
* Use the default operator instead of ternary when the condition and
first value are the same.
x = foo ? foo : bar; -> x = foo || bar;
Because contrary to some programming language (PHP...), in JS the
default operator does not enforce a boolean result but returns the
original value, hence it being called the 'default' operator, as
opposed to the 'or' operator.
* No need to call addClass() twice, it takes a space-separated list
(jQuery splits by space and adds if needed)
* Use .on( event[, selector], fn ) instead of the deprecated
routers to it such as .bind(), .delegate() and .live().
All these three are now built-in and fully compatible with .on()
* Add 'XXX:' comments for suspicious code that I don't want to change
as part of a clean up commit.
* Remove unused variables (several var x = this; where x was not
used anywhere, possibly from boilerplate copy/paste)
* Follows-up Trevor's commit that converts test suites to the new
QUnit format. Also removed the globals since we no longer use those
any more.
Change-Id: I7e37c9bff812e371c7f65a6fd85d9e2af3e0a22f
2012-07-27 08:43:33 +00:00
|
|
|
function () {
|
2012-05-15 07:25:27 +00:00
|
|
|
ve.dm.TransactionProcessor.commit( testDocument, tx );
|
2012-05-14 21:31:32 +00:00
|
|
|
},
|
|
|
|
cases[msg].exception,
|
|
|
|
'commit: ' + msg
|
|
|
|
);
|
|
|
|
}
|
|
|
|
}
|
2012-05-04 22:50:50 +00:00
|
|
|
} );
|