User "Krinkle" posted a comment on MediaWiki.r100110.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/100110#c24497
Commit summary:

Qunit coverage for mw.Title.js

Follow up comment on r93702 asking to cover:
  new mw.Title( 'user:NeilK' );

Also added a case conversion test

Comment:

<pre>
+       equal( title.getNamespaceId(), 2 );
+       equal( title.getNamespacePrefix(), 'User:' );
</pre>

That's kinda overlapping with 
[http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/tests/qunit/suites/resources/mediawiki/mediawiki.Title.test.js?view=markup&pathrev=100110#l92
 test('Main text for filename')] which tests all public methods (except for url 
and exists) for a 'normal' title.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to