From 5d5ea015114092c157311c4948f7cc3d8c8e7f8a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Tue, 4 Jan 2022 16:27:18 +0100 Subject: [PATCH] Docs: Replace `#NUMBER` Trac issue references with `trac-NUMBER` The GitHub UI treats `#NUMBER` as referring to its own issues which is confusing when in jQuery source it's usually referring to the old deprecated Trac instance at https://bugs.jquery.com. This change replaces all such Trac references with `trac-NUMBER`. A few of the references came with the Sizzle integration and referred to the Sizzle GitHub bug tracker. Those have been replaced with full links instead. A new entry describing issue reference conventions has been added to README. Closes gh-4993 --- README.md | 6 ++ src/ajax.js | 12 ++-- src/ajax/xhr.js | 4 +- src/attributes/prop.js | 2 +- src/attributes/val.js | 2 +- src/core/init.js | 4 +- src/core/ready-no-deferred.js | 2 +- src/core/ready.js | 2 +- src/css.js | 8 +-- src/css/cssCamelCase.js | 2 +- src/data.js | 2 +- src/data/Data.js | 2 +- src/event.js | 6 +- src/event/trigger.js | 6 +- src/exports/global.js | 4 +- src/manipulation.js | 2 +- src/manipulation/_evalUrl.js | 2 +- src/manipulation/buildFragment.js | 2 +- src/manipulation/getAll.js | 2 +- src/selector.js | 11 +-- src/wrapper.js | 2 +- test/data/ajax/onunload.html | 2 +- test/data/core/aliased.html | 2 +- test/data/core/onready.html | 4 +- test/data/data/dataAttrs.html | 4 +- test/data/event/focusElem.html | 2 +- test/data/event/focusinCrossFrame.html | 2 +- test/data/event/promiseReady.html | 2 +- test/data/event/syncReady.html | 2 +- test/data/readywait.html | 4 +- test/data/testsuite.css | 8 +-- test/networkerror.html | 4 +- test/unit/ajax.js | 62 ++++++++--------- test/unit/attributes.js | 66 +++++++++--------- test/unit/callbacks.js | 2 +- test/unit/core.js | 36 +++++----- test/unit/css.js | 80 +++++++++++----------- test/unit/data.js | 32 ++++----- test/unit/deprecated.js | 2 +- test/unit/dimensions.js | 92 +++++++++++++------------- test/unit/effects.js | 58 ++++++++-------- test/unit/event.js | 54 +++++++-------- test/unit/manipulation.js | 84 +++++++++++------------ test/unit/offset.js | 16 ++--- test/unit/queue.js | 2 +- test/unit/selector.js | 41 ++++++------ test/unit/serialize.js | 2 +- test/unit/support.js | 2 +- test/unit/traversing.js | 26 ++++---- test/unit/wrap.js | 12 ++-- 50 files changed, 400 insertions(+), 390 deletions(-) diff --git a/README.md b/README.md index 085ce3905..1f2c22f88 100644 --- a/README.md +++ b/README.md @@ -14,6 +14,12 @@ In the spirit of open source software development, jQuery always encourages comm 2. [Core Style Guide](https://contribute.jquery.org/style-guide/js/) 3. [Writing Code for jQuery Foundation Projects](https://contribute.jquery.org/code/) +### References to issues/PRs + +GitHub issues/PRs are usually referenced via `gh-NUMBER`, where `NUMBER` is the numerical ID of the issue/PR. You can find such an issue/PR under `https://github.com/jquery/jquery/issues/NUMBER`. + +jQuery has used a different bug tracker - based on Trac - in the past, available under [bugs.jquery.com](https://bugs.jquery.com/). It is being kept in read only mode so that referring to past discussions is possible. When jQuery source references one of those issues, it uses the pattern `trac-NUMBER`, where `NUMBER` is the numerical ID of the issue. You can find such an issue under `https://bugs.jquery.com/ticket/NUMBER`. + Environments in which to use jQuery -------------------------------------- diff --git a/src/ajax.js b/src/ajax.js index 3921de6b5..dc9fb242b 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -17,7 +17,7 @@ var rantiCache = /([?&])_=[^&]*/, rheaders = /^(.*?):[ \t]*([^\r\n]*)$/mg, - // #7653, #8125, #8152: local protocol detection + // trac-7653, trac-8125, trac-8152: local protocol detection rlocalProtocol = /^(?:about|app|app-storage|.+-extension|file|res|widget):$/, rnoContent = /^(?:GET|HEAD)$/, rprotocol = /^\/\//, @@ -40,7 +40,7 @@ var */ transports = {}, - // Avoid comment-prolog char sequence (#10098); must appease lint and evade compression + // Avoid comment-prolog char sequence (trac-10098); must appease lint and evade compression allTypes = "*/".concat( "*" ), // Anchor tag for parsing the document origin @@ -111,7 +111,7 @@ function inspectPrefiltersOrTransports( structure, options, originalOptions, jqX // A special extend for ajax options // that takes "flat" options (not to be deep extended) -// Fixes #9887 +// Fixes trac-9887 function ajaxExtend( target, src ) { var key, deep, flatOptions = jQuery.ajaxSettings.flatOptions || {}; @@ -528,12 +528,12 @@ jQuery.extend( { deferred.promise( jqXHR ); // Add protocol if not provided (prefilters might expect it) - // Handle falsy url in the settings object (#10093: consistency with old signature) + // Handle falsy url in the settings object (trac-10093: consistency with old signature) // We also use the url parameter if available s.url = ( ( url || s.url || location.href ) + "" ) .replace( rprotocol, location.protocol + "//" ); - // Alias method option to type as per ticket #12004 + // Alias method option to type as per ticket trac-12004 s.type = options.method || options.type || s.method || s.type; // Extract dataTypes list @@ -605,7 +605,7 @@ jQuery.extend( { if ( s.data && ( s.processData || typeof s.data === "string" ) ) { cacheURL += ( rquery.test( cacheURL ) ? "&" : "?" ) + s.data; - // #9682: remove data so that it's not used in an eventual retry + // trac-9682: remove data so that it's not used in an eventual retry delete s.data; } diff --git a/src/ajax/xhr.js b/src/ajax/xhr.js index 2c9bc8125..a024bb236 100644 --- a/src/ajax/xhr.js +++ b/src/ajax/xhr.js @@ -66,7 +66,7 @@ jQuery.ajaxTransport( function( options ) { } else if ( type === "error" ) { complete( - // File: protocol always yields status 0; see #8605, #14207 + // File: protocol always yields status 0; see trac-8605, trac-14207 xhr.status, xhr.statusText ); @@ -99,7 +99,7 @@ jQuery.ajaxTransport( function( options ) { xhr.send( options.hasContent && options.data || null ); } catch ( e ) { - // #14683: Only rethrow if this hasn't been notified as an error yet + // trac-14683: Only rethrow if this hasn't been notified as an error yet if ( callback ) { throw e; } diff --git a/src/attributes/prop.js b/src/attributes/prop.js index 453d4c7cd..e80532eb6 100644 --- a/src/attributes/prop.js +++ b/src/attributes/prop.js @@ -60,7 +60,7 @@ jQuery.extend( { // elem.tabIndex doesn't always return the // correct value when it hasn't been explicitly set // https://web.archive.org/web/20141116233347/http://fluidproject.org/blog/2008/01/09/getting-setting-and-removing-tabindex-values-with-javascript/ - // Use proper attribute retrieval(#12072) + // Use proper attribute retrieval (trac-12072) var tabindex = elem.getAttribute( "tabindex" ); if ( tabindex ) { diff --git a/src/attributes/val.js b/src/attributes/val.js index f200f97ee..536ed88a4 100644 --- a/src/attributes/val.js +++ b/src/attributes/val.js @@ -149,7 +149,7 @@ if ( isIE ) { val : // Support: IE <=10 - 11+ - // option.text throws exceptions (#14686, #14858) + // option.text throws exceptions (trac-14686, trac-14858) // Strip and collapse whitespace // https://html.spec.whatwg.org/#strip-and-collapse-whitespace stripAndCollapse( jQuery.text( elem ) ); diff --git a/src/core/init.js b/src/core/init.js index 8fc24d8dd..9fea5e6d4 100644 --- a/src/core/init.js +++ b/src/core/init.js @@ -10,8 +10,8 @@ import "../traversing/findFilter.js"; var rootjQuery, // A simple way to check for HTML strings - // Prioritize #id over to avoid XSS via location.hash (#9521) - // Strict HTML recognition (#11290: must start with <) + // Prioritize #id over to avoid XSS via location.hash (trac-9521) + // Strict HTML recognition (trac-11290: must start with <) // Shortcut simple #id case for speed rquickExpr = /^(?:\s*(<[\w\W]+>)[^>]*|#([\w-]+))$/, diff --git a/src/core/ready-no-deferred.js b/src/core/ready-no-deferred.js index 51f6d7f6b..17aaf5bcd 100644 --- a/src/core/ready-no-deferred.js +++ b/src/core/ready-no-deferred.js @@ -25,7 +25,7 @@ jQuery.extend( { isReady: false, // A counter to track how many items to wait for before - // the ready event fires. See #6781 + // the ready event fires. See trac-6781 readyWait: 1, ready: function( wait ) { diff --git a/src/core/ready.js b/src/core/ready.js index d6c507e41..88dcc3433 100644 --- a/src/core/ready.js +++ b/src/core/ready.js @@ -28,7 +28,7 @@ jQuery.extend( { isReady: false, // A counter to track how many items to wait for before - // the ready event fires. See #6781 + // the ready event fires. See trac-6781 readyWait: 1, // Handle when the DOM is ready diff --git a/src/css.js b/src/css.js index b50aa3d91..29dca0e42 100644 --- a/src/css.js +++ b/src/css.js @@ -220,15 +220,15 @@ jQuery.extend( { if ( value !== undefined ) { type = typeof value; - // Convert "+=" or "-=" to relative numbers (#7345) + // Convert "+=" or "-=" to relative numbers (trac-7345) if ( type === "string" && ( ret = rcssNum.exec( value ) ) && ret[ 1 ] ) { value = adjustCSS( elem, name, ret ); - // Fixes bug #9237 + // Fixes bug trac-9237 type = "number"; } - // Make sure that null and NaN values aren't set (#7116) + // Make sure that null and NaN values aren't set (trac-7116) if ( value == null || value !== value ) { return; } @@ -239,7 +239,7 @@ jQuery.extend( { } // Support: IE <=9 - 11+ - // background-* props of a cloned element affect the source element (#8908) + // background-* props of a cloned element affect the source element (trac-8908) if ( isIE && value === "" && name.indexOf( "background" ) === 0 ) { style[ name ] = "inherit"; } diff --git a/src/css/cssCamelCase.js b/src/css/cssCamelCase.js index a3d5fe628..c75dcd950 100644 --- a/src/css/cssCamelCase.js +++ b/src/css/cssCamelCase.js @@ -6,7 +6,7 @@ var rmsPrefix = /^-ms-/; // Convert dashed to camelCase, handle vendor prefixes. // Used by the css & effects modules. // Support: IE <=9 - 11+ -// Microsoft forgot to hump their vendor prefix (#9572) +// Microsoft forgot to hump their vendor prefix (trac-9572) function cssCamelCase( string ) { return camelCase( string.replace( rmsPrefix, "ms-" ) ); } diff --git a/src/data.js b/src/data.js index cd658b386..e3e8eabe1 100644 --- a/src/data.js +++ b/src/data.js @@ -105,7 +105,7 @@ jQuery.fn.extend( { while ( i-- ) { // Support: IE 11+ - // The attrs elements can be null (#14894) + // The attrs elements can be null (trac-14894) if ( attrs[ i ] ) { name = attrs[ i ].name; if ( name.indexOf( "data-" ) === 0 ) { diff --git a/src/data/Data.js b/src/data/Data.js index a5eb09089..8c9e1f9f8 100644 --- a/src/data/Data.js +++ b/src/data/Data.js @@ -21,7 +21,7 @@ Data.prototype = { value = Object.create( null ); // We can accept data for non-element nodes in modern browsers, - // but we should not, see #8335. + // but we should not, see trac-8335. // Always return an empty object. if ( acceptData( owner ) ) { diff --git a/src/event.js b/src/event.js index 4b6eb00e4..0c66d3856 100644 --- a/src/event.js +++ b/src/event.js @@ -369,15 +369,15 @@ jQuery.event = { for ( ; cur !== this; cur = cur.parentNode || this ) { - // Don't check non-elements (#13208) - // Don't process clicks on disabled elements (#6911, #8165, #11382, #11764) + // Don't check non-elements (trac-13208) + // Don't process clicks on disabled elements (trac-6911, trac-8165, trac-11382, trac-11764) if ( cur.nodeType === 1 && !( event.type === "click" && cur.disabled === true ) ) { matchedHandlers = []; matchedSelectors = {}; for ( i = 0; i < delegateCount; i++ ) { handleObj = handlers[ i ]; - // Don't conflict with Object.prototype properties (#13203) + // Don't conflict with Object.prototype properties (trac-13203) sel = handleObj.selector + " "; if ( matchedSelectors[ sel ] === undefined ) { diff --git a/src/event/trigger.js b/src/event/trigger.js index 7919a5013..5fad460e1 100644 --- a/src/event/trigger.js +++ b/src/event/trigger.js @@ -71,8 +71,8 @@ jQuery.extend( jQuery.event, { return; } - // Determine event propagation path in advance, per W3C events spec (#9951) - // Bubble up to document, then to window; watch for a global ownerDocument var (#9724) + // Determine event propagation path in advance, per W3C events spec (trac-9951) + // Bubble up to document, then to window; watch for a global ownerDocument var (trac-9724) if ( !onlyHandlers && !special.noBubble && !isWindow( elem ) ) { bubbleType = special.delegateType || type; @@ -124,7 +124,7 @@ jQuery.extend( jQuery.event, { acceptData( elem ) ) { // Call a native DOM method on the target with the same name as the event. - // Don't do default actions on window, that's where global variables be (#6170) + // Don't do default actions on window, that's where global variables be (trac-6170) if ( ontype && typeof elem[ type ] === "function" && !isWindow( elem ) ) { // Don't re-trigger an onFOO event when we call its FOO() method diff --git a/src/exports/global.js b/src/exports/global.js index b49496fbb..965a4b400 100644 --- a/src/exports/global.js +++ b/src/exports/global.js @@ -21,8 +21,8 @@ jQuery.noConflict = function( deep ) { }; // Expose jQuery and $ identifiers, even in AMD -// (#7102#comment:10, https://github.com/jquery/jquery/pull/557) -// and CommonJS for browser emulators (#13566) +// (trac-7102#comment:10, gh-557) +// and CommonJS for browser emulators (trac-13566) if ( typeof noGlobal === "undefined" ) { window.jQuery = window.$ = jQuery; } diff --git a/src/manipulation.js b/src/manipulation.js index 19c60fcbc..cad13bd92 100644 --- a/src/manipulation.js +++ b/src/manipulation.js @@ -120,7 +120,7 @@ function domManip( collection, args, callback, ignored ) { // Use the original fragment for the last item // instead of the first because it can end up - // being emptied incorrectly in certain situations (#8070). + // being emptied incorrectly in certain situations (trac-8070). for ( ; i < l; i++ ) { node = fragment; diff --git a/src/manipulation/_evalUrl.js b/src/manipulation/_evalUrl.js index 8a8d63d9d..1af889ddc 100644 --- a/src/manipulation/_evalUrl.js +++ b/src/manipulation/_evalUrl.js @@ -4,7 +4,7 @@ jQuery._evalUrl = function( url, options, doc ) { return jQuery.ajax( { url: url, - // Make this explicit, since user can override this through ajaxSetup (#11264) + // Make this explicit, since user can override this through ajaxSetup (trac-11264) type: "GET", dataType: "script", cache: true, diff --git a/src/manipulation/buildFragment.js b/src/manipulation/buildFragment.js index d6f8e5783..dd0d673e6 100644 --- a/src/manipulation/buildFragment.js +++ b/src/manipulation/buildFragment.js @@ -52,7 +52,7 @@ function buildFragment( elems, context, scripts, selection, ignored ) { // Remember the top-level container tmp = fragment.firstChild; - // Ensure the created nodes are orphaned (#12392) + // Ensure the created nodes are orphaned (trac-12392) tmp.textContent = ""; } } diff --git a/src/manipulation/getAll.js b/src/manipulation/getAll.js index 995d22c0c..f4943dfe4 100644 --- a/src/manipulation/getAll.js +++ b/src/manipulation/getAll.js @@ -4,7 +4,7 @@ import nodeName from "../core/nodeName.js"; function getAll( context, tag ) { // Support: IE <=9 - 11+ - // Use typeof to avoid zero-argument method invocation on host objects (#15151) + // Use typeof to avoid zero-argument method invocation on host objects (trac-15151) var ret; if ( typeof context.getElementsByTagName !== "undefined" ) { diff --git a/src/selector.js b/src/selector.js index 4b9c8b6ed..bc60e61e4 100644 --- a/src/selector.js +++ b/src/selector.js @@ -282,7 +282,8 @@ function createCache() { function cache( key, value ) { - // Use (key + " ") to avoid collision with native prototype properties (see Issue #157) + // Use (key + " ") to avoid collision with native prototype properties + // (see https://github.com/jquery/sizzle/issues/157) if ( keys.push( key + " " ) > Expr.cacheLength ) { // Only keep the most recent entries @@ -432,7 +433,7 @@ function setDocument( node ) { documentIsHTML = !jQuery.isXMLDoc( document ); // Support: IE 9 - 11+ - // Accessing iframe documents after unload throws "permission denied" errors (jQuery #13936) + // Accessing iframe documents after unload throws "permission denied" errors (see trac-13936) // Support: IE 11+ // IE sometimes throws a "Permission denied" error when strict-comparing // two documents; shallow comparisons work. @@ -843,7 +844,8 @@ Expr = jQuery.expr = { input[ 0 ] = elem; matcher( input, null, xml, results ); - // Don't keep the element (issue #299) + // Don't keep the element + // (see https://github.com/jquery/sizzle/issues/299) input[ 0 ] = null; return !results.pop(); }; @@ -1348,7 +1350,8 @@ function matcherFromTokens( tokens ) { matchContext( elem, context, xml ) : matchAnyContext( elem, context, xml ) ); - // Avoid hanging onto element (issue #299) + // Avoid hanging onto element + // (see https://github.com/jquery/sizzle/issues/299) checkContext = null; return ret; } ]; diff --git a/src/wrapper.js b/src/wrapper.js index 8a0b2a1ca..8a3dc53e0 100644 --- a/src/wrapper.js +++ b/src/wrapper.js @@ -20,7 +20,7 @@ // (such as Node.js), expose a factory as module.exports. // This accentuates the need for the creation of a real `window`. // e.g. var jQuery = require("jquery")(window); - // See ticket #14549 for more info. + // See ticket trac-14549 for more info. module.exports = global.document ? factory( global, true ) : function( w ) { diff --git a/test/data/ajax/onunload.html b/test/data/ajax/onunload.html index ec6e4b702..70084f66d 100644 --- a/test/data/ajax/onunload.html +++ b/test/data/ajax/onunload.html @@ -2,7 +2,7 @@ - onunload ajax requests (#14379) + onunload ajax requests (trac-14379) diff --git a/test/data/core/aliased.html b/test/data/core/aliased.html index 87b5871f4..519fccfb1 100644 --- a/test/data/core/aliased.html +++ b/test/data/core/aliased.html @@ -2,7 +2,7 @@ - alias-masked DOM properties (#14074) + alias-masked DOM properties (trac-14074) - \ No newline at end of file + diff --git a/test/data/data/dataAttrs.html b/test/data/data/dataAttrs.html index 785150eb8..854143fe8 100644 --- a/test/data/data/dataAttrs.html +++ b/test/data/data/dataAttrs.html @@ -2,7 +2,7 @@ - IE11 onpageshow strangeness (#14894) + IE11 onpageshow strangeness (trac-14894) - Test for #14894 + Test for trac-14894 diff --git a/test/data/event/focusElem.html b/test/data/event/focusElem.html index 1940e8b61..2c09b6d83 100644 --- a/test/data/event/focusElem.html +++ b/test/data/event/focusElem.html @@ -2,7 +2,7 @@ - .focus() (activeElement access #13393) + .focus() (activeElement access trac-13393) diff --git a/test/data/event/focusinCrossFrame.html b/test/data/event/focusinCrossFrame.html index 6dd187e90..0230eb8a5 100644 --- a/test/data/event/focusinCrossFrame.html +++ b/test/data/event/focusinCrossFrame.html @@ -2,7 +2,7 @@ - focusin event cross-frame (#14180) + focusin event cross-frame (trac-14180) diff --git a/test/data/event/promiseReady.html b/test/data/event/promiseReady.html index c6e086245..7ed656b6f 100644 --- a/test/data/event/promiseReady.html +++ b/test/data/event/promiseReady.html @@ -2,7 +2,7 @@ -Test case for jQuery ticket #11470 +Test case for jQuery ticket trac-11470 diff --git a/test/data/readywait.html b/test/data/readywait.html index d7de0b082..8f88245d5 100644 --- a/test/data/readywait.html +++ b/test/data/readywait.html @@ -45,9 +45,9 @@

This is a test page for jQuery.readyWait and jQuery.holdReady, see - #6781 + trac-6781 and - #8803. + trac-8803.

Test for jQuery.holdReady, which can be used diff --git a/test/data/testsuite.css b/test/data/testsuite.css index b5baaa7ad..bfa77c7aa 100644 --- a/test/data/testsuite.css +++ b/test/data/testsuite.css @@ -112,16 +112,16 @@ div#fx-tests div.noback { #nothiddendivchild.em { font-size: 2em; } #nothiddendivchild.prct { font-size: 150%; } -/* #9239 Attach a background to the body( avoid crashes in removing the test element in support ) */ +/* trac-9239 Attach a background to the body( avoid crashes in removing the test element in support ) */ body, div { background: url(1x1.jpg) no-repeat -1000px 0; } -/* #10501 */ +/* trac-10501 */ section { background:#f0f; display:block; } -/* #11971 */ +/* trac-11971 */ #foo { background: url(1x1.jpg) right bottom no-repeat; } -/* #14824 */ +/* trac-14824 */ #span-14824 { display: block; } #display { display: list-item !important; } diff --git a/test/networkerror.html b/test/networkerror.html index f666ee048..40848bce5 100644 --- a/test/networkerror.html +++ b/test/networkerror.html @@ -1,7 +1,7 @@ " ); assert.ok( j.length >= 2, "Check node,textnode,comment creation (some browsers delete comments)" ); - assert.ok( !jQuery( "" )[ 0 ].selected, "Make sure that options are auto-selected #2050" ); + assert.ok( !jQuery( "" )[ 0 ].selected, "Make sure that options are auto-selected trac-2050" ); assert.ok( jQuery( "

" )[ 0 ], "Create a div with closing tag." ); assert.ok( jQuery( "
" )[ 0 ], "Create a table with closing tag." ); @@ -516,7 +516,7 @@ QUnit.test( "jQuery(element with non-alphanumeric name)", function( assert ) { } ); } ); -QUnit.test( "jQuery('massive html #7990')", function( assert ) { +QUnit.test( "jQuery('massive html trac-7990')", function( assert ) { assert.expect( 3 ); var i, @@ -538,7 +538,7 @@ QUnit.test( "jQuery('html', context)", function( assert ) { var $div = jQuery( "
" )[ 0 ], $span = jQuery( "", $div ); - assert.equal( $span.length, 1, "verify a span created with a div context works, #1763" ); + assert.equal( $span.length, 1, "verify a span created with a div context works, trac-1763" ); } ); QUnit.test( "jQuery(selector, xml).text(str) - loaded via xml document", function( assert ) { @@ -546,7 +546,7 @@ QUnit.test( "jQuery(selector, xml).text(str) - loaded via xml document", functio var xml = createDashboardXML(), - // tests for #1419 where ie was a problem + // tests for trac-1419 where ie was a problem tab = jQuery( "tab", xml ).eq( 0 ); assert.equal( tab.text(), "blabla", "verify initial text correct" ); tab.text( "newtext" ); @@ -803,7 +803,7 @@ QUnit.test( "jQuery.map", function( assert ) { result = jQuery.map( Array( 4 ), function( v, k ) { return k % 2 ? k : [ k, k, k ]; } ); - assert.equal( result.join( "" ), "00012223", "Array results flattened (#2616)" ); + assert.equal( result.join( "" ), "00012223", "Array results flattened (trac-2616)" ); result = jQuery.map( [ [ [ 1, 2 ], 3 ], 4 ], function( v, k ) { return v; @@ -817,7 +817,7 @@ QUnit.test( "jQuery.map", function( assert ) { result = jQuery.map( Array( 300000 ), function( v, k ) { return k; } ); - assert.equal( result.length, 300000, "Able to map 300000 records without any problems (#4320)" ); + assert.equal( result.length, 300000, "Able to map 300000 records without any problems (trac-4320)" ); } else { assert.ok( "skip", "Array#flat isn't supported in IE" ); } @@ -854,14 +854,14 @@ QUnit.test( "jQuery.merge()", function( assert ) { "First empty" ); - // Fixed at [5998], #3641 + // Fixed at [5998], trac-3641 assert.deepEqual( jQuery.merge( [ -2, -1 ], [ 0, 1, 2 ] ), [ -2, -1, 0, 1, 2 ], "Second array including a zero (falsy)" ); - // After fixing #5527 + // After fixing trac-5527 assert.deepEqual( jQuery.merge( [], [ null, undefined ] ), [ null, undefined ], @@ -1005,7 +1005,7 @@ QUnit.test( "jQuery.extend(Object, Object)", function( assert ) { assert.ok( jQuery.extend( true, {}, nestedarray )[ "arr" ] !== arr, "Deep extend of object must clone child array" ); - // #5991 + // trac-5991 assert.ok( Array.isArray( jQuery.extend( true, { "arr": {} }, nestedarray )[ "arr" ] ), "Cloned array have to be an Array" ); assert.ok( jQuery.isPlainObject( jQuery.extend( true, { "arr": arr }, { "arr": {} } )[ "arr" ] ), "Cloned object have to be an plain object" ); @@ -1053,13 +1053,13 @@ QUnit.test( "jQuery.extend(Object, Object)", function( assert ) { assert.deepEqual( target, { bar:5 }, "Check to make sure a recursive obj doesn't go never-ending loop by not copying it over" ); ret = jQuery.extend( true, { foo: [] }, { foo: [ 0 ] } ); // 1907 - assert.equal( ret.foo.length, 1, "Check to make sure a value with coercion 'false' copies over when necessary to fix #1907" ); + assert.equal( ret.foo.length, 1, "Check to make sure a value with coercion 'false' copies over when necessary to fix trac-1907" ); ret = jQuery.extend( true, { foo: "1,2,3" }, { foo: [ 1, 2, 3 ] } ); assert.ok( typeof ret.foo !== "string", "Check to make sure values equal with coercion (but not actually equal) overwrite correctly" ); ret = jQuery.extend( true, { foo:"bar" }, { foo:null } ); - assert.ok( typeof ret.foo !== "undefined", "Make sure a null value doesn't crash with deep extend, for #1908" ); + assert.ok( typeof ret.foo !== "undefined", "Make sure a null value doesn't crash with deep extend, for trac-1908" ); obj = { foo:null }; jQuery.extend( true, obj, { foo:"notnull" } ); @@ -1365,7 +1365,7 @@ QUnit.test( "jQuery.parseHTML", function( assert ) { assert.equal( jQuery.parseHTML( "text" )[ 0 ].nodeType, 3, "Parsing text returns a text node" ); assert.equal( jQuery.parseHTML( "\t
" )[ 0 ].nodeValue, "\t", "Preserve leading whitespace" ); - assert.equal( jQuery.parseHTML( "
" )[ 0 ].nodeType, 3, "Leading spaces are treated as text nodes (#11290)" ); + assert.equal( jQuery.parseHTML( "
" )[ 0 ].nodeType, 3, "Leading spaces are treated as text nodes (trac-11290)" ); html = jQuery.parseHTML( "
test div
" ); @@ -1460,7 +1460,7 @@ QUnit.testUnlessIE( "jQuery.parseXML - error reporting", function( assert ) { } ); testIframe( - "document ready when jQuery loaded asynchronously (#13655)", + "document ready when jQuery loaded asynchronously (trac-13655)", "core/dynamic_ready.html", function( assert, jQuery, window, document, ready ) { assert.expect( 1 ); @@ -1469,7 +1469,7 @@ testIframe( ); testIframe( - "Tolerating alias-masked DOM properties (#14074)", + "Tolerating alias-masked DOM properties (trac-14074)", "core/aliased.html", function( assert, jQuery, window, document, errors ) { assert.expect( 1 ); @@ -1478,7 +1478,7 @@ testIframe( ); testIframe( - "Don't call window.onready (#14802)", + "Don't call window.onready (trac-14802)", "core/onready.html", function( assert, jQuery, window, document, error ) { assert.expect( 1 ); diff --git a/test/unit/css.js b/test/unit/css.js index 100b8c8c5..0c1e43dec 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -33,7 +33,7 @@ QUnit.test( "css(String|Hash)", function( assert ) { div2.remove(); - // handle negative numbers by setting to zero #11604 + // handle negative numbers by setting to zero trac-11604 jQuery( "#nothiddendiv" ).css( { "width": 1, "height": 1 } ); width = parseFloat( jQuery( "#nothiddendiv" ).css( "width" ) ); @@ -113,7 +113,7 @@ QUnit.test( "css(String|Hash)", function( assert ) { div = jQuery( "
" ).css( { position: "absolute", "z-index": 1000 } ).appendTo( "#qunit-fixture" ); assert.strictEqual( div.css( "z-index" ), "1000", - "Make sure that a string z-index is returned from css('z-index') (#14432)." ); + "Make sure that a string z-index is returned from css('z-index') (trac-14432)." ); } ); QUnit.test( "css() explicit and relative values", function( assert ) { @@ -325,11 +325,11 @@ QUnit.test( "css(String, Object)", function( assert ) { catch ( e ) { success = false; } - assert.ok( success, "Setting RGBA values does not throw Error (#5509)" ); + assert.ok( success, "Setting RGBA values does not throw Error (trac-5509)" ); jQuery( "#foo" ).css( "font", "7px/21px sans-serif" ); assert.strictEqual( jQuery( "#foo" ).css( "line-height" ), "21px", - "Set font shorthand property (#14759)" ); + "Set font shorthand property (trac-14759)" ); } ); QUnit.test( "css(String, Object) with negative values", function( assert ) { @@ -699,7 +699,7 @@ QUnit.testUnlessIE( } ); } ); -QUnit.test( "hide hidden elements (bug #7141)", function( assert ) { +QUnit.test( "hide hidden elements (bug trac-7141)", function( assert ) { assert.expect( 3 ); var div = jQuery( "
" ).appendTo( "#qunit-fixture" ); @@ -712,7 +712,7 @@ QUnit.test( "hide hidden elements (bug #7141)", function( assert ) { div.remove(); } ); -QUnit.test( "show() after hide() should always set display to initial value (#14750)", function( assert ) { +QUnit.test( "show() after hide() should always set display to initial value (trac-14750)", function( assert ) { assert.expect( 1 ); var div = jQuery( "
" ), @@ -988,7 +988,7 @@ QUnit[ QUnit.jQuerySelectors && jQuery.fn.toggle ? "test" : "skip" ]( "toggle()" assert.strictEqual( x.toggle().css( "display" ), "none", "is hidden" ); assert.strictEqual( x.toggle().css( "display" ), "block", "is visible" ); - // Ensure hide() is called when toggled (#12148) + // Ensure hide() is called when toggled (trac-12148) oldHide = jQuery.fn.hide; jQuery.fn.hide = function() { assert.ok( true, name + " method called on toggle" ); @@ -1048,7 +1048,7 @@ QUnit[ QUnit.jQuerySelectors && jQuery.fn.toggle && !QUnit.isIE ? "test" : "skip assert.strictEqual( $shadowChild.css( "display" ), "block", "is visible" ); } ); -QUnit.test( "jQuery.css(elem, 'height') doesn't clear radio buttons (bug #1095)", function( assert ) { +QUnit.test( "jQuery.css(elem, 'height') doesn't clear radio buttons (bug trac-1095)", function( assert ) { assert.expect( 4 ); var $checkedtest = jQuery( "#checkedtest" ); @@ -1060,7 +1060,7 @@ QUnit.test( "jQuery.css(elem, 'height') doesn't clear radio buttons (bug #1095)" assert.ok( !jQuery( "input[type='checkbox']", $checkedtest ).last().attr( "checked" ), "Check last checkbox still NOT checked." ); } ); -QUnit.test( "internal ref to elem.runtimeStyle (bug #7608)", function( assert ) { +QUnit.test( "internal ref to elem.runtimeStyle (bug trac-7608)", function( assert ) { assert.expect( 1 ); var result = true; @@ -1099,18 +1099,18 @@ QUnit.test( "computed margins (trac-3333; gh-2237)", function( assert ) { assert.equal( $child.css( "marginLeft" ), "25px", "auto margins are computed to pixels" ); } ); -QUnit.test( "box model properties incorrectly returning % instead of px, see #10639 and #12088", function( assert ) { +QUnit.test( "box model properties incorrectly returning % instead of px, see trac-10639 and trac-12088", function( assert ) { assert.expect( 2 ); var container = jQuery( "
" ).width( 400 ).appendTo( "#qunit-fixture" ), el = jQuery( "
" ).css( { "width": "50%", "marginRight": "50%" } ).appendTo( container ), el2 = jQuery( "
" ).css( { "width": "50%", "minWidth": "300px", "marginLeft": "25%" } ).appendTo( container ); - assert.equal( el.css( "marginRight" ), "200px", "css('marginRight') returning % instead of px, see #10639" ); - assert.equal( el2.css( "marginLeft" ), "100px", "css('marginLeft') returning incorrect pixel value, see #12088" ); + assert.equal( el.css( "marginRight" ), "200px", "css('marginRight') returning % instead of px, see trac-10639" ); + assert.equal( el2.css( "marginLeft" ), "100px", "css('marginLeft') returning incorrect pixel value, see trac-12088" ); } ); -QUnit.test( "widows & orphans #8936", function( assert ) { +QUnit.test( "widows & orphans trac-8936", function( assert ) { var $p = jQuery( "

" ).appendTo( "#qunit-fixture" ); @@ -1127,15 +1127,15 @@ QUnit.test( "widows & orphans #8936", function( assert ) { $p.remove(); } ); -QUnit.test( "can't get css for disconnected in IE<9, see #10254 and #8388", function( assert ) { +QUnit.test( "can't get css for disconnected in IE<9, see trac-10254 and trac-8388", function( assert ) { assert.expect( 2 ); var span, div; span = jQuery( "" ).css( "background-image", "url(" + baseURL + "1x1.jpg)" ); - assert.notEqual( span.css( "background-image" ), null, "can't get background-image in IE<9, see #10254" ); + assert.notEqual( span.css( "background-image" ), null, "can't get background-image in IE<9, see trac-10254" ); div = jQuery( "

" ).css( "top", 10 ); - assert.equal( div.css( "top" ), "10px", "can't get top in IE<9, see #8388" ); + assert.equal( div.css( "top" ), "10px", "can't get top in IE<9, see trac-8388" ); } ); QUnit.test( "Ensure styles are retrieving from parsed html on document fragments", function( assert ) { @@ -1148,7 +1148,7 @@ QUnit.test( "Ensure styles are retrieving from parsed html on document fragments assert.equal( $span.css( "font-size" ), "14px", "Font-size retrievable on parsed HTML node" ); } ); -QUnit.test( "can't get background-position in IE<9, see #10796", function( assert ) { +QUnit.test( "can't get background-position in IE<9, see trac-10796", function( assert ) { var div = jQuery( "
" ).appendTo( "#qunit-fixture" ), units = [ "0 0", @@ -1167,22 +1167,22 @@ QUnit.test( "can't get background-position in IE<9, see #10796", function( asser for ( ; i < l; i++ ) { div.css( "background-position", units [ i ] ); - assert.ok( div.css( "background-position" ), "can't get background-position in IE<9, see #10796" ); + assert.ok( div.css( "background-position" ), "can't get background-position in IE<9, see trac-10796" ); } } ); if ( jQuery.fn.offset ) { - QUnit.test( "percentage properties for left and top should be transformed to pixels, see #9505", function( assert ) { + QUnit.test( "percentage properties for left and top should be transformed to pixels, see trac-9505", function( assert ) { assert.expect( 2 ); var parent = jQuery( "
" ).appendTo( "#qunit-fixture" ), div = jQuery( "
" ).appendTo( parent ); - assert.equal( div.css( "top" ), "100px", "position properties not transformed to pixels, see #9505" ); - assert.equal( div.css( "left" ), "100px", "position properties not transformed to pixels, see #9505" ); + assert.equal( div.css( "top" ), "100px", "position properties not transformed to pixels, see trac-9505" ); + assert.equal( div.css( "left" ), "100px", "position properties not transformed to pixels, see trac-9505" ); } ); } -QUnit.test( "Do not append px (#9548, #12990, #2792)", function( assert ) { +QUnit.test( "Do not append px (trac-9548, trac-12990, trac-2792)", function( assert ) { assert.expect( 4 ); var $div = jQuery( "
" ).appendTo( "#qunit-fixture" ); @@ -1335,16 +1335,16 @@ QUnit.test( "Append px to whitelisted properties", function( assert ) { } } ); -QUnit.test( "css('width') and css('height') should respect box-sizing, see #11004", function( assert ) { +QUnit.test( "css('width') and css('height') should respect box-sizing, see trac-11004", function( assert ) { assert.expect( 4 ); var el_dis = jQuery( "
test
" ), el = el_dis.clone().appendTo( "#qunit-fixture" ); - assert.equal( el.css( "width" ), el.css( "width", el.css( "width" ) ).css( "width" ), "css('width') is not respecting box-sizing, see #11004" ); - assert.equal( el_dis.css( "width" ), el_dis.css( "width", el_dis.css( "width" ) ).css( "width" ), "css('width') is not respecting box-sizing for disconnected element, see #11004" ); - assert.equal( el.css( "height" ), el.css( "height", el.css( "height" ) ).css( "height" ), "css('height') is not respecting box-sizing, see #11004" ); - assert.equal( el_dis.css( "height" ), el_dis.css( "height", el_dis.css( "height" ) ).css( "height" ), "css('height') is not respecting box-sizing for disconnected element, see #11004" ); + assert.equal( el.css( "width" ), el.css( "width", el.css( "width" ) ).css( "width" ), "css('width') is not respecting box-sizing, see trac-11004" ); + assert.equal( el_dis.css( "width" ), el_dis.css( "width", el_dis.css( "width" ) ).css( "width" ), "css('width') is not respecting box-sizing for disconnected element, see trac-11004" ); + assert.equal( el.css( "height" ), el.css( "height", el.css( "height" ) ).css( "height" ), "css('height') is not respecting box-sizing, see trac-11004" ); + assert.equal( el_dis.css( "height" ), el_dis.css( "height", el_dis.css( "height" ) ).css( "height" ), "css('height') is not respecting box-sizing for disconnected element, see trac-11004" ); } ); QUnit.test( "table rows width/height should be unaffected by inline styles", function( assert ) { @@ -1368,7 +1368,7 @@ QUnit.test( "table rows width/height should be unaffected by inline styles", fun } ); testIframe( - "css('width') should work correctly before document ready (#14084)", + "css('width') should work correctly before document ready (trac-14084)", "css/cssWidthBeforeDocReady.html", function( assert, jQuery, window, document, cssWidthBeforeDocReady ) { assert.expect( 1 ); @@ -1409,13 +1409,13 @@ QUnit.testUnlessIE( "css('width') and css('height') should return fractional val "css('height') should return fractional values" ); } ); -QUnit.test( "certain css values of 'normal' should be convertable to a number, see #8627", function( assert ) { +QUnit.test( "certain css values of 'normal' should be convertable to a number, see trac-8627", function( assert ) { assert.expect( 3 ); var el = jQuery( "
test
" ).appendTo( "#qunit-fixture" ); - assert.ok( !isNaN( parseFloat( el.css( "letterSpacing" ) ) ), "css('letterSpacing') not convertable to number, see #8627" ); - assert.ok( !isNaN( parseFloat( el.css( "fontWeight" ) ) ), "css('fontWeight') not convertable to number, see #8627" ); + assert.ok( !isNaN( parseFloat( el.css( "letterSpacing" ) ) ), "css('letterSpacing') not convertable to number, see trac-8627" ); + assert.ok( !isNaN( parseFloat( el.css( "fontWeight" ) ) ), "css('fontWeight') not convertable to number, see trac-8627" ); assert.equal( typeof el.css( "fontWeight" ), "string", ".css() returns a string" ); } ); @@ -1459,7 +1459,7 @@ QUnit.test( "cssHooks - expand", function( assert ) { } ); -QUnit.test( "css opacity consistency across browsers (#12685)", function( assert ) { +QUnit.test( "css opacity consistency across browsers (trac-12685)", function( assert ) { assert.expect( 3 ); var el, @@ -1487,8 +1487,8 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( ":visible/:hidden selectors", assert.ok( !jQuery( "#nothiddendiv" ).is( ":visible" ), "Modified CSS display: Assert element is hidden" ); jQuery( "#nothiddendiv" ).css( { "display": "block" } ); assert.ok( jQuery( "#nothiddendiv" ).is( ":visible" ), "Modified CSS display: Assert element is visible" ); - assert.ok( !jQuery( window ).is( ":visible" ), "Calling is(':visible') on window does not throw an exception (#10267)." ); - assert.ok( !jQuery( document ).is( ":visible" ), "Calling is(':visible') on document does not throw an exception (#10267)." ); + assert.ok( !jQuery( window ).is( ":visible" ), "Calling is(':visible') on window does not throw an exception (trac-10267)." ); + assert.ok( !jQuery( document ).is( ":visible" ), "Calling is(':visible') on document does not throw an exception (trac-10267)." ); assert.ok( jQuery( "#nothiddendiv" ).is( ":visible" ), "Modifying CSS display: Assert element is visible" ); jQuery( "#nothiddendiv" ).css( "display", "none" ); @@ -1507,9 +1507,9 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( ":visible/:hidden selectors", $table = jQuery( "#table" ); $table.html( "cellcell" ); - assert.equal( jQuery( "#table td:visible" ).length, 1, "hidden cell is not perceived as visible (#4512). Works on table elements" ); + assert.equal( jQuery( "#table td:visible" ).length, 1, "hidden cell is not perceived as visible (trac-4512). Works on table elements" ); $table.css( "display", "none" ).html( "cellcell" ); - assert.equal( jQuery( "#table td:visible" ).length, 0, "hidden cell children not perceived as visible (#4512)" ); + assert.equal( jQuery( "#table td:visible" ).length, 0, "hidden cell children not perceived as visible (trac-4512)" ); if ( QUnit.jQuerySelectorsPos ) { assert.t( "Is Visible", "#qunit-fixture div:visible:lt(2)", [ "foo", "nothiddendiv" ] ); @@ -1524,7 +1524,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( ":visible/:hidden selectors", assert.ok( $a.is( ":visible" ), "Anchor tag with flow content is visible (gh-2227)" ); } ); -QUnit.test( "Keep the last style if the new one isn't recognized by the browser (#14836)", function( assert ) { +QUnit.test( "Keep the last style if the new one isn't recognized by the browser (trac-14836)", function( assert ) { assert.expect( 1 ); var el = jQuery( "
" ).css( "position", "absolute" ).css( "position", "fake value" ); @@ -1551,7 +1551,7 @@ QUnit.test( "Reset the style if set to an empty string", function( assert ) { } ); QUnit.test( - "Clearing a Cloned Element's Style Shouldn't Clear the Original Element's Style (#8908)", + "Clearing a Cloned Element's Style Shouldn't Clear the Original Element's Style (trac-8908)", function( assert ) { assert.expect( 24 ); var done = assert.async(); @@ -1626,7 +1626,7 @@ QUnit.test( } ); -QUnit.test( "Don't append px to CSS \"order\" value (#14049)", function( assert ) { +QUnit.test( "Don't append px to CSS \"order\" value (trac-14049)", function( assert ) { assert.expect( 1 ); var $elem = jQuery( "
" ); @@ -1635,7 +1635,7 @@ QUnit.test( "Don't append px to CSS \"order\" value (#14049)", function( assert assert.equal( $elem.css( "order" ), "2", "2 on order" ); } ); -QUnit.test( "Do not throw on frame elements from css method (#15098)", function( assert ) { +QUnit.test( "Do not throw on frame elements from css method (trac-15098)", function( assert ) { assert.expect( 1 ); var frameWin, frameDoc, diff --git a/test/unit/data.js b/test/unit/data.js index 301bee917..49a3500ac 100644 --- a/test/unit/data.js +++ b/test/unit/data.js @@ -143,7 +143,7 @@ QUnit.test( "jQuery.data({})", function( assert ) { QUnit.test( "jQuery.data(window)", function( assert ) { assert.expect( 25 ); - // remove bound handlers from window object to stop potential false positives caused by fix for #5280 in + // remove bound handlers from window object to stop potential false positives caused by fix for trac-5280 in // transports/xhr.js jQuery( window ).off( "unload" ); @@ -172,7 +172,7 @@ QUnit.test( "jQuery.data(object/flash)", function( assert ) { } ); // attempting to access the data of an undefined jQuery element should be undefined -QUnit.test( "jQuery().data() === undefined (#14101)", function( assert ) { +QUnit.test( "jQuery().data() === undefined (trac-14101)", function( assert ) { assert.expect( 2 ); assert.strictEqual( jQuery().data(), undefined ); @@ -201,7 +201,7 @@ QUnit.test( ".data()", function( assert ) { dataObj = jQuery.extend( true, {}, jQuery( obj ).data() ); - assert.deepEqual( dataObj, { "foo": "baz" }, "Retrieve data object from a wrapped JS object (#7524)" ); + assert.deepEqual( dataObj, { "foo": "baz" }, "Retrieve data object from a wrapped JS object (trac-7524)" ); } ); function testDataTypes( $obj, assert ) { @@ -232,7 +232,7 @@ QUnit.test( "jQuery(Element).data(String, Object).data(String)", function( asser assert.strictEqual( div.data( "test" ), undefined, "No data exists initially" ); assert.strictEqual( div.data( "test", "success" ).data( "test" ), "success", "Data added" ); assert.strictEqual( div.data( "test", "overwritten" ).data( "test" ), "overwritten", "Data overwritten" ); - assert.strictEqual( div.data( "test", undefined ).data( "test" ), "overwritten", ".data(key,undefined) does nothing but is chainable (#5571)" ); + assert.strictEqual( div.data( "test", undefined ).data( "test" ), "overwritten", ".data(key,undefined) does nothing but is chainable (trac-5571)" ); assert.strictEqual( div.data( "notexist" ), undefined, "No data exists for unset key" ); testDataTypes( div, assert ); @@ -242,7 +242,7 @@ QUnit.test( "jQuery(Element).data(String, Object).data(String)", function( asser QUnit.test( "jQuery(plain Object).data(String, Object).data(String)", function( assert ) { assert.expect( 16 ); - // #3748 + // trac-3748 var $obj = jQuery( { exists: true } ); assert.strictEqual( $obj.data( "nothing" ), undefined, "Non-existent data returns undefined" ); assert.strictEqual( $obj.data( "exists" ), undefined, "Object properties are not returned as data" ); @@ -253,7 +253,7 @@ QUnit.test( "jQuery(plain Object).data(String, Object).data(String)", function( assert.deepEqual( $obj[ 0 ], { exists: true }, "removeData does not clear the object" ); } ); -QUnit.test( ".data(object) does not retain references. #13815", function( assert ) { +QUnit.test( ".data(object) does not retain references. trac-13815", function( assert ) { assert.expect( 2 ); var $divs = jQuery( "
" ).appendTo( "#qunit-fixture" ); @@ -290,7 +290,7 @@ QUnit.test( "data-* attributes", function( assert ) { child.appendTo( "#qunit-fixture" ); assert.equal( child.data( "myobj" ), "old data", "Value accessed from data-* attribute" ); - assert.equal( child.data( "foo-42" ), "boosh", "camelCasing does not affect numbers (#1751)" ); + assert.equal( child.data( "foo-42" ), "boosh", "camelCasing does not affect numbers (trac-1751)" ); child.data( "myobj", "replaced" ); assert.equal( child.data( "myobj" ), "replaced", "Original data overwritten" ); @@ -506,7 +506,7 @@ QUnit.test( ".removeData()", function( assert ) { assert.equal( div.data( "test.foo" ), undefined, "Make sure data is intact" ); } ); -QUnit.test( "JSON serialization (#8108)", function( assert ) { +QUnit.test( "JSON serialization (trac-8108)", function( assert ) { assert.expect( 1 ); var obj = { "foo": "bar" }; @@ -560,7 +560,7 @@ QUnit.test( ".data should not miss preset data-* w/ hyphenated property names", } ); } ); -QUnit.test( "jQuery.data should not miss data-* w/ hyphenated property names #14047", function( assert ) { +QUnit.test( "jQuery.data should not miss data-* w/ hyphenated property names trac-14047", function( assert ) { assert.expect( 1 ); @@ -774,7 +774,7 @@ QUnit.test( ".data supports interoperable removal of hyphenated/camelCase proper } ); } ); -QUnit.test( ".data supports interoperable removal of properties SET TWICE #13850", function( assert ) { +QUnit.test( ".data supports interoperable removal of properties SET TWICE trac-13850", function( assert ) { var div = jQuery( "
" ).appendTo( "#qunit-fixture" ), datas = { "non-empty": "a string", @@ -800,7 +800,7 @@ QUnit.test( ".data supports interoperable removal of properties SET TWICE #13850 } ); } ); -QUnit.test( ".removeData supports removal of hyphenated properties via array (#12786, gh-2257)", function( assert ) { +QUnit.test( ".removeData supports removal of hyphenated properties via array (trac-12786, gh-2257)", function( assert ) { assert.expect( 4 ); var div, plain, compare; @@ -833,7 +833,7 @@ QUnit.test( ".removeData supports removal of hyphenated properties via array (#1 } ); // Test originally by Moschel -QUnit.test( ".removeData should not throw exceptions. (#10080)", function( assert ) { +QUnit.test( ".removeData should not throw exceptions. (trac-10080)", function( assert ) { var done = assert.async(); assert.expect( 1 ); var frame = jQuery( "#loadediframe" ); @@ -846,7 +846,7 @@ QUnit.test( ".removeData should not throw exceptions. (#10080)", function( asser frame.attr( "src", baseURL + "iframe.html?param=true" ); } ); -QUnit.test( ".data only checks element attributes once. #8909", function( assert ) { +QUnit.test( ".data only checks element attributes once. trac-8909", function( assert ) { assert.expect( 2 ); var testing = { "test": "testing", @@ -874,7 +874,7 @@ QUnit.test( "data-* with JSON value can have newlines", function( assert ) { x.remove(); } ); -QUnit.test( ".data doesn't throw when calling selection is empty. #13551", function( assert ) { +QUnit.test( ".data doesn't throw when calling selection is empty. trac-13551", function( assert ) { assert.expect( 1 ); try { @@ -911,7 +911,7 @@ QUnit.test( "acceptData", function( assert ) { assert.equal( jQuery( form ) .data( "test", 42 ).data( "test" ), 42, "form with aliased DOM properties" ); } ); -QUnit.test( "Check proper data removal of non-element descendants nodes (#8335)", function( assert ) { +QUnit.test( "Check proper data removal of non-element descendants nodes (trac-8335)", function( assert ) { assert.expect( 1 ); var div = jQuery( "
text
" ), @@ -924,7 +924,7 @@ QUnit.test( "Check proper data removal of non-element descendants nodes (#8335)" } ); testIframe( - "enumerate data attrs on body (#14894)", + "enumerate data attrs on body (trac-14894)", "data/dataAttrs.html", function( assert, jQuery, window, document, result ) { assert.expect( 1 ); diff --git a/test/unit/deprecated.js b/test/unit/deprecated.js index d6a15cd7e..dc82eb6cf 100644 --- a/test/unit/deprecated.js +++ b/test/unit/deprecated.js @@ -77,7 +77,7 @@ QUnit[ jQuery.fn.click ? "test" : "skip" ]( "trigger() shortcuts", function( ass elem.remove(); jQuery( "#check1" ).click( function() { - assert.ok( true, "click event handler for checkbox gets fired twice, see #815" ); + assert.ok( true, "click event handler for checkbox gets fired twice, see trac-815" ); } ).click(); counter = 0; diff --git a/test/unit/dimensions.js b/test/unit/dimensions.js index a44857910..48225999c 100644 --- a/test/unit/dimensions.js +++ b/test/unit/dimensions.js @@ -38,7 +38,7 @@ function testWidth( val, assert ) { $div.css( "display", "none" ); assert.equal( $div.width(), 30, "Test hidden div" ); $div.css( "display", "" ); - $div.width( val( -1 ) ); // handle negative numbers by setting to 0 #11604 + $div.width( val( -1 ) ); // handle negative numbers by setting to 0 trac-11604 assert.equal( $div.width(), 0, "Test negative width normalized to 0" ); $div.css( "padding", "20px" ); assert.equal( $div.width(), 0, "Test padding specified with pixels" ); @@ -89,7 +89,7 @@ function testHeight( val, assert ) { $div.css( "display", "none" ); assert.equal( $div.height(), 30, "Test hidden div" ); $div.css( "display", "" ); - $div.height( val( -1 ) ); // handle negative numbers by setting to 0 #11604 + $div.height( val( -1 ) ); // handle negative numbers by setting to 0 trac-11604 assert.equal( $div.height(), 0, "Test negative height normalized to 0" ); $div.css( "padding", "20px" ); assert.equal( $div.height(), 0, "Test padding specified with pixels" ); @@ -279,7 +279,7 @@ QUnit.test( "outerHeight()", function( assert ) { div.remove(); } ); -QUnit.test( "child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see #9441 #9300", function( assert ) { +QUnit.test( "child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see trac-9441 trac-9300", function( assert ) { assert.expect( 16 ); // setup html @@ -290,33 +290,33 @@ QUnit.test( "child of a hidden elem (or unconnected node) has accurate inner/out $divNormal.appendTo( "body" ); // tests that child div of a hidden div works the same as a normal div - assert.equal( $divChild.width(), $divNormal.width(), "child of a hidden element width() is wrong see #9441" ); - assert.equal( $divChild.innerWidth(), $divNormal.innerWidth(), "child of a hidden element innerWidth() is wrong see #9441" ); - assert.equal( $divChild.outerWidth(), $divNormal.outerWidth(), "child of a hidden element outerWidth() is wrong see #9441" ); - assert.equal( $divChild.outerWidth( true ), $divNormal.outerWidth( true ), "child of a hidden element outerWidth( true ) is wrong see #9300" ); + assert.equal( $divChild.width(), $divNormal.width(), "child of a hidden element width() is wrong see trac-9441" ); + assert.equal( $divChild.innerWidth(), $divNormal.innerWidth(), "child of a hidden element innerWidth() is wrong see trac-9441" ); + assert.equal( $divChild.outerWidth(), $divNormal.outerWidth(), "child of a hidden element outerWidth() is wrong see trac-9441" ); + assert.equal( $divChild.outerWidth( true ), $divNormal.outerWidth( true ), "child of a hidden element outerWidth( true ) is wrong see trac-9300" ); - assert.equal( $divChild.height(), $divNormal.height(), "child of a hidden element height() is wrong see #9441" ); - assert.equal( $divChild.innerHeight(), $divNormal.innerHeight(), "child of a hidden element innerHeight() is wrong see #9441" ); - assert.equal( $divChild.outerHeight(), $divNormal.outerHeight(), "child of a hidden element outerHeight() is wrong see #9441" ); - assert.equal( $divChild.outerHeight( true ), $divNormal.outerHeight( true ), "child of a hidden element outerHeight( true ) is wrong see #9300" ); + assert.equal( $divChild.height(), $divNormal.height(), "child of a hidden element height() is wrong see trac-9441" ); + assert.equal( $divChild.innerHeight(), $divNormal.innerHeight(), "child of a hidden element innerHeight() is wrong see trac-9441" ); + assert.equal( $divChild.outerHeight(), $divNormal.outerHeight(), "child of a hidden element outerHeight() is wrong see trac-9441" ); + assert.equal( $divChild.outerHeight( true ), $divNormal.outerHeight( true ), "child of a hidden element outerHeight( true ) is wrong see trac-9300" ); // tests that child div of an unconnected div works the same as a normal div - assert.equal( $divUnconnected.width(), $divNormal.width(), "unconnected element width() is wrong see #9441" ); - assert.equal( $divUnconnected.innerWidth(), $divNormal.innerWidth(), "unconnected element innerWidth() is wrong see #9441" ); - assert.equal( $divUnconnected.outerWidth(), $divNormal.outerWidth(), "unconnected element outerWidth() is wrong see #9441" ); - assert.equal( $divUnconnected.outerWidth( true ), $divNormal.outerWidth( true ), "unconnected element outerWidth( true ) is wrong see #9300" ); + assert.equal( $divUnconnected.width(), $divNormal.width(), "unconnected element width() is wrong see trac-9441" ); + assert.equal( $divUnconnected.innerWidth(), $divNormal.innerWidth(), "unconnected element innerWidth() is wrong see trac-9441" ); + assert.equal( $divUnconnected.outerWidth(), $divNormal.outerWidth(), "unconnected element outerWidth() is wrong see trac-9441" ); + assert.equal( $divUnconnected.outerWidth( true ), $divNormal.outerWidth( true ), "unconnected element outerWidth( true ) is wrong see trac-9300" ); - assert.equal( $divUnconnected.height(), $divNormal.height(), "unconnected element height() is wrong see #9441" ); - assert.equal( $divUnconnected.innerHeight(), $divNormal.innerHeight(), "unconnected element innerHeight() is wrong see #9441" ); - assert.equal( $divUnconnected.outerHeight(), $divNormal.outerHeight(), "unconnected element outerHeight() is wrong see #9441" ); - assert.equal( $divUnconnected.outerHeight( true ), $divNormal.outerHeight( true ), "unconnected element outerHeight( true ) is wrong see #9300" ); + assert.equal( $divUnconnected.height(), $divNormal.height(), "unconnected element height() is wrong see trac-9441" ); + assert.equal( $divUnconnected.innerHeight(), $divNormal.innerHeight(), "unconnected element innerHeight() is wrong see trac-9441" ); + assert.equal( $divUnconnected.outerHeight(), $divNormal.outerHeight(), "unconnected element outerHeight() is wrong see trac-9441" ); + assert.equal( $divUnconnected.outerHeight( true ), $divNormal.outerHeight( true ), "unconnected element outerHeight( true ) is wrong see trac-9300" ); // teardown html $divHiddenParent.remove(); $divNormal.remove(); } ); -QUnit.test( "getting dimensions shouldn't modify runtimeStyle see #9233", function( assert ) { +QUnit.test( "getting dimensions shouldn't modify runtimeStyle see trac-9233", function( assert ) { assert.expect( 1 ); var $div = jQuery( "
" ).appendTo( "#qunit-fixture" ), @@ -331,9 +331,9 @@ QUnit.test( "getting dimensions shouldn't modify runtimeStyle see #9233", functi $div.outerWidth( true ); if ( runtimeStyle ) { - assert.equal( div.runtimeStyle.left, "11em", "getting dimensions modifies runtimeStyle, see #9233" ); + assert.equal( div.runtimeStyle.left, "11em", "getting dimensions modifies runtimeStyle, see trac-9233" ); } else { - assert.ok( true, "this browser doesn't support runtimeStyle, see #9233" ); + assert.ok( true, "this browser doesn't support runtimeStyle, see trac-9233" ); } $div.remove(); @@ -348,8 +348,8 @@ QUnit.test( "table dimensions", function( assert ) { table.find( "td" ).css( { "margin": 0, "padding": 0 } ); - assert.equal( tdElem.width(), tdElem.width(), "width() doesn't alter dimension values of empty cells, see #11293" ); - assert.equal( colElem.width(), 300, "col elements have width(), see #12243" ); + assert.equal( tdElem.width(), tdElem.width(), "width() doesn't alter dimension values of empty cells, see trac-11293" ); + assert.equal( colElem.width(), 300, "col elements have width(), see trac-12243" ); } ); QUnit.test( "SVG dimensions (basic content-box)", function( assert ) { @@ -412,7 +412,7 @@ QUnit.test( "SVG dimensions (border-box)", function( assert ) { svg.remove(); } ); -QUnit.test( "box-sizing:border-box child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see #10413", function( assert ) { +QUnit.test( "box-sizing:border-box child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see trac-10413", function( assert ) { assert.expect( 16 ); // setup html @@ -423,38 +423,38 @@ QUnit.test( "box-sizing:border-box child of a hidden elem (or unconnected node) $divNormal.appendTo( "body" ); // tests that child div of a hidden div works the same as a normal div - assert.equal( $divChild.width(), $divNormal.width(), "child of a hidden element width() is wrong see #10413" ); - assert.equal( $divChild.innerWidth(), $divNormal.innerWidth(), "child of a hidden element innerWidth() is wrong see #10413" ); - assert.equal( $divChild.outerWidth(), $divNormal.outerWidth(), "child of a hidden element outerWidth() is wrong see #10413" ); - assert.equal( $divChild.outerWidth( true ), $divNormal.outerWidth( true ), "child of a hidden element outerWidth( true ) is wrong see #10413" ); + assert.equal( $divChild.width(), $divNormal.width(), "child of a hidden element width() is wrong see trac-10413" ); + assert.equal( $divChild.innerWidth(), $divNormal.innerWidth(), "child of a hidden element innerWidth() is wrong see trac-10413" ); + assert.equal( $divChild.outerWidth(), $divNormal.outerWidth(), "child of a hidden element outerWidth() is wrong see trac-10413" ); + assert.equal( $divChild.outerWidth( true ), $divNormal.outerWidth( true ), "child of a hidden element outerWidth( true ) is wrong see trac-10413" ); - assert.equal( $divChild.height(), $divNormal.height(), "child of a hidden element height() is wrong see #10413" ); - assert.equal( $divChild.innerHeight(), $divNormal.innerHeight(), "child of a hidden element innerHeight() is wrong see #10413" ); - assert.equal( $divChild.outerHeight(), $divNormal.outerHeight(), "child of a hidden element outerHeight() is wrong see #10413" ); - assert.equal( $divChild.outerHeight( true ), $divNormal.outerHeight( true ), "child of a hidden element outerHeight( true ) is wrong see #10413" ); + assert.equal( $divChild.height(), $divNormal.height(), "child of a hidden element height() is wrong see trac-10413" ); + assert.equal( $divChild.innerHeight(), $divNormal.innerHeight(), "child of a hidden element innerHeight() is wrong see trac-10413" ); + assert.equal( $divChild.outerHeight(), $divNormal.outerHeight(), "child of a hidden element outerHeight() is wrong see trac-10413" ); + assert.equal( $divChild.outerHeight( true ), $divNormal.outerHeight( true ), "child of a hidden element outerHeight( true ) is wrong see trac-10413" ); // tests that child div of an unconnected div works the same as a normal div - assert.equal( $divUnconnected.width(), $divNormal.width(), "unconnected element width() is wrong see #10413" ); - assert.equal( $divUnconnected.innerWidth(), $divNormal.innerWidth(), "unconnected element innerWidth() is wrong see #10413" ); - assert.equal( $divUnconnected.outerWidth(), $divNormal.outerWidth(), "unconnected element outerWidth() is wrong see #10413" ); - assert.equal( $divUnconnected.outerWidth( true ), $divNormal.outerWidth( true ), "unconnected element outerWidth( true ) is wrong see #10413" ); + assert.equal( $divUnconnected.width(), $divNormal.width(), "unconnected element width() is wrong see trac-10413" ); + assert.equal( $divUnconnected.innerWidth(), $divNormal.innerWidth(), "unconnected element innerWidth() is wrong see trac-10413" ); + assert.equal( $divUnconnected.outerWidth(), $divNormal.outerWidth(), "unconnected element outerWidth() is wrong see trac-10413" ); + assert.equal( $divUnconnected.outerWidth( true ), $divNormal.outerWidth( true ), "unconnected element outerWidth( true ) is wrong see trac-10413" ); - assert.equal( $divUnconnected.height(), $divNormal.height(), "unconnected element height() is wrong see #10413" ); - assert.equal( $divUnconnected.innerHeight(), $divNormal.innerHeight(), "unconnected element innerHeight() is wrong see #10413" ); - assert.equal( $divUnconnected.outerHeight(), $divNormal.outerHeight(), "unconnected element outerHeight() is wrong see #10413" ); - assert.equal( $divUnconnected.outerHeight( true ), $divNormal.outerHeight( true ), "unconnected element outerHeight( true ) is wrong see #10413" ); + assert.equal( $divUnconnected.height(), $divNormal.height(), "unconnected element height() is wrong see trac-10413" ); + assert.equal( $divUnconnected.innerHeight(), $divNormal.innerHeight(), "unconnected element innerHeight() is wrong see trac-10413" ); + assert.equal( $divUnconnected.outerHeight(), $divNormal.outerHeight(), "unconnected element outerHeight() is wrong see trac-10413" ); + assert.equal( $divUnconnected.outerHeight( true ), $divNormal.outerHeight( true ), "unconnected element outerHeight( true ) is wrong see trac-10413" ); // teardown html $divHiddenParent.remove(); $divNormal.remove(); } ); -QUnit.test( "passing undefined is a setter #5571", function( assert ) { +QUnit.test( "passing undefined is a setter trac-5571", function( assert ) { assert.expect( 4 ); - assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).height( undefined ).height(), 30, ".height(undefined) is chainable (#5571)" ); - assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).innerHeight( undefined ).height(), 30, ".innerHeight(undefined) is chainable (#5571)" ); - assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).outerHeight( undefined ).height(), 30, ".outerHeight(undefined) is chainable (#5571)" ); - assert.equal( jQuery( "#nothiddendiv" ).width( 30 ).width( undefined ).width(), 30, ".width(undefined) is chainable (#5571)" ); + assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).height( undefined ).height(), 30, ".height(undefined) is chainable (trac-5571)" ); + assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).innerHeight( undefined ).height(), 30, ".innerHeight(undefined) is chainable (trac-5571)" ); + assert.equal( jQuery( "#nothiddendiv" ).height( 30 ).outerHeight( undefined ).height(), 30, ".outerHeight(undefined) is chainable (trac-5571)" ); + assert.equal( jQuery( "#nothiddendiv" ).width( 30 ).width( undefined ).width(), 30, ".width(undefined) is chainable (trac-5571)" ); } ); QUnit.test( "setters with and without box-sizing:border-box", function( assert ) { diff --git a/test/unit/effects.js b/test/unit/effects.js index 2278b3afb..a4c91b331 100644 --- a/test/unit/effects.js +++ b/test/unit/effects.js @@ -141,7 +141,7 @@ supportjQuery.each( hideOptions, function( type, setup ) { // Note: inline elements are expected to be inline-block // because we're showing width/height // Can't animate width/height inline - // See #14344 + // See trac-14344 var test = { "div": "block", "p": "block", @@ -184,7 +184,7 @@ supportjQuery.each( hideOptions, function( type, setup ) { } ); } ); -// Supports #7397 +// Supports trac-7397 supportjQuery.each( hideOptions, function( type, setup ) { QUnit.test( "Persist correct display value - " + type + " hidden", function( assert ) { assert.expect( 3 ); @@ -677,7 +677,7 @@ QUnit.test( "stop()", function( assert ) { } ); this.clock.tick( 100 ); $two.fadeTo( 100, 0, function() { - assert.equal( $two.css( "opacity" ), "0", "Stop does not interfere with animations on other elements (#6641)" ); + assert.equal( $two.css( "opacity" ), "0", "Stop does not interfere with animations on other elements (trac-6641)" ); // Reset styles $one.add( $two ).css( "opacity", "" ); @@ -832,7 +832,7 @@ QUnit.test( "jQuery.fx.prototype.cur() - <1.8 Back Compat", function( assert ) { assert.equal( ( new jQuery.fx( div, {}, "color" ) ).cur(), jQuery.css( div, "color" ), - "Return the same value as jQuery.css for complex properties (bug #7912)" + "Return the same value as jQuery.css for complex properties (bug trac-7912)" ); assert.strictEqual( @@ -869,7 +869,7 @@ QUnit.test( "jQuery.fx.prototype.cur() - <1.8 Back Compat", function( assert ) { assert.equal( ( new jQuery.fx( div, {}, "marginBottom" ) ).cur(), -11000, - "support negative values < -10000 (bug #7193)" + "support negative values < -10000 (bug trac-7193)" ); jQuery( div ).remove(); @@ -1121,7 +1121,7 @@ jQuery.makeTest = function( text ) { jQuery.makeTest.id = 1; -QUnit.test( "jQuery.show('fast') doesn't clear radio buttons (bug #1095)", function( assert ) { +QUnit.test( "jQuery.show('fast') doesn't clear radio buttons (bug trac-1095)", function( assert ) { assert.expect( 4 ); var $checkedtest = jQuery( "#checkedtest" ); @@ -1279,7 +1279,7 @@ QUnit.test( "animate with CSS shorthand properties", function( assert ) { this.clock.tick( 400 ); } ); -QUnit.test( "hide hidden elements, with animation (bug #7141)", function( assert ) { +QUnit.test( "hide hidden elements, with animation (bug trac-7141)", function( assert ) { assert.expect( 4 ); var div = jQuery( "" ).appendTo( "#qunit-fixture" ); @@ -1294,7 +1294,7 @@ QUnit.test( "hide hidden elements, with animation (bug #7141)", function( assert assert.equal( div.css( "display" ), "block", "Element is visible after animations" ); } ); -QUnit.test( "animate unit-less properties (#4966)", function( assert ) { +QUnit.test( "animate unit-less properties (trac-4966)", function( assert ) { assert.expect( 2 ); var div = jQuery( "
" ).appendTo( "#qunit-fixture" ); @@ -1305,7 +1305,7 @@ QUnit.test( "animate unit-less properties (#4966)", function( assert ) { this.clock.tick( 400 ); } ); -QUnit.test( "animate properties missing px w/ opacity as last (#9074)", function( assert ) { +QUnit.test( "animate properties missing px w/ opacity as last (trac-9074)", function( assert ) { assert.expect( 6 ); var ml, l, @@ -1333,7 +1333,7 @@ QUnit.test( "animate properties missing px w/ opacity as last (#9074)", function div.stop().remove(); } ); -QUnit.test( "callbacks should fire in correct order (#9100)", function( assert ) { +QUnit.test( "callbacks should fire in correct order (trac-9100)", function( assert ) { assert.expect( 1 ); var a = 1, @@ -1352,7 +1352,7 @@ QUnit.test( "callbacks should fire in correct order (#9100)", function( assert ) this.clock.tick( 20 ); } ); -QUnit.test( "callbacks that throw exceptions will be removed (#5684)", function( assert ) { +QUnit.test( "callbacks that throw exceptions will be removed (trac-5684)", function( assert ) { assert.expect( 2 ); var foo = jQuery( "#foo" ); @@ -1405,7 +1405,7 @@ QUnit.test( "animate will scale margin properties individually", function( asser } ); } ); -QUnit.test( "Do not append px to 'fill-opacity' #9548", function( assert ) { +QUnit.test( "Do not append px to 'fill-opacity' trac-9548", function( assert ) { assert.expect( 1 ); var $div = jQuery( "
" ).appendTo( "#qunit-fixture" ); @@ -1416,7 +1416,7 @@ QUnit.test( "Do not append px to 'fill-opacity' #9548", function( assert ) { } ); } ); -QUnit.test( "line-height animates correctly (#13855)", function( assert ) { +QUnit.test( "line-height animates correctly (trac-13855)", function( assert ) { assert.expect( 12 ); var t0, @@ -1552,7 +1552,7 @@ QUnit.test( "Animate callbacks have correct context", function( assert ) { this.clock.tick( 10 ); } ); -QUnit.test( "User supplied callback called after show when fx off (#8892)", function( assert ) { +QUnit.test( "User supplied callback called after show when fx off (trac-8892)", function( assert ) { assert.expect( 2 ); var foo = jQuery( "#foo" ); @@ -1628,7 +1628,7 @@ QUnit.test( "animate should set display for disconnected nodes", function( asser clock.tick( 400 ); } ); -QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Animation callback should not show animated element as :animated (#7157)", function( assert ) { +QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Animation callback should not show animated element as :animated (trac-7157)", function( assert ) { assert.expect( 1 ); var foo = jQuery( "#foo" ); @@ -1641,7 +1641,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Animation callback should not this.clock.tick( 100 ); } ); -QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Initial step callback should show element as :animated (#14623)", function( assert ) { +QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Initial step callback should show element as :animated (trac-14623)", function( assert ) { assert.expect( 1 ); var foo = jQuery( "#foo" ); @@ -1658,7 +1658,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "Initial step callback should foo.stop(); } ); -QUnit.test( "hide called on element within hidden parent should set display to none (#10045)", function( assert ) { +QUnit.test( "hide called on element within hidden parent should set display to none (trac-10045)", function( assert ) { assert.expect( 3 ); var hidden = jQuery( ".hidden" ), @@ -1711,7 +1711,7 @@ QUnit.test( "hide, fadeOut and slideUp called on element width height and width this.clock.tick( 400 ); } ); -QUnit.test( "hide should not leave hidden inline elements visible (#14848)", function( assert ) { +QUnit.test( "hide should not leave hidden inline elements visible (trac-14848)", function( assert ) { assert.expect( 2 ); var el = jQuery( "#simon1" ); @@ -1814,7 +1814,7 @@ QUnit.test( "multiple unqueued and promise", function( assert ) { this.clock.tick( 1000 ); } ); -QUnit.test( "animate does not change start value for non-px animation (#7109)", function( assert ) { +QUnit.test( "animate does not change start value for non-px animation (trac-7109)", function( assert ) { assert.expect( 1 ); var parent = jQuery( "
" ).css( { width: 284, height: 1 } ).appendTo( "#qunit-fixture" ), @@ -1837,7 +1837,7 @@ QUnit.test( "animate does not change start value for non-px animation (#7109)", this.clock.tick( 10 ); } ); -QUnit.test( "non-px animation handles non-numeric start (#11971)", function( assert ) { +QUnit.test( "non-px animation handles non-numeric start (trac-11971)", function( assert ) { assert.expect( 2 ); var foo = jQuery( "#foo" ), @@ -1869,7 +1869,7 @@ QUnit.test( "non-px animation handles non-numeric start (#11971)", function( ass this.clock.tick( 10 ); } ); -QUnit.test( "Animation callbacks (#11797)", function( assert ) { +QUnit.test( "Animation callbacks (trac-11797)", function( assert ) { assert.expect( 15 ); var prog = 0, @@ -1949,7 +1949,7 @@ QUnit.test( "Animation callbacks (#11797)", function( assert ) { this.clock.tick( 10 ); } ); -QUnit.test( "Animation callbacks in order (#2292)", function( assert ) { +QUnit.test( "Animation callbacks in order (trac-2292)", function( assert ) { assert.expect( 9 ); var done = assert.async(), @@ -1989,7 +1989,7 @@ QUnit.test( "Animation callbacks in order (#2292)", function( assert ) { this.clock.tick( dur + 10 ); } ); -QUnit.test( "Animate properly sets overflow hidden when animating width/height (#12117)", function( assert ) { +QUnit.test( "Animate properly sets overflow hidden when animating width/height (trac-12117)", function( assert ) { assert.expect( 8 ); jQuery.each( [ "height", "width" ], function( _, prop ) { @@ -2007,7 +2007,7 @@ QUnit.test( "Animate properly sets overflow hidden when animating width/height ( } ); } ); -QUnit.test( "Each tick of the timer loop uses a fresh time (#12837)", function( assert ) { +QUnit.test( "Each tick of the timer loop uses a fresh time (trac-12837)", function( assert ) { var lastVal, tmp = jQuery( { test: 0 @@ -2032,7 +2032,7 @@ QUnit.test( "Each tick of the timer loop uses a fresh time (#12837)", function( tmp.stop(); } ); -QUnit.test( "Animations with 0 duration don't ease (#12273)", function( assert ) { +QUnit.test( "Animations with 0 duration don't ease (trac-12273)", function( assert ) { assert.expect( 1 ); jQuery.easing.test = function() { @@ -2057,7 +2057,7 @@ jQuery.map( [ "toggle", "slideToggle", "fadeToggle" ], function( method ) { // this test would look a lot better if we were using something to override // the default timers var duration = 1500; - QUnit.test( "toggle state tests: " + method + " (#8685)", function( assert ) { + QUnit.test( "toggle state tests: " + method + " (trac-8685)", function( assert ) { function secondToggle() { var stopped = parseFloat( element.css( check ) ); tested = false; @@ -2286,7 +2286,7 @@ QUnit.test( ".finish() calls finish of custom queue functions", function( assert div.remove(); } ); -QUnit.test( ".finish() is applied correctly when multiple elements were animated (#13937)", function( assert ) { +QUnit.test( ".finish() is applied correctly when multiple elements were animated (trac-13937)", function( assert ) { assert.expect( 3 ); var elems = jQuery( "012" ); @@ -2303,7 +2303,7 @@ QUnit.test( ".finish() is applied correctly when multiple elements were animated this.clock.tick( 1500 ); } ); -QUnit.test( "slideDown() after stop() (#13483)", function( assert ) { +QUnit.test( "slideDown() after stop() (trac-13483)", function( assert ) { assert.expect( 2 ); var ul = jQuery( "
    " ) @@ -2336,7 +2336,7 @@ QUnit.test( "slideDown() after stop() (#13483)", function( assert ) { clock.tick( 10 ); } ); -QUnit.test( "Respect display value on inline elements (#14824)", function( assert ) { +QUnit.test( "Respect display value on inline elements (trac-14824)", function( assert ) { assert.expect( 2 ); var clock = this.clock, diff --git a/test/unit/event.js b/test/unit/event.js index 1f76c54b9..8ca06ced8 100644 --- a/test/unit/event.js +++ b/test/unit/event.js @@ -8,7 +8,7 @@ QUnit.module( "event", { QUnit.test( "null or undefined handler", function( assert ) { assert.expect( 4 ); - // Supports Fixes bug #7229 + // Supports Fixes bug trac-7229 try { jQuery( "#firstp" ).on( "click", null ); assert.ok( true, "Passing a null handler will not throw an exception" ); @@ -484,7 +484,7 @@ QUnit.test( "trigger() works with events that were previously stopped", function QUnit.test( "on(), iframes", function( assert ) { assert.expect( 1 ); - // events don't work with iframes, see #939 - this test fails in IE because of contentDocument + // events don't work with iframes, see trac-939 - this test fails in IE because of contentDocument var doc = jQuery( "#loadediframe" ).contents(); jQuery( "div", doc ).on( "click", function() { @@ -552,7 +552,7 @@ QUnit.test( "on(), namespaced events, cloned events", function( assert ) { assert.equal( this.nodeType, 1, "Check node,textnode,comment on just does real nodes" ); } ).trigger( "tester" ); - // Make sure events stick with appendTo'd elements (which are cloned) #2027 + // Make sure events stick with appendTo'd elements (which are cloned) trac-2027 jQuery( "test" ).on( "click", function() { return false; } ).appendTo( "#qunit-fixture" ); assert.ok( jQuery( "a.test" ).eq( 0 ).triggerHandler( "click" ) === false, "Handler is bound to appendTo'd elements" ); } ); @@ -589,7 +589,7 @@ QUnit.test( "on(), multi-namespaced events", function( assert ) { check( "click.test.abc", "Namespaced click triggered" ); } ); - // Those would not trigger/off (#5303) + // Those would not trigger/off (trac-5303) jQuery( "#firstp" ).trigger( "click.a.test" ); jQuery( "#firstp" ).off( "click.a.test" ); @@ -812,7 +812,7 @@ QUnit.test( "off(type)", function( assert ) { .off( "error1 error2", error ) .trigger( "error1" ).triggerHandler( "error2" ); - message = "unbind many"; // #3538 + message = "unbind many"; // trac-3538 $elem.on( "error1 error2", error ) .off( "error1 error2" ) .trigger( "error1" ).triggerHandler( "error2" ); @@ -1041,7 +1041,7 @@ QUnit.test( "trigger(type, [data], [fn])", function( assert ) { } catch ( e ) { pass = false; } - assert.ok( pass, "Trigger on a table with a colon in the even type, see #3533" ); + assert.ok( pass, "Trigger on a table with a colon in the even type, see trac-3533" ); form = jQuery( "
    " ).appendTo( "body" ); @@ -1069,7 +1069,7 @@ QUnit.test( "trigger(type, [data], [fn])", function( assert ) { form.remove(); } ); -QUnit.test( "submit event bubbles on copied forms (#11649)", function( assert ) { +QUnit.test( "submit event bubbles on copied forms (trac-11649)", function( assert ) { assert.expect( 3 ); var $formByClone, $formByHTML, @@ -1105,7 +1105,7 @@ QUnit.test( "submit event bubbles on copied forms (#11649)", function( assert ) $testForm.off( "submit", noSubmit ); } ); -QUnit.test( "change event bubbles on copied forms (#11796)", function( assert ) { +QUnit.test( "change event bubbles on copied forms (trac-11796)", function( assert ) { assert.expect( 3 ); var $formByClone, $formByHTML, @@ -1227,7 +1227,7 @@ QUnit.test( "trigger(eventObject, [data], [fn])", function( assert ) { assert.equal( event.isDefaultPrevented(), false, "default not prevented" ); } ); -QUnit.test( ".trigger() bubbling on disconnected elements (#10489)", function( assert ) { +QUnit.test( ".trigger() bubbling on disconnected elements (trac-10489)", function( assert ) { assert.expect( 2 ); jQuery( window ).on( "click", function() { @@ -1251,7 +1251,7 @@ QUnit.test( ".trigger() bubbling on disconnected elements (#10489)", function( a jQuery( window ).off( "click" ); } ); -QUnit.test( ".trigger() doesn't bubble load event (#10717)", function( assert ) { +QUnit.test( ".trigger() doesn't bubble load event (trac-10717)", function( assert ) { assert.expect( 1 ); jQuery( window ).on( "load", function() { @@ -1269,7 +1269,7 @@ QUnit.test( ".trigger() doesn't bubble load event (#10717)", function( assert ) jQuery( window ).off( "load" ); } ); -QUnit.test( "Delegated events in SVG (#10791; #13180)", function( assert ) { +QUnit.test( "Delegated events in SVG (trac-10791; trac-13180)", function( assert ) { assert.expect( 2 ); var useElem, e, @@ -1295,7 +1295,7 @@ QUnit.test( "Delegated events in SVG (#10791; #13180)", function( assert ) { .end(); // Fire a native click on an SVGElementInstance (the instance tree of an SVG ) - // to confirm that it doesn't break our event delegation handling (#13180) + // to confirm that it doesn't break our event delegation handling (trac-13180) useElem = svg.find( "#use" )[ 0 ]; if ( document.createEvent && useElem && useElem.instanceRoot ) { e = document.createEvent( "MouseEvents" ); @@ -1321,7 +1321,7 @@ QUnit.test( "Delegated events with malformed selectors (gh-3071)", function( ass assert.ok( true, "malformed selector does not throw on event" ); } ); -QUnit.test( "Delegated events in forms (#10844; #11145; #8165; #11382, #11764)", function( assert ) { +QUnit.test( "Delegated events in forms (trac-10844; trac-11145; trac-8165; trac-11382, trac-11764)", function( assert ) { assert.expect( 5 ); // Alias names like "id" cause havoc @@ -1376,7 +1376,7 @@ QUnit.test( "Delegated events in forms (#10844; #11145; #8165; #11382, #11764)", form.remove(); } ); -QUnit.test( "Submit event can be stopped (#11049)", function( assert ) { +QUnit.test( "Submit event can be stopped (trac-11049)", function( assert ) { assert.expect( 1 ); // Since we manually bubble in IE, make sure inner handlers get a chance to cancel @@ -1450,7 +1450,7 @@ QUnit.test( "jQuery.Event( type, props )", function( assert ) { assert.equal( event.type, "keydown", "Verify type" ); // ensure "type" in props won't clobber the one set by constructor - assert.equal( jQuery.inArray( "type", jQuery.event.props ), -1, "'type' property not in props (#10375)" ); + assert.equal( jQuery.inArray( "type", jQuery.event.props ), -1, "'type' property not in props (trac-10375)" ); assert.ok( "keyCode" in event, "Special 'keyCode' property exists" ); @@ -1844,7 +1844,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "delegated event with delegate assert.expect( 3 ); var markup = jQuery( "
    " ).appendTo( "#qunit-fixture" ); - // Non-positional selector (#12383) + // Non-positional selector (trac-12383) markup.find( "#ul0" ) .on( "click", "div li a", function() { assert.ok( false, "div is ABOVE the delegation point!" ); @@ -1859,7 +1859,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "delegated event with delegate .off( "click" ); if ( QUnit.jQuerySelectorsPos ) { - // Positional selector (#11315) + // Positional selector (trac-11315) markup.find( "ul" ).eq( 0 ) .on( "click", ">li>a", function() { assert.ok( this.id === "a0", "child li was clicked" ); @@ -1879,7 +1879,7 @@ QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "delegated event with delegate markup.remove(); } ); -QUnit.test( "delegated event with selector matching Object.prototype property (#13203)", function( assert ) { +QUnit.test( "delegated event with selector matching Object.prototype property (trac-13203)", function( assert ) { assert.expect( 1 ); var matched = 0; @@ -1893,7 +1893,7 @@ QUnit.test( "delegated event with selector matching Object.prototype property (# assert.equal( matched, 0, "Nothing matched 'toString'" ); } ); -QUnit.test( "delegated event with intermediate DOM manipulation (#13208)", function( assert ) { +QUnit.test( "delegated event with intermediate DOM manipulation (trac-13208)", function( assert ) { assert.expect( 1 ); jQuery( "#foo" ).on( "click", "[id=sap]", function() {} ); @@ -2187,7 +2187,7 @@ QUnit.test( "focusin bubbles", function( assert ) { jQuery( "body" ).off( "focusin.focusinBubblesTest" ); } ); -QUnit.test( "custom events with colons (#3533, #8272)", function( assert ) { +QUnit.test( "custom events with colons (trac-3533, trac-8272)", function( assert ) { assert.expect( 1 ); var tab = jQuery( "
    trigger
    " ).appendTo( "body" ); @@ -2370,7 +2370,7 @@ QUnit.test( "special on name mapping", function( assert ) { delete jQuery.event.special[ "gutfeeling" ]; } ); -QUnit.test( ".on and .off, selective mixed removal (#10705)", function( assert ) { +QUnit.test( ".on and .off, selective mixed removal (trac-10705)", function( assert ) { assert.expect( 7 ); var timingx = function( e ) { @@ -2392,7 +2392,7 @@ QUnit.test( ".on and .off, selective mixed removal (#10705)", function( assert ) .trigger( "click" ); // 0 } ); -QUnit.test( ".on( event-map, null-selector, data ) #11130", function( assert ) { +QUnit.test( ".on( event-map, null-selector, data ) trac-11130", function( assert ) { assert.expect( 1 ); @@ -2408,7 +2408,7 @@ QUnit.test( ".on( event-map, null-selector, data ) #11130", function( assert ) { $p.on( map, null, data ).trigger( "foo" ); } ); -QUnit.test( "clone() delegated events (#11076)", function( assert ) { +QUnit.test( "clone() delegated events (trac-11076)", function( assert ) { assert.expect( 3 ); var counter = { "center": 0, "fold": 0, "centerfold": 0 }, @@ -2745,7 +2745,7 @@ QUnit.test( "trigger click on checkbox, fires change event", function( assert ) } ).trigger( "click" ); } ); -QUnit.test( "Namespace preserved when passed an Event (#12739)", function( assert ) { +QUnit.test( "Namespace preserved when passed an Event (trac-12739)", function( assert ) { assert.expect( 4 ); var markup = jQuery( @@ -2833,7 +2833,7 @@ QUnit.test( "make sure events cloned correctly", function( assert ) { clone.find( "#check1" ).trigger( "change" ); // 0 events should fire } ); -QUnit.test( "String.prototype.namespace does not cause trigger() to throw (#13360)", function( assert ) { +QUnit.test( "String.prototype.namespace does not cause trigger() to throw (trac-13360)", function( assert ) { assert.expect( 1 ); var errored = false; @@ -2848,7 +2848,7 @@ QUnit.test( "String.prototype.namespace does not cause trigger() to throw (#1336 delete String.prototype.namespace; } ); -QUnit.test( "Inline event result is returned (#13993)", function( assert ) { +QUnit.test( "Inline event result is returned (trac-13993)", function( assert ) { assert.expect( 1 ); var result = jQuery( "

    hello

    " ).triggerHandler( "click" ); @@ -3137,7 +3137,7 @@ QUnit.test( "focusout/focusin support", function( assert ) { }, 50 ); } ); -QUnit.test( "focus-blur order (#12868)", function( assert ) { +QUnit.test( "focus-blur order (trac-12868)", function( assert ) { assert.expect( 5 ); var order, diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js index 30bf169ac..a6633ec45 100644 --- a/test/unit/manipulation.js +++ b/test/unit/manipulation.js @@ -39,9 +39,9 @@ QUnit.test( "text()", function( assert ) { // Check serialization of text values assert.equal( jQuery( document.createTextNode( "foo" ) ).text(), "foo", "Text node was retrieved from .text()." ); - assert.notEqual( jQuery( document ).text(), "", "Retrieving text for the document retrieves all text (#10724)." ); + assert.notEqual( jQuery( document ).text(), "", "Retrieving text for the document retrieves all text (trac-10724)." ); - // Retrieve from document fragments #10864 + // Retrieve from document fragments trac-10864 frag = document.createDocumentFragment(); frag.appendChild( document.createTextNode( "foo" ) ); @@ -49,7 +49,7 @@ QUnit.test( "text()", function( assert ) { $newLineTest = jQuery( "
    test
    testy
    " ).appendTo( "#moretests" ); $newLineTest.find( "br" ).replaceWith( "\n" ); - assert.equal( $newLineTest.text(), "test\ntesty", "text() does not remove new lines (#11153)" ); + assert.equal( $newLineTest.text(), "test\ntesty", "text() does not remove new lines (trac-11153)" ); $newLineTest.remove(); } ); @@ -58,7 +58,7 @@ QUnit.test( "text(undefined)", function( assert ) { assert.expect( 1 ); - assert.equal( jQuery( "#foo" ).text( "Hello
    " ).add( "
    World
    " ); $multipleElements.text( expected ); - assert.equal( $multipleElements.eq( 0 ).text(), expected, "text() updates multiple elements (#11809)" ); - assert.equal( $multipleElements.eq( 1 ).text(), expected, "text() updates multiple elements (#11809)" ); + assert.equal( $multipleElements.eq( 0 ).text(), expected, "text() updates multiple elements (trac-11809)" ); + assert.equal( $multipleElements.eq( 1 ).text(), expected, "text() updates multiple elements (trac-11809)" ); - // Prevent memory leaks #11809 + // Prevent memory leaks trac-11809 $childDiv = jQuery( "
    " ); $childDiv.data( "leak", true ); $parentDiv = jQuery( "
    " ); @@ -187,7 +187,7 @@ function testAppendForObject( valueObj, isFragment, assert ) { assert.equal( $base.clone().append( valueObj( document.getElementById( "form" ).cloneNode( true ) ) ).children( "form" ).length, 1, - "Check for appending a form (#910)" + type + "Check for appending a form (trac-910)" + type ); } @@ -272,7 +272,7 @@ function testAppend( valueObj, assert ) { .append( valueObj( "" ) ); assert.t( "Append Select", "#appendSelect1, #appendSelect2", [ "appendSelect1", "appendSelect2" ] ); - assert.equal( "Two nodes", jQuery( "
    " ).append( "Two", " nodes" ).text(), "Appending two text nodes (#4011)" ); + assert.equal( "Two nodes", jQuery( "
    " ).append( "Two", " nodes" ).text(), "Appending two text nodes (trac-4011)" ); assert.equal( jQuery( "
    " ).append( "1", "", 3 ).text(), "13", "If median is false-like value, subsequent arguments should not be ignored" ); // using contents will get comments regular, text, and comment nodes @@ -310,7 +310,7 @@ QUnit.test( "append(Function)", function( assert ) { testAppend( manipulationFunctionReturningObj, assert ); } ); -QUnit.test( "append(param) to object, see #11280", function( assert ) { +QUnit.test( "append(param) to object, see trac-11280", function( assert ) { assert.expect( 5 ); @@ -402,7 +402,7 @@ QUnit.test( "append(Function) returns Number", function( assert ) { assert.ok( jQuery( "#sap" )[ 0 ].innerHTML.match( /5$/ ), "Check for appending a number" ); } ); -QUnit.test( "XML DOM manipulation (#9960)", function( assert ) { +QUnit.test( "XML DOM manipulation (trac-9960)", function( assert ) { assert.expect( 5 ); @@ -430,7 +430,7 @@ QUnit.test( "XML DOM manipulation (#9960)", function( assert ) { assert.deepEqual( jQuery( "state", xml2 ).get(), scxml1.find( "state" ).get(), "replaceWith" ); } ); -QUnit.test( "append HTML5 sectioning elements (Bug #6485)", function( assert ) { +QUnit.test( "append HTML5 sectioning elements (Bug trac-6485)", function( assert ) { assert.expect( 2 ); @@ -445,7 +445,7 @@ QUnit.test( "append HTML5 sectioning elements (Bug #6485)", function( assert ) { assert.equal( aside.length, 1, "HTML5 elements do not collapse their children" ); } ); -QUnit[ jQuery.fn.css ? "test" : "skip" ]( "HTML5 Elements inherit styles from style rules (Bug #10501)", function( assert ) { +QUnit[ jQuery.fn.css ? "test" : "skip" ]( "HTML5 Elements inherit styles from style rules (Bug trac-10501)", function( assert ) { assert.expect( 1 ); @@ -456,7 +456,7 @@ QUnit[ jQuery.fn.css ? "test" : "skip" ]( "HTML5 Elements inherit styles from st assert.notEqual( jQuery( "section" ).css( "background-color" ), "transparent", "HTML5 elements inherit styles" ); } ); -QUnit.test( "html(String) with HTML5 (Bug #6485)", function( assert ) { +QUnit.test( "html(String) with HTML5 (Bug trac-6485)", function( assert ) { assert.expect( 2 ); @@ -465,7 +465,7 @@ QUnit.test( "html(String) with HTML5 (Bug #6485)", function( assert ) { assert.equal( jQuery( "#qunit-fixture" ).children().children().children().length, 1, "Make sure nested HTML5 elements can hold children." ); } ); -QUnit.test( "html(String) tag-hyphenated elements (Bug #1987)", function( assert ) { +QUnit.test( "html(String) tag-hyphenated elements (Bug trac-1987)", function( assert ) { assert.expect( 27 ); @@ -561,7 +561,7 @@ QUnit.test( "IE8 serialization bug", function( assert ) { assert.equal( wrapper.children( "link" ).length, 1, "Link elements are insertable with .html()" ); } ); -QUnit.test( "html() object element #10324", function( assert ) { +QUnit.test( "html() object element trac-10324", function( assert ) { assert.expect( 1 ); @@ -959,7 +959,7 @@ QUnit.test( "before(no-op)", function( assert ) { assert.equal( set.length, 1, "Insert the element before the disconnected node. should be a no-op" ); } ); -QUnit.test( "before and after w/ empty object (#10812)", function( assert ) { +QUnit.test( "before and after w/ empty object (trac-10812)", function( assert ) { assert.expect( 1 ); @@ -1195,7 +1195,7 @@ function testReplaceWith( val, assert ) { assert.ok( !jQuery( "#baz" )[ 0 ], "Verify that original element is gone, after element" ); jQuery( "#bar" ).replaceWith( "
    ", "...", "
    " ); - assert.deepEqual( jQuery( "#yahoo, #baz" ).get(), q( "yahoo", "baz" ), "Replace element with multiple arguments (#13722)" ); + assert.deepEqual( jQuery( "#yahoo, #baz" ).get(), q( "yahoo", "baz" ), "Replace element with multiple arguments (trac-13722)" ); assert.strictEqual( jQuery( "#yahoo" )[ 0 ].nextSibling, jQuery( "#baz" )[ 0 ].previousSibling, "Argument order preserved" ); assert.deepEqual( jQuery( "#bar" ).get(), [], "Verify that original element is gone, after multiple arguments" ); @@ -1208,7 +1208,7 @@ function testReplaceWith( val, assert ) { assert.ok( !jQuery( "#groups" )[ 0 ], "Verify that original element is gone, after jQuery collection" ); jQuery( "#mark, #first" ).replaceWith( val( "" ) ); - assert.equal( jQuery( "#qunit-fixture .replacement" ).length, 4, "Replace multiple elements (#12449)" ); + assert.equal( jQuery( "#qunit-fixture .replacement" ).length, 4, "Replace multiple elements (trac-12449)" ); assert.deepEqual( jQuery( "#mark, #first" ).get(), [], "Verify that original elements are gone, after replace multiple" ); tmp = jQuery( "content" )[ 0 ]; @@ -1256,7 +1256,7 @@ function testReplaceWith( val, assert ) { "Self-replacement" ); $div.replaceWith( child ); assert.deepEqual( jQuery( "#qunit-fixture" ).children().first().get(), child.get(), - "Replacement with following sibling (#13810)" ); + "Replacement with following sibling (trac-13810)" ); assert.deepEqual( jQuery( ".pathological", "#qunit-fixture" ).get(), [], "Replacement with following sibling (context removed)" ); @@ -1376,7 +1376,7 @@ QUnit.test( "replaceAll(jQuery)", function( assert ) { assert.ok( !jQuery( "#yahoo" )[ 0 ], "Verify that original element is gone, after set of elements" ); } ); -QUnit.test( "jQuery.clone() (#8017)", function( assert ) { +QUnit.test( "jQuery.clone() (trac-8017)", function( assert ) { assert.expect( 2 ); @@ -1388,7 +1388,7 @@ QUnit.test( "jQuery.clone() (#8017)", function( assert ) { assert.equal( main.childNodes.length, clone.childNodes.length, "Simple child length to ensure a large dom tree copies correctly" ); } ); -QUnit.test( "append to multiple elements (#8070)", function( assert ) { +QUnit.test( "append to multiple elements (trac-8070)", function( assert ) { assert.expect( 2 ); @@ -1543,7 +1543,7 @@ QUnit.test( "clone()", function( assert ) { body.remove(); } ); -QUnit.test( "clone(script type=non-javascript) (#11359)", function( assert ) { +QUnit.test( "clone(script type=non-javascript) (trac-11359)", function( assert ) { assert.expect( 3 ); @@ -1556,7 +1556,7 @@ QUnit.test( "clone(script type=non-javascript) (#11359)", function( assert ) { dest.remove(); } ); -QUnit.test( "clone(form element) (Bug #3879, #6655)", function( assert ) { +QUnit.test( "clone(form element) (Bug trac-3879, trac-6655)", function( assert ) { assert.expect( 5 ); @@ -1581,7 +1581,7 @@ QUnit.test( "clone(form element) (Bug #3879, #6655)", function( assert ) { assert.equal( clone[ 0 ].defaultValue, "foo", "Textarea defaultValue cloned correctly" ); } ); -QUnit.test( "clone(multiple selected options) (Bug #8129)", function( assert ) { +QUnit.test( "clone(multiple selected options) (Bug trac-8129)", function( assert ) { assert.expect( 1 ); @@ -1630,14 +1630,14 @@ QUnit.test( "html(undefined)", function( assert ) { assert.expect( 1 ); - assert.equal( jQuery( "#foo" ).html( "test" ).html( undefined ).html().toLowerCase(), "test", ".html(undefined) is chainable (#5571)" ); + assert.equal( jQuery( "#foo" ).html( "test" ).html( undefined ).html().toLowerCase(), "test", ".html(undefined) is chainable (trac-5571)" ); } ); QUnit.test( "html() on empty set", function( assert ) { assert.expect( 1 ); - assert.strictEqual( jQuery().html(), undefined, ".html() returns undefined for empty sets (#11962)" ); + assert.strictEqual( jQuery().html(), undefined, ".html() returns undefined for empty sets (trac-11962)" ); } ); function childNodeNames( node ) { @@ -1738,7 +1738,7 @@ function testHtml( valueObj, assert ) { fixture.html( valueObj( "" ) ); fixture.html( valueObj( "" ) ); fixture.html( valueObj( "" ) ); - fixture.html( valueObj( "foo
    " ) ); + fixture.html( valueObj( "foo
    " ) ); jQuery.scriptorder = 0; fixture.html( valueObj( [ @@ -1888,7 +1888,7 @@ QUnit.test( "html(Function) with incoming value -- jQuery.contents()", function( } ).html().replace( />/g, ">" ), " " + insert, "Verify escaped insertion." ); } ); -QUnit.test( "clone()/html() don't expose jQuery/Sizzle expandos (#12858)", function( assert ) { +QUnit.test( "clone()/html() don't expose jQuery/Sizzle expandos (trac-12858)", function( assert ) { assert.expect( 2 ); @@ -1975,7 +1975,7 @@ QUnit.test( "remove() event cleaning ", function( assert ) { cleanUp.remove(); } ); -QUnit.test( "remove() in document order #13779", function( assert ) { +QUnit.test( "remove() in document order trac-13779", function( assert ) { assert.expect( 1 ); var last, @@ -2336,7 +2336,7 @@ QUnit[ }, 2000 ); } ); -QUnit.test( "jQuery.clone - no exceptions for object elements #9587", function( assert ) { +QUnit.test( "jQuery.clone - no exceptions for object elements trac-9587", function( assert ) { assert.expect( 1 ); @@ -2348,7 +2348,7 @@ QUnit.test( "jQuery.clone - no exceptions for object elements #9587", function( } } ); -QUnit.test( "Cloned, detached HTML5 elems (#10667,10670)", function( assert ) { +QUnit.test( "Cloned, detached HTML5 elems (trac-10667, trac-10670)", function( assert ) { assert.expect( 7 ); @@ -2428,7 +2428,7 @@ QUnit.test( "Guard against exceptions when clearing safeChildNodes", function( a assert.ok( div && div.jquery, "Created nodes safely, guarded against exceptions on safeChildNodes[ -1 ]" ); } ); -QUnit.test( "Ensure oldIE creates a new set on appendTo (#8894)", function( assert ) { +QUnit.test( "Ensure oldIE creates a new set on appendTo (trac-8894)", function( assert ) { assert.expect( 5 ); @@ -2439,7 +2439,7 @@ QUnit.test( "Ensure oldIE creates a new set on appendTo (#8894)", function( asse assert.strictEqual( jQuery( "

    " ).appendTo( "
    " ).end().length, jQuery( "

    test

    " ).appendTo( "
    " ).end().length, "Elements created with createElement and with createDocumentFragment should be treated alike" ); } ); -QUnit.test( "html() - script exceptions bubble (#11743)", function( assert ) { +QUnit.test( "html() - script exceptions bubble (trac-11743)", function( assert ) { assert.expect( 2 ); var done = assert.async(), onerror = window.onerror; @@ -2481,7 +2481,7 @@ QUnit.test( "checked state is cloned with clone()", function( assert ) { assert.equal( jQuery( elem ).clone().attr( "id", "clone" )[ 0 ].checked, true, "Checked true state correctly cloned" ); } ); -QUnit.test( "manipulate mixed jQuery and text (#12384, #12346)", function( assert ) { +QUnit.test( "manipulate mixed jQuery and text (trac-12384, trac-12346)", function( assert ) { assert.expect( 2 ); @@ -2497,7 +2497,7 @@ QUnit.test( "manipulate mixed jQuery and text (#12384, #12346)", function( asser assert.equal( div.find( "*" ).length, 3, "added 2 paragraphs after inner div" ); } ); -QUnit.test( "script evaluation (#11795)", function( assert ) { +QUnit.test( "script evaluation (trac-11795)", function( assert ) { assert.expect( 13 ); @@ -2551,7 +2551,7 @@ QUnit.test( "script evaluation (#11795)", function( assert ) { } } ); -QUnit[ jQuery.ajax ? "test" : "skip" ]( "jQuery._evalUrl (#12838)", function( assert ) { +QUnit[ jQuery.ajax ? "test" : "skip" ]( "jQuery._evalUrl (trac-12838)", function( assert ) { assert.expect( 5 ); @@ -2620,7 +2620,7 @@ QUnit.test( "jQuery.htmlPrefilter (gh-1747)", function( assert ) { }, 100 ); } ); -QUnit.test( "insertAfter, insertBefore, etc do not work when destination is original element. Element is removed (#4087)", function( assert ) { +QUnit.test( "insertAfter, insertBefore, etc do not work when destination is original element. Element is removed (trac-4087)", function( assert ) { assert.expect( 10 ); @@ -2654,7 +2654,7 @@ QUnit.test( "insertAfter, insertBefore, etc do not work when destination is orig } ); } ); -QUnit.test( "Index for function argument should be received (#13094)", function( assert ) { +QUnit.test( "Index for function argument should be received (trac-13094)", function( assert ) { assert.expect( 2 ); var i = 0; @@ -2676,7 +2676,7 @@ QUnit.test( "Make sure jQuery.fn.remove can work on elements in documentFragment assert.equal( fragment.childNodes.length, 0, "div element was removed from documentFragment" ); } ); -QUnit.test( "Make sure specific elements with content created correctly (#13232)", function( assert ) { +QUnit.test( "Make sure specific elements with content created correctly (trac-13232)", function( assert ) { assert.expect( 20 ); var results = [], @@ -2711,7 +2711,7 @@ QUnit.test( "Make sure specific elements with content created correctly (#13232) } ); } ); -QUnit.test( "Validate creation of multiple quantities of certain elements (#13818)", function( assert ) { +QUnit.test( "Validate creation of multiple quantities of certain elements (trac-13818)", function( assert ) { assert.expect( 22 ); var tags = [ "thead", "tbody", "tfoot", "colgroup", "col", "caption", "tr", "th", "td", "optgroup", "option" ]; diff --git a/test/unit/offset.js b/test/unit/offset.js index 82360c874..d3af15e1e 100644 --- a/test/unit/offset.js +++ b/test/unit/offset.js @@ -54,8 +54,8 @@ QUnit.module( "offset", { beforeEach: function( assert ) { QUnit.test( "empty set", function( assert ) { assert.expect( 2 ); - assert.strictEqual( jQuery().offset(), undefined, "offset() returns undefined for empty set (#11962)" ); - assert.strictEqual( jQuery().position(), undefined, "position() returns undefined for empty set (#11962)" ); + assert.strictEqual( jQuery().offset(), undefined, "offset() returns undefined for empty set (trac-11962)" ); + assert.strictEqual( jQuery().position(), undefined, "position() returns undefined for empty set (trac-11962)" ); } ); QUnit.test( "disconnected element", function( assert ) { @@ -187,7 +187,7 @@ testIframe( "absolute", "offset/absolute.html", function( assert, $ ) { assert.equal( $( this.id ).position().left, this.left, "jQuery('" + this.id + "').position().left" ); } ); - // test #5781 + // test trac-5781 offset = $( "#positionTest" ).offset( { "top": 10, "left": 10 } ).offset(); assert.equal( offset.top, 10, "Setting offset on element with position absolute but 'auto' values." ); assert.equal( offset.left, 10, "Setting offset on element with position absolute but 'auto' values." ); @@ -468,8 +468,8 @@ testIframe( "scroll", "offset/scroll.html", function( assert, $, win ) { assert.equal( $( "#scroll-1-1" ).scrollLeft(), 0, "jQuery('#scroll-1-1').scrollLeft()" ); // scroll method chaining - assert.equal( $( "#scroll-1" ).scrollTop( undefined ).scrollTop(), 5, ".scrollTop(undefined) is chainable (#5571)" ); - assert.equal( $( "#scroll-1" ).scrollLeft( undefined ).scrollLeft(), 5, ".scrollLeft(undefined) is chainable (#5571)" ); + assert.equal( $( "#scroll-1" ).scrollTop( undefined ).scrollTop(), 5, ".scrollTop(undefined) is chainable (trac-5571)" ); + assert.equal( $( "#scroll-1" ).scrollLeft( undefined ).scrollLeft(), 5, ".scrollLeft(undefined) is chainable (trac-5571)" ); win.name = "test"; @@ -519,7 +519,7 @@ QUnit.test( "chaining", function( assert ) { var coords = { "top": 1, "left": 1 }; assert.equal( jQuery( "#absolute-1" ).offset( coords ).jquery, jQuery.fn.jquery, "offset(coords) returns jQuery object" ); assert.equal( jQuery( "#non-existent" ).offset( coords ).jquery, jQuery.fn.jquery, "offset(coords) with empty jQuery set returns jQuery object" ); - assert.equal( jQuery( "#absolute-1" ).offset( undefined ).jquery, jQuery.fn.jquery, "offset(undefined) returns jQuery object (#5571)" ); + assert.equal( jQuery( "#absolute-1" ).offset( undefined ).jquery, jQuery.fn.jquery, "offset(undefined) returns jQuery object (trac-5571)" ); } ); // Test complex content under a variety of / positioning styles @@ -729,11 +729,11 @@ QUnit.test( "offsetParent", function( assert ) { area.remove(); div = jQuery( "
    " ).css( { "position": "absolute" } ).appendTo( "body" ); - assert.equal( div.offsetParent()[ 0 ], document.documentElement, "Absolutely positioned div returns html as offset parent, see #12139" ); + assert.equal( div.offsetParent()[ 0 ], document.documentElement, "Absolutely positioned div returns html as offset parent, see trac-12139" ); div.remove(); } ); -QUnit.test( "fractions (see #7730 and #7885)", function( assert ) { +QUnit.test( "fractions (see trac-7730 and trac-7885)", function( assert ) { assert.expect( 2 ); jQuery( "body" ).append( "
    " ); diff --git a/test/unit/queue.js b/test/unit/queue.js index 7735544bc..7e124210b 100644 --- a/test/unit/queue.js +++ b/test/unit/queue.js @@ -41,7 +41,7 @@ QUnit.test( "queue() with other types", function( assert ) { assert.equal( $div.queue( "foo" ).length, 4, "Testing queue length" ); - assert.equal( $div.queue( "foo", undefined ).queue( "foo" ).length, 4, ".queue('name',undefined) does nothing but is chainable (#5571)" ); + assert.equal( $div.queue( "foo", undefined ).queue( "foo" ).length, 4, ".queue('name',undefined) does nothing but is chainable (trac-5571)" ); $div.dequeue( "foo" ); diff --git a/test/unit/selector.js b/test/unit/selector.js index 41d9d0f70..0c81e4862 100644 --- a/test/unit/selector.js +++ b/test/unit/selector.js @@ -55,10 +55,10 @@ QUnit.test( "element", function( assert ) { assert.deepEqual( jQuery( "p", jQuery( "#qunit-fixture" ) ).get(), q( "firstp", "ap", "sndp", "en", "sap", "first" ), "Finding elements with a jQuery object context." ); assert.deepEqual( jQuery( "#qunit-fixture" ).find( "p" ).get(), q( "firstp", "ap", "sndp", "en", "sap", "first" ), "Finding elements with a context via .find()." ); - assert.ok( jQuery( "#length" ).length, " cannot be found under IE, see #945" ); - assert.ok( jQuery( "#lengthtest input" ).length, " cannot be found under IE, see #945" ); + assert.ok( jQuery( "#length" ).length, " cannot be found under IE, see trac-945" ); + assert.ok( jQuery( "#lengthtest input" ).length, " cannot be found under IE, see trac-945" ); - // #7533 + // trac-7533 assert.equal( jQuery( "

    foo

    " ).find( "p" ).length, 1, "Find where context root is a node and has an ID with CSS3 meta characters" ); assert.equal( jQuery( "" ).length, 0, "Empty selector returns an empty array" ); @@ -142,7 +142,8 @@ QUnit.test( "element", function( assert ) { "No stack or performance problems with large amounts of descendants" ); html.remove(); - // Real use case would be using .watch in browsers with window.watch (see Issue #157) + // Real use case would be using .watch in browsers with window.watch + // (see https://github.com/jquery/sizzle/pull/157) q( "qunit-fixture" )[ 0 ].appendChild( document.createElement( "toString" ) ).id = "toString"; assert.t( "Element name matches Object.prototype property", "toString#toString", [ "toString" ] ); } ); @@ -169,7 +170,7 @@ QUnit.test( "XML Document Selectors", function( assert ) { xml = jQuery.parseXML( "" ); assert.equal( jQuery( "elem:not(:has(*))", xml ).length, 1, - "Non-qSA path correctly handles numeric ids (jQuery #14142)" ); + "Non-qSA path correctly handles numeric ids (jQuery trac-14142)" ); } else { assert.ok( "skip", "namespaced elements not matching correctly in selector-native" ); assert.ok( "skip", ":not(complex selector) not supported in selector-native" ); @@ -214,7 +215,7 @@ QUnit.test( "broken selectors throw", function( assert ) { broken( "Last-last-child", ":last-last-child" ); broken( "Only-last-child", ":only-last-child" ); - // Make sure attribute value quoting works correctly. See: #6093 + // Make sure attribute value quoting works correctly. See: trac-6093 jQuery( "" + "" ) .appendTo( "#qunit-fixture" ); @@ -267,7 +268,7 @@ QUnit.test( "id", function( assert ) { fiddle.remove(); - assert.t( "ID Selector, child ID present", "#form > #radio1", [ "radio1" ] ); // bug #267 + assert.t( "ID Selector, child ID present", "#form > #radio1", [ "radio1" ] ); // bug trac-267 assert.t( "ID Selector, not an ancestor ID", "#form #first", [] ); assert.t( "ID Selector, not a child ID", "#form > #option1a", [] ); @@ -288,7 +289,7 @@ QUnit.test( "id", function( assert ) { assert.t( "ID Selector on Form with an input that has a name of 'id'", "#lengthtest", [ "lengthtest" ] ); - assert.t( "ID selector with non-existent ancestor", "#asdfasdf #foobar", [] ); // bug #986 + assert.t( "ID selector with non-existent ancestor", "#asdfasdf #foobar", [] ); // bug trac-986 assert.deepEqual( jQuery( "div#form", document.body ).get(), [], "ID selector within the context of another element" ); @@ -485,7 +486,7 @@ QUnit.test( "child and adjacent", function( assert ) { } if ( QUnit.jQuerySelectorsPos ) { - assert.equal( jQuery( "#listWithTabIndex li:eq(2) ~ li" ).length, 1, "Find by general sibling combinator (#8310)" ); + assert.equal( jQuery( "#listWithTabIndex li:eq(2) ~ li" ).length, 1, "Find by general sibling combinator (trac-8310)" ); nothiddendiv = document.getElementById( "nothiddendiv" ); assert.deepEqual( jQuery( "> :first", nothiddendiv ).get(), q( "nothiddendivchild" ), @@ -505,9 +506,9 @@ QUnit.test( "child and adjacent", function( assert ) { assert.t( "Multiple combinators selects all levels", "#siblingTest > em *", [ "siblingchild", "siblinggrandchild", "siblinggreatgrandchild" ] ); assert.t( "Multiple sibling combinators doesn't miss general siblings", "#siblingTest > em:first-child + em ~ span", [ "siblingspan" ] ); - assert.equal( jQuery( "#listWithTabIndex" ).length, 1, "Parent div for next test is found via ID (#8310)" ); - assert.equal( jQuery( "#__sizzle__" ).length, 0, "Make sure the temporary id assigned by sizzle is cleared out (#8310)" ); - assert.equal( jQuery( "#listWithTabIndex" ).length, 1, "Parent div for previous test is still found via ID (#8310)" ); + assert.equal( jQuery( "#listWithTabIndex" ).length, 1, "Parent div for next test is found via ID (trac-8310)" ); + assert.equal( jQuery( "#__sizzle__" ).length, 0, "Make sure the temporary id assigned by sizzle is cleared out (trac-8310)" ); + assert.equal( jQuery( "#listWithTabIndex" ).length, 1, "Parent div for previous test is still found via ID (trac-8310)" ); assert.t( "Verify deep class selector", "div.blah > p > a", [] ); assert.t( "No element deep selector", "div.foo > span > a", [] ); @@ -555,7 +556,7 @@ QUnit.test( "attributes - equals", function( assert ) { assert.t( "input[type=search]", "#form input[type=search]", [ "search" ] ); withScript = supportjQuery( "
    " ); - assert.ok( withScript.find( "#moretests script[src]" ).has( "script" ), "script[src] (jQuery #13777)" ); + assert.ok( withScript.find( "#moretests script[src]" ).has( "script" ), "script[src] (jQuery trac-13777)" ); assert.t( "Boolean attribute equals name", "#select2 option[selected='selected']", [ "option2d" ] ); assert.t( "for Attribute in form", "#form [for=action]", [ "label-for" ] ); @@ -741,7 +742,7 @@ QUnit.test( "attributes - others", function( assert ) { assert.t( "Object.prototype property \"constructor\"", "[constructor='foo']", [ "foo" ] ); assert.t( "Gecko Object.prototype property \"watch\"", "[watch='bar']", [ "foo" ] ); - // #11115 + // trac-11115 assert.ok( jQuery( "" ).prop( "checked", false ).is( "[checked]" ), "[checked] selects by attribute (positive)" ); @@ -1283,7 +1284,7 @@ QUnit[ QUnit.jQuerySelectorsPos ? "test" : "skip" ]( "pseudo - position", functi q( "ap" ), "Seeded pos with trailing relative" ); - // jQuery #12526 + // jQuery trac-12526 var context = jQuery( "#qunit-fixture" ).append( "
    " )[ 0 ]; assert.deepEqual( jQuery( ":last", context ).get(), q( "jquery12526" ), "Post-manipulation positional" ); @@ -1654,7 +1655,7 @@ QUnit.test( "context", function( assert ) { assert.deepEqual( jQuery( "em + :not(:has(*)):not(:empty), foo", context.firstChild ).get(), expected.slice( 0, 1 ), - "Non-qSA path correctly sets detached context for sibling selectors (jQuery #14351)" + "Non-qSA path correctly sets detached context for sibling selectors (jQuery trac-14351)" ); } else { assert.ok( "skip", ":has not supported in selector-native" ); @@ -1705,7 +1706,7 @@ QUnit.test( "caching does not introduce bugs", function( assert ) { ); if ( QUnit.jQuerySelectorsPos ) { - assert.t( "Deep ancestry caching in post-positional element matcher (jQuery #14657)", + assert.t( "Deep ancestry caching in post-positional element matcher (jQuery trac-14657)", "#qunit-fixture a:lt(3):parent", [ "simon1", "google", "groups" ] ); } else { @@ -1836,9 +1837,9 @@ testIframe( } ); t( "Enumerated attribute", "[spellcheck]", [ "span1" ] ); - t( "tabindex selector does not retrieve all elements in IE6/7 (#8473)", + t( "tabindex selector does not retrieve all elements in IE6/7 (trac-8473)", "form, [tabindex]", [ "form1", "text1" ] ); - t( "Improperly named form elements do not interfere with form selections (#9570)", "form[name='formName']", [ "form1" ] ); + t( "Improperly named form elements do not interfere with form selections (trac-9570)", "form[name='formName']", [ "form1" ] ); } ); @@ -1987,7 +1988,7 @@ testIframe( } ); -QUnit.test( "Iframe dispatch should not affect jQuery (#13936)", function( assert ) { +QUnit.test( "Iframe dispatch should not affect jQuery (trac-13936)", function( assert ) { assert.expect( 1 ); var loaded = false, thrown = false, diff --git a/test/unit/serialize.js b/test/unit/serialize.js index c4e6a6c5e..18f7b299d 100644 --- a/test/unit/serialize.js +++ b/test/unit/serialize.js @@ -34,7 +34,7 @@ QUnit.test( "jQuery.param()", function( assert ) { assert.equal( decodeURIComponent( jQuery.param( { "a": [ 1, 2, 3 ], "b[]": [ 4, 5, 6 ], "c[d]": [ 7, 8, 9 ], "e": { "f": [ 10 ], "g": [ 11, 12 ], "h": 13 } } ) ), "a[]=1&a[]=2&a[]=3&b[]=4&b[]=5&b[]=6&c[d][]=7&c[d][]=8&c[d][]=9&e[f][]=10&e[g][]=11&e[g][]=12&e[h]=13", "Make sure params are not double-encoded." ); - // #7945 + // trac-7945 assert.equal( jQuery.param( { "jquery": "1.4.2" } ), "jquery=1.4.2", "Check that object with a jQuery property get serialized correctly" ); params = { "foo":"bar", "baz":42, "quux":"All your base are belong to us" }; diff --git a/test/unit/support.js b/test/unit/support.js index f0755cbf6..04c2ddcb1 100644 --- a/test/unit/support.js +++ b/test/unit/support.js @@ -19,7 +19,7 @@ function getComputedSupport( support ) { if ( jQuery.css ) { testIframe( - "body background is not lost if set prior to loading jQuery (#9239)", + "body background is not lost if set prior to loading jQuery (trac-9239)", "support/bodyBackground.html", function( assert, jQuery, window, document, color, support ) { assert.expect( 2 ); diff --git a/test/unit/traversing.js b/test/unit/traversing.js index 53b9f11a6..eea81240f 100644 --- a/test/unit/traversing.js +++ b/test/unit/traversing.js @@ -47,7 +47,7 @@ QUnit.test( "find(node|jQuery object)", function( assert ) { assert.equal( $two.find( $first ).length, 0, "first is in the collection and not within two" ); assert.equal( $two.find( $first ).length, 0, "first is in the collection and not within two(node)" ); - assert.equal( $two.find( $foo[ 0 ] ).addBack().length, 2, "find preserves the pushStack, see #12009" ); + assert.equal( $two.find( $foo[ 0 ] ).addBack().length, 2, "find preserves the pushStack, see trac-12009" ); } ); QUnit.test( "is(falsy|invalid)", function( assert ) { @@ -135,7 +135,7 @@ QUnit.test( "is(String)", function( assert ) { assert.ok( !jQuery( disconnected ).is( "* > *" ), "child combinator fails in fragment" ); } ); -QUnit.test( "is() against non-elements (#10178)", function( assert ) { +QUnit.test( "is() against non-elements (trac-10178)", function( assert ) { assert.expect( 14 ); var label, i, test, @@ -241,10 +241,10 @@ QUnit[ QUnit.jQuerySelectorsPos ? "test" : "skip" ]( "is() with positional selec isit( "#posp em", "#posp a em:last", true ); isit( "#posp em", "#posp a em:eq(2)", false ); - assert.ok( jQuery( "#option1b" ).is( "#select1 option:not(:first)" ), "POS inside of :not() (#10970)" ); + assert.ok( jQuery( "#option1b" ).is( "#select1 option:not(:first)" ), "POS inside of :not() (trac-10970)" ); - assert.ok( jQuery( posp[ 0 ] ).is( "p:last" ), "context constructed from a single node (#13797)" ); - assert.ok( !jQuery( posp[ 0 ] ).find( "#firsta" ).is( "a:first" ), "context derived from a single node (#13797)" ); + assert.ok( jQuery( posp[ 0 ] ).is( "p:last" ), "context constructed from a single node (trac-13797)" ); + assert.ok( !jQuery( posp[ 0 ] ).find( "#firsta" ).is( "a:first" ), "context derived from a single node (trac-13797)" ); } ); QUnit.test( "index()", function( assert ) { @@ -405,14 +405,14 @@ QUnit.test( "closest()", function( assert ) { "Non-string match target" ); - // Bug #7369 + // Bug trac-7369 assert.equal( jQuery( "
    " ).closest( "[foo]" ).length, 1, "Disconnected nodes with attribute selector" ); assert.equal( jQuery( "
    text
    " ).closest( "[lang]" ).length, 0, "Disconnected nodes with text and non-existent attribute selector" ); assert.ok( !jQuery( document ).closest( "#foo" ).length, "Calling closest on a document fails silently" ); jq = jQuery( "
    text
    " ); - assert.deepEqual( jq.contents().closest( "*" ).get(), jq.get(), "Text node input (#13332)" ); + assert.deepEqual( jq.contents().closest( "*" ).get(), jq.get(), "Text node input (trac-13332)" ); } ); QUnit[ QUnit.jQuerySelectorsPos ? "test" : "skip" ]( "closest() with positional selectors", function( assert ) { @@ -624,8 +624,8 @@ QUnit.test( "siblings([String])", function( assert ) { var set = q( "sndp", "en", "sap" ); assert.deepEqual( jQuery( "#en, #sndp" ).siblings().get(), set, "Check for unique results from siblings" ); - assert.deepEqual( jQuery( "#option5a" ).siblings( "option[data-attr]" ).get(), q( "option5c" ), "Has attribute selector in siblings (#9261)" ); - assert.equal( jQuery( "" ).siblings().length, 0, "Detached elements have no siblings (#11370)" ); + assert.deepEqual( jQuery( "#option5a" ).siblings( "option[data-attr]" ).get(), q( "option5c" ), "Has attribute selector in siblings (trac-9261)" ); + assert.equal( jQuery( "" ).siblings().length, 0, "Detached elements have no siblings (trac-11370)" ); } ); QUnit[ QUnit.jQuerySelectors ? "test" : "skip" ]( "siblings([String])", function( assert ) { @@ -657,7 +657,7 @@ QUnit.test( "parent([String])", function( assert ) { assert.deepEqual( jQuery( "#en, #sndp" ).parent().get(), q( "foo" ), "Check for unique results from parent" ); $el = jQuery( "
    text
    " ); - assert.deepEqual( $el.contents().parent().get(), $el.get(), "Check for parent of text node (#13265)" ); + assert.deepEqual( $el.contents().parent().get(), $el.get(), "Check for parent of text node (trac-13265)" ); } ); QUnit.test( "parents([String])", function( assert ) { @@ -1067,7 +1067,7 @@ QUnit.test( "add(String, Context)", function( assert ) { assert.deepEqual( jQuery( document.getElementById( "firstp" ) ).add( "#ap", document.getElementsByTagName( "body" )[ 0 ] ).get(), q( "firstp", "ap" ), "Add gEBId to selector, in context" ); } ); -QUnit.test( "eq('-1') #10616", function( assert ) { +QUnit.test( "eq('-1') trac-10616", function( assert ) { assert.expect( 3 ); var $divs = jQuery( "div" ); @@ -1076,7 +1076,7 @@ QUnit.test( "eq('-1') #10616", function( assert ) { assert.deepEqual( $divs.eq( "-1" ), $divs.eq( -1 ), "String and number -1 match" ); } ); -QUnit.test( "index(no arg) #10977", function( assert ) { +QUnit.test( "index(no arg) trac-10977", function( assert ) { assert.expect( 2 ); var $list, fragment, div; @@ -1091,7 +1091,7 @@ QUnit.test( "index(no arg) #10977", function( assert ) { assert.equal( jQuery( div ).index(), 0, "If jQuery#index called on element whose parent is fragment, it still should work correctly" ); } ); -QUnit.test( "traversing non-elements with attribute filters (#12523)", function( assert ) { +QUnit.test( "traversing non-elements with attribute filters (trac-12523)", function( assert ) { assert.expect( 5 ); var nonnodes = jQuery( "#nonnodes" ).contents(); diff --git a/test/unit/wrap.js b/test/unit/wrap.js index 3c0990eac..1f920129e 100644 --- a/test/unit/wrap.js +++ b/test/unit/wrap.js @@ -47,11 +47,11 @@ function testWrap( val, assert ) { var checkbox = this; assert.ok( - checkbox.checked, "Checkbox's state is erased after wrap() action, see #769" + checkbox.checked, "Checkbox's state is erased after wrap() action, see trac-769" ); jQuery( checkbox ).wrap( val( "" ) ); assert.ok( - checkbox.checked, "Checkbox's state is erased after wrap() action, see #769" + checkbox.checked, "Checkbox's state is erased after wrap() action, see trac-769" ); } ).prop( "checked", false )[ 0 ].click(); @@ -131,7 +131,7 @@ QUnit.test( "wrap(Function)", function( assert ) { testWrap( manipulationFunctionReturningObj, assert ); } ); -QUnit.test( "wrap(Function) with index (#10177)", function( assert ) { +QUnit.test( "wrap(Function) with index (trac-10177)", function( assert ) { var expectedIndex = 0, targets = jQuery( "#qunit-fixture p" ); @@ -147,7 +147,7 @@ QUnit.test( "wrap(Function) with index (#10177)", function( assert ) { } ); } ); -QUnit.test( "wrap(String) consecutive elements (#10177)", function( assert ) { +QUnit.test( "wrap(String) consecutive elements (trac-10177)", function( assert ) { var targets = jQuery( "#qunit-fixture p" ); assert.expect( targets.length * 2 ); @@ -484,7 +484,7 @@ QUnit.test( "unwrap( selector )", function( assert ) { jQuery( "body > span.unwrap" ).remove(); } ); -QUnit.test( "jQuery() & wrap[Inner/All]() handle unknown elems (#10667)", function( assert ) { +QUnit.test( "jQuery() & wrap[Inner/All]() handle unknown elems (trac-10667)", function( assert ) { assert.expect( 2 ); @@ -503,7 +503,7 @@ QUnit.test( "jQuery() & wrap[Inner/All]() handle unknown elems (#10667)", f ); } ); -QUnit.test( "wrapping scripts (#10470)", function( assert ) { +QUnit.test( "wrapping scripts (trac-10470)", function( assert ) { assert.expect( 2 ); -- 2.39.5