mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-16 10:59:56 +00:00
91291d3e98
Previous, reverted attempt:da9b6fffbd. This attempt also includes6037fefbe0, and fixes minor conflicts with other changes. * In normal images, parse relative 'href' attributes instead of expanding them to absolute. This resolves Parsoid generating |link= options for copy-pasted images (T193253). Keep them in the underscore-form to avoid causing dirty diffs like T237040 again. Unlike in the previous attempt, we don't need to be super-careful about the 'resource' attribute, thanks to the Parsoid changes in T108504. * In gallery images stuff, prefix the 'resource' attribute with './', same as normal images do. This causes no functional changes, but it makes updating tests easier, and the consistency is probably good. * Update test examples to also prefix 'resource' and relative 'href' attributes with './', like the real Parsoid does. Bug: T193253 Change-Id: I91131728a87c9406bf069d46d3c94c9a8905a003
55 lines
2.2 KiB
JavaScript
55 lines
2.2 KiB
JavaScript
/*!
|
|
* VisualEditor MW-specific DiffElement tests.
|
|
*
|
|
* @copyright 2011-2020 VisualEditor Team and others; see http://ve.mit-license.org
|
|
*/
|
|
|
|
QUnit.module( 've.ui.DiffElement (MW)', ve.test.utils.mwEnvironment );
|
|
|
|
QUnit.test( 'Diffing', function ( assert ) {
|
|
var i, len,
|
|
fixBase = function ( body ) {
|
|
return '<html><head><base href="' + ve.dm.example.baseUri + '"></head><body>' + body + '</body>';
|
|
},
|
|
cases = [
|
|
{
|
|
msg: 'Change template param',
|
|
oldDoc: fixBase( ve.dm.mwExample.MWTransclusion.blockOpen + ve.dm.mwExample.MWTransclusion.blockContent ),
|
|
newDoc: fixBase( ve.dm.mwExample.MWTransclusion.blockOpenModified + ve.dm.mwExample.MWTransclusion.blockContent ),
|
|
expected:
|
|
'<div class="ve-ui-diffElement-doc-child-change">' +
|
|
( ve.dm.mwExample.MWTransclusion.blockOpenModified + ve.dm.mwExample.MWTransclusion.blockContent )
|
|
// FIXME: Use DOM modification instead of string replaces
|
|
.replace( /#mwt1"/g, '#mwt1" data-diff-action="structural-change" data-diff-id="0"' ) +
|
|
'</div>',
|
|
expectedDescriptions: [
|
|
'<div>visualeditor-changedesc-mwtransclusion</div>' +
|
|
'<div><ul><li>visualeditor-changedesc-changed-diff,1,<span>Hello, <del>world</del><ins>globe</ins>!</span></li></ul></div>'
|
|
]
|
|
},
|
|
{
|
|
msg: 'Changed width of block image',
|
|
oldDoc: fixBase( ve.dm.mwExample.MWBlockImage.html ),
|
|
newDoc: fixBase( ve.dm.mwExample.MWBlockImage.html.replace( 'width="1"', 'width="3"' ) ),
|
|
expected:
|
|
'<div class="ve-ui-diffElement-doc-child-change">' +
|
|
ve.dm.mwExample.MWBlockImage.html
|
|
// FIXME: Use DOM modification instead of string replaces
|
|
.replace( 'width="1"', 'width="3"' )
|
|
.replace( 'href="./Foo"', 'href="' + ve.resolveUrl( './Foo', ve.dm.example.base ) + '"' )
|
|
.replace( 'foobar"', 'foobar" data-diff-action="structural-change" data-diff-id="0"' ) +
|
|
'</div>',
|
|
expectedDescriptions: [
|
|
'<div>visualeditor-changedesc-image-size,' +
|
|
'<del>1visualeditor-dimensionswidget-times2visualeditor-dimensionswidget-px</del>,' +
|
|
'<ins>3visualeditor-dimensionswidget-times2visualeditor-dimensionswidget-px</ins></div>'
|
|
]
|
|
}
|
|
];
|
|
|
|
for ( i = 0, len = cases.length; i < len; i++ ) {
|
|
ve.test.utils.runDiffElementTest( assert, cases[ i ] );
|
|
}
|
|
|
|
} );
|