Henning Snater has uploaded a new change for review. https://gerrit.wikimedia.org/r/94139
Change subject: Fixed Reference and Statement QUnit tests ...................................................................... Fixed Reference and Statement QUnit tests Using property id strings instead of numbers in the Reference and Statement QUnit tests. Change-Id: I9a16d071e590d66f15fe727bfe9ebbd916aa2282 --- M lib/resources/wikibase.datamodel/wikibase.PropertyValueSnak.js M lib/tests/qunit/wikibase.datamodel/Wikibase.reference.tests.js M lib/tests/qunit/wikibase.datamodel/wikibase.Statement.tests.js 3 files changed, 41 insertions(+), 54 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase refs/changes/39/94139/1 diff --git a/lib/resources/wikibase.datamodel/wikibase.PropertyValueSnak.js b/lib/resources/wikibase.datamodel/wikibase.PropertyValueSnak.js index 356b6af..b101ffd 100644 --- a/lib/resources/wikibase.datamodel/wikibase.PropertyValueSnak.js +++ b/lib/resources/wikibase.datamodel/wikibase.PropertyValueSnak.js @@ -23,7 +23,7 @@ * @since 0.2 * @see https://meta.wikimedia.org/wiki/Wikidata/Data_model#PropertyValueSnak * - * @param {Number} propertyId + * @param {string} propertyId * @param {dv.DataValue} value */ var SELF = wb.PropertyValueSnak = wb.utilities.inherit( 'WbPropertyValueSnak', PARENT, constructor, { diff --git a/lib/tests/qunit/wikibase.datamodel/Wikibase.reference.tests.js b/lib/tests/qunit/wikibase.datamodel/Wikibase.reference.tests.js index 741ed9b..659ca00 100644 --- a/lib/tests/qunit/wikibase.datamodel/Wikibase.reference.tests.js +++ b/lib/tests/qunit/wikibase.datamodel/Wikibase.reference.tests.js @@ -1,29 +1,22 @@ /** - * QUnit tests for wikibase.Reference - * @see https://www.mediawiki.org/wiki/Extension:Wikibase - * - * @since 0.3 - * @ingroup WikibaseLib - * * @licence GNU GPL v2+ - * @author Jeroen De Dauw < [email protected] > * @author Daniel Werner < [email protected] > * @author H. Snater < [email protected] > */ -( function( wb, dv, $, QUnit, undefined ) { +( function( wb, dv, $, QUnit ) { 'use strict'; QUnit.module( 'wikibase.datamodel.Reference', QUnit.newWbEnvironment() ); var snakLists = [ new wb.SnakList( [ - new wb.PropertyNoValueSnak( 9001 ), - new wb.PropertySomeValueSnak( 42 ), - new wb.PropertyValueSnak( 23, new dv.StringValue( '~=[,,_,,]:3' ) ) + new wb.PropertyNoValueSnak( 'P9001' ), + new wb.PropertySomeValueSnak( 'P42' ), + new wb.PropertyValueSnak( 'P23', new dv.StringValue( '~=[,,_,,]:3' ) ) ] ), new wb.SnakList( [] ), - new wb.SnakList( [ new wb.PropertyNoValueSnak( 9001 ) ] ) + new wb.SnakList( [ new wb.PropertyNoValueSnak( 'P9001' ) ] ) ]; QUnit.test( 'constructor', function( assert ) { @@ -78,8 +71,8 @@ var reference = new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 42, new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 9001 ) + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ), 'hash12390213' @@ -97,8 +90,8 @@ new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 42, new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 9001 ) + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ), 'hash12390213' @@ -106,8 +99,8 @@ new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 345, new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 9001 ) + new wb.PropertyValueSnak( 'P345', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ) ) diff --git a/lib/tests/qunit/wikibase.datamodel/wikibase.Statement.tests.js b/lib/tests/qunit/wikibase.datamodel/wikibase.Statement.tests.js index 4356dd8..600e9ee 100644 --- a/lib/tests/qunit/wikibase.datamodel/wikibase.Statement.tests.js +++ b/lib/tests/qunit/wikibase.datamodel/wikibase.Statement.tests.js @@ -1,10 +1,4 @@ /** - * QUnit tests for wikibase.Statement - * @see https://www.mediawiki.org/wiki/Extension:Wikibase - * - * @since 0.4 - * @ingroup WikibaseLib - * * @licence GNU GPL v2+ * @author H. Snater < [email protected] > */ @@ -15,7 +9,7 @@ QUnit.module( 'wikibase.datamodel.Statement', QUnit.newWbEnvironment() ); QUnit.test( 'toJSON', function( assert ) { - var statement = new wb.Statement( new wb.PropertyNoValueSnak( 'p42' ) ); + var statement = new wb.Statement( new wb.PropertyNoValueSnak( 'P42' ) ); assert.ok( statement.equals( wb.Claim.newFromJSON( statement.toJSON() ) ), @@ -23,27 +17,27 @@ ); statement = new wb.Statement( - new wb.PropertyValueSnak( 'p23', new dv.StringValue( '~=[,,_,,]:3' ) ), + new wb.PropertyValueSnak( 'P23', new dv.StringValue( '~=[,,_,,]:3' ) ), new wb.SnakList( [ - new wb.PropertyNoValueSnak( 'p9001' ), - new wb.PropertySomeValueSnak( 'p42' ) + new wb.PropertyNoValueSnak( 'P9001' ), + new wb.PropertySomeValueSnak( 'P42' ) ] ), [ new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p3', new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 'p245' ) + new wb.PropertyValueSnak( 'P3', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P245' ) ] ) ), new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p856', new dv.StringValue( 'another string' ) ), - new wb.PropertySomeValueSnak( 'p97' ) + new wb.PropertyValueSnak( 'P856', new dv.StringValue( 'another string' ) ), + new wb.PropertySomeValueSnak( 'P97' ) ] ) ) @@ -60,67 +54,67 @@ QUnit.test( 'equals()', function( assert ) { var statements = [ - new wb.Statement( new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ) ), + new wb.Statement( new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ) ), new wb.Statement( - new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ), + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ), new wb.SnakList( [ - new wb.PropertyValueSnak( 'p2', new dv.StringValue( 'some string' ) ), - new wb.PropertySomeValueSnak( 'p9001' ) + new wb.PropertyValueSnak( 'P2', new dv.StringValue( 'some string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ), [ new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p3', new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 'p245' ) + new wb.PropertyValueSnak( 'P3', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P245' ) ] ) ), new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p856', new dv.StringValue( 'another string' ) ), - new wb.PropertySomeValueSnak( 'p97' ) + new wb.PropertyValueSnak( 'P856', new dv.StringValue( 'another string' ) ), + new wb.PropertySomeValueSnak( 'P97' ) ] ) ) ], wb.Statement.RANK.PREFERRED ), - new wb.Statement( new wb.PropertyValueSnak( 41, new dv.StringValue( 'string' ) ) ), + new wb.Statement( new wb.PropertyValueSnak( 'P41', new dv.StringValue( 'string' ) ) ), new wb.Statement( - new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ), + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ), new wb.SnakList( [ - new wb.PropertyValueSnak( 2, new dv.StringValue( 'some string' ) ), - new wb.PropertySomeValueSnak( 'p9001' ) + new wb.PropertyValueSnak( 'P2', new dv.StringValue( 'some string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ) ), new wb.Statement( - new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ), + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ), new wb.SnakList( [ - new wb.PropertyValueSnak( 'p2', new dv.StringValue( 'some string' ) ), - new wb.PropertySomeValueSnak( 'p9001' ) + new wb.PropertyValueSnak( 'P2', new dv.StringValue( 'some string' ) ), + new wb.PropertySomeValueSnak( 'P9001' ) ] ), [ new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p3', new dv.StringValue( 'string' ) ), - new wb.PropertySomeValueSnak( 'p245' ) + new wb.PropertyValueSnak( 'P3', new dv.StringValue( 'string' ) ), + new wb.PropertySomeValueSnak( 'P245' ) ] ) ), new wb.Reference( new wb.SnakList( [ - new wb.PropertyValueSnak( 'p123', new dv.StringValue( 'another string' ) ), - new wb.PropertySomeValueSnak( 'p97' ) + new wb.PropertyValueSnak( 'P123', new dv.StringValue( 'another string' ) ), + new wb.PropertySomeValueSnak( 'P97' ) ] ) ) @@ -152,9 +146,9 @@ } ); // Compare claim to statement: - var claim = new wb.Claim( new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ) ), + var claim = new wb.Claim( new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ) ), statement = new wb.Statement( - new wb.PropertyValueSnak( 'p42', new dv.StringValue( 'string' ) ) + new wb.PropertyValueSnak( 'P42', new dv.StringValue( 'string' ) ) ); assert.ok( -- To view, visit https://gerrit.wikimedia.org/r/94139 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9a16d071e590d66f15fe727bfe9ebbd916aa2282 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Henning Snater <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
