User "Brion VIBBER" posted a comment on MediaWiki.r89082. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89082#c17388 Commit summary:
Adding or adjusting front-end function and variable documentation per our conventions. Also did some trivial clean up, JSHint fixes and performance optimizations while at it. (per http://www.mediawiki.org/wiki/JavaScript_performance ) * Missing semicolons. * [mediawiki.js Line 540] Unreachable 'registry' after 'throw'. * [mediawiki.log.js] Checked for window.console but used console. * Added callback option to mw.util.toggleToc (forwarded to jQuery.fn.slideUp/Down) * Made mw.log argument more descriptive (logmsg instead of string) * Using deepEqual(, true) instead of ok() when asserting true. (ok() is like casting a boolean (or a plain if() statement) to verify that something is not falsy) Added missing test suites for: * mw.util.toggleToc (follow-up r88732, Dummy check was introduced in r87898) /me has been infected by Reedy and will be doing more of this tonight. Comment: Sounds like you're trying to combine two tests here: # the expression returns the expected boolean result (something evaluating to true) # the expression's return value is always a boolean-typed value For #1 there, <code>ok()</code> is both sufficient and correct. For #2, you'd also want to test a range of cases to actually cover it correctly. But honestly there's not much need to be doing that test on language constructs like the <code><=</code> and <code>in</code> operators, is there? _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
