diff --git a/Gruntfile.js b/Gruntfile.js index 72236561f59..257f4cad5bc 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -225,9 +225,11 @@ grunt.initConfig({ destPrefix: "external" }, files: { - "qunit/qunit.js": "qunit/qunit/qunit.js", - "qunit/qunit.css": "qunit/qunit/qunit.css", "qunit/MIT-LICENSE.txt": "qunit/MIT-LICENSE.txt", + "qunit/qunit.css": "qunit/qunit/qunit.css", + "qunit/qunit.js": "qunit/qunit/qunit.js", + + "requirejs/require.js": "requirejs/require.js", "jquery-mousewheel/jquery.mousewheel.js": "jquery-mousewheel/jquery.mousewheel.js", "jquery-mousewheel/LICENSE.txt": "jquery-mousewheel/LICENSE.txt", diff --git a/bower.json b/bower.json index 6067e9512f5..99674948d70 100644 --- a/bower.json +++ b/bower.json @@ -15,6 +15,7 @@ "jquery-simulate": "1.0.0", "jshint": "2.4.4", "qunit": "1.14.0", + "requirejs": "2.1.14", "jquery-1.6.0": "jquery#1.6.0", "jquery-1.6.1": "jquery#1.6.1", diff --git a/external/requirejs/require.js b/external/requirejs/require.js new file mode 100644 index 00000000000..7f31fa20417 --- /dev/null +++ b/external/requirejs/require.js @@ -0,0 +1,2076 @@ +/** vim: et:ts=4:sw=4:sts=4 + * @license RequireJS 2.1.14 Copyright (c) 2010-2014, The Dojo Foundation All Rights Reserved. + * Available via the MIT or new BSD license. + * see: http://github.com/jrburke/requirejs for details + */ +//Not using strict: uneven strict support in browsers, #392, and causes +//problems with requirejs.exec()/transpiler plugins that may not be strict. +/*jslint regexp: true, nomen: true, sloppy: true */ +/*global window, navigator, document, importScripts, setTimeout, opera */ + +var requirejs, require, define; +(function (global) { + var req, s, head, baseElement, dataMain, src, + interactiveScript, currentlyAddingScript, mainScript, subPath, + version = '2.1.14', + commentRegExp = /(\/\*([\s\S]*?)\*\/|([^:]|^)\/\/(.*)$)/mg, + cjsRequireRegExp = /[^.]\s*require\s*\(\s*["']([^'"\s]+)["']\s*\)/g, + jsSuffixRegExp = /\.js$/, + currDirRegExp = /^\.\//, + op = Object.prototype, + ostring = op.toString, + hasOwn = op.hasOwnProperty, + ap = Array.prototype, + apsp = ap.splice, + isBrowser = !!(typeof window !== 'undefined' && typeof navigator !== 'undefined' && window.document), + isWebWorker = !isBrowser && typeof importScripts !== 'undefined', + //PS3 indicates loaded and complete, but need to wait for complete + //specifically. Sequence is 'loading', 'loaded', execution, + // then 'complete'. The UA check is unfortunate, but not sure how + //to feature test w/o causing perf issues. + readyRegExp = isBrowser && navigator.platform === 'PLAYSTATION 3' ? + /^complete$/ : /^(complete|loaded)$/, + defContextName = '_', + //Oh the tragedy, detecting opera. See the usage of isOpera for reason. + isOpera = typeof opera !== 'undefined' && opera.toString() === '[object Opera]', + contexts = {}, + cfg = {}, + globalDefQueue = [], + useInteractive = false; + + function isFunction(it) { + return ostring.call(it) === '[object Function]'; + } + + function isArray(it) { + return ostring.call(it) === '[object Array]'; + } + + /** + * Helper function for iterating over an array. If the func returns + * a true value, it will break out of the loop. + */ + function each(ary, func) { + if (ary) { + var i; + for (i = 0; i < ary.length; i += 1) { + if (ary[i] && func(ary[i], i, ary)) { + break; + } + } + } + } + + /** + * Helper function for iterating over an array backwards. If the func + * returns a true value, it will break out of the loop. + */ + function eachReverse(ary, func) { + if (ary) { + var i; + for (i = ary.length - 1; i > -1; i -= 1) { + if (ary[i] && func(ary[i], i, ary)) { + break; + } + } + } + } + + function hasProp(obj, prop) { + return hasOwn.call(obj, prop); + } + + function getOwn(obj, prop) { + return hasProp(obj, prop) && obj[prop]; + } + + /** + * Cycles over properties in an object and calls a function for each + * property value. If the function returns a truthy value, then the + * iteration is stopped. + */ + function eachProp(obj, func) { + var prop; + for (prop in obj) { + if (hasProp(obj, prop)) { + if (func(obj[prop], prop)) { + break; + } + } + } + } + + /** + * Simple function to mix in properties from source into target, + * but only if target does not already have a property of the same name. + */ + function mixin(target, source, force, deepStringMixin) { + if (source) { + eachProp(source, function (value, prop) { + if (force || !hasProp(target, prop)) { + if (deepStringMixin && typeof value === 'object' && value && + !isArray(value) && !isFunction(value) && + !(value instanceof RegExp)) { + + if (!target[prop]) { + target[prop] = {}; + } + mixin(target[prop], value, force, deepStringMixin); + } else { + target[prop] = value; + } + } + }); + } + return target; + } + + //Similar to Function.prototype.bind, but the 'this' object is specified + //first, since it is easier to read/figure out what 'this' will be. + function bind(obj, fn) { + return function () { + return fn.apply(obj, arguments); + }; + } + + function scripts() { + return document.getElementsByTagName('script'); + } + + function defaultOnError(err) { + throw err; + } + + //Allow getting a global that is expressed in + //dot notation, like 'a.b.c'. + function getGlobal(value) { + if (!value) { + return value; + } + var g = global; + each(value.split('.'), function (part) { + g = g[part]; + }); + return g; + } + + /** + * Constructs an error with a pointer to an URL with more information. + * @param {String} id the error ID that maps to an ID on a web page. + * @param {String} message human readable error. + * @param {Error} [err] the original error, if there is one. + * + * @returns {Error} + */ + function makeError(id, msg, err, requireModules) { + var e = new Error(msg + '\nhttp://requirejs.org/docs/errors.html#' + id); + e.requireType = id; + e.requireModules = requireModules; + if (err) { + e.originalError = err; + } + return e; + } + + if (typeof define !== 'undefined') { + //If a define is already in play via another AMD loader, + //do not overwrite. + return; + } + + if (typeof requirejs !== 'undefined') { + if (isFunction(requirejs)) { + //Do not overwrite an existing requirejs instance. + return; + } + cfg = requirejs; + requirejs = undefined; + } + + //Allow for a require config object + if (typeof require !== 'undefined' && !isFunction(require)) { + //assume it is a config object. + cfg = require; + require = undefined; + } + + function newContext(contextName) { + var inCheckLoaded, Module, context, handlers, + checkLoadedTimeoutId, + config = { + //Defaults. Do not set a default for map + //config to speed up normalize(), which + //will run faster if there is no default. + waitSeconds: 7, + baseUrl: './', + paths: {}, + bundles: {}, + pkgs: {}, + shim: {}, + config: {} + }, + registry = {}, + //registry of just enabled modules, to speed + //cycle breaking code when lots of modules + //are registered, but not activated. + enabledRegistry = {}, + undefEvents = {}, + defQueue = [], + defined = {}, + urlFetched = {}, + bundlesMap = {}, + requireCounter = 1, + unnormalizedCounter = 1; + + /** + * Trims the . and .. from an array of path segments. + * It will keep a leading path segment if a .. will become + * the first path segment, to help with module name lookups, + * which act like paths, but can be remapped. But the end result, + * all paths that use this function should look normalized. + * NOTE: this method MODIFIES the input array. + * @param {Array} ary the array of path segments. + */ + function trimDots(ary) { + var i, part; + for (i = 0; i < ary.length; i++) { + part = ary[i]; + if (part === '.') { + ary.splice(i, 1); + i -= 1; + } else if (part === '..') { + // If at the start, or previous value is still .., + // keep them so that when converted to a path it may + // still work when converted to a path, even though + // as an ID it is less than ideal. In larger point + // releases, may be better to just kick out an error. + if (i === 0 || (i == 1 && ary[2] === '..') || ary[i - 1] === '..') { + continue; + } else if (i > 0) { + ary.splice(i - 1, 2); + i -= 2; + } + } + } + } + + /** + * Given a relative module name, like ./something, normalize it to + * a real name that can be mapped to a path. + * @param {String} name the relative name + * @param {String} baseName a real name that the name arg is relative + * to. + * @param {Boolean} applyMap apply the map config to the value. Should + * only be done if this normalization is for a dependency ID. + * @returns {String} normalized name + */ + function normalize(name, baseName, applyMap) { + var pkgMain, mapValue, nameParts, i, j, nameSegment, lastIndex, + foundMap, foundI, foundStarMap, starI, normalizedBaseParts, + baseParts = (baseName && baseName.split('/')), + map = config.map, + starMap = map && map['*']; + + //Adjust any relative paths. + if (name) { + name = name.split('/'); + lastIndex = name.length - 1; + + // If wanting node ID compatibility, strip .js from end + // of IDs. Have to do this here, and not in nameToUrl + // because node allows either .js or non .js to map + // to same file. + if (config.nodeIdCompat && jsSuffixRegExp.test(name[lastIndex])) { + name[lastIndex] = name[lastIndex].replace(jsSuffixRegExp, ''); + } + + // Starts with a '.' so need the baseName + if (name[0].charAt(0) === '.' && baseParts) { + //Convert baseName to array, and lop off the last part, + //so that . matches that 'directory' and not name of the baseName's + //module. For instance, baseName of 'one/two/three', maps to + //'one/two/three.js', but we want the directory, 'one/two' for + //this normalization. + normalizedBaseParts = baseParts.slice(0, baseParts.length - 1); + name = normalizedBaseParts.concat(name); + } + + trimDots(name); + name = name.join('/'); + } + + //Apply map config if available. + if (applyMap && map && (baseParts || starMap)) { + nameParts = name.split('/'); + + outerLoop: for (i = nameParts.length; i > 0; i -= 1) { + nameSegment = nameParts.slice(0, i).join('/'); + + if (baseParts) { + //Find the longest baseName segment match in the config. + //So, do joins on the biggest to smallest lengths of baseParts. + for (j = baseParts.length; j > 0; j -= 1) { + mapValue = getOwn(map, baseParts.slice(0, j).join('/')); + + //baseName segment has config, find if it has one for + //this name. + if (mapValue) { + mapValue = getOwn(mapValue, nameSegment); + if (mapValue) { + //Match, update name to the new value. + foundMap = mapValue; + foundI = i; + break outerLoop; + } + } + } + } + + //Check for a star map match, but just hold on to it, + //if there is a shorter segment match later in a matching + //config, then favor over this star map. + if (!foundStarMap && starMap && getOwn(starMap, nameSegment)) { + foundStarMap = getOwn(starMap, nameSegment); + starI = i; + } + } + + if (!foundMap && foundStarMap) { + foundMap = foundStarMap; + foundI = starI; + } + + if (foundMap) { + nameParts.splice(0, foundI, foundMap); + name = nameParts.join('/'); + } + } + + // If the name points to a package's name, use + // the package main instead. + pkgMain = getOwn(config.pkgs, name); + + return pkgMain ? pkgMain : name; + } + + function removeScript(name) { + if (isBrowser) { + each(scripts(), function (scriptNode) { + if (scriptNode.getAttribute('data-requiremodule') === name && + scriptNode.getAttribute('data-requirecontext') === context.contextName) { + scriptNode.parentNode.removeChild(scriptNode); + return true; + } + }); + } + } + + function hasPathFallback(id) { + var pathConfig = getOwn(config.paths, id); + if (pathConfig && isArray(pathConfig) && pathConfig.length > 1) { + //Pop off the first array value, since it failed, and + //retry + pathConfig.shift(); + context.require.undef(id); + + //Custom require that does not do map translation, since + //ID is "absolute", already mapped/resolved. + context.makeRequire(null, { + skipMap: true + })([id]); + + return true; + } + } + + //Turns a plugin!resource to [plugin, resource] + //with the plugin being undefined if the name + //did not have a plugin prefix. + function splitPrefix(name) { + var prefix, + index = name ? name.indexOf('!') : -1; + if (index > -1) { + prefix = name.substring(0, index); + name = name.substring(index + 1, name.length); + } + return [prefix, name]; + } + + /** + * Creates a module mapping that includes plugin prefix, module + * name, and path. If parentModuleMap is provided it will + * also normalize the name via require.normalize() + * + * @param {String} name the module name + * @param {String} [parentModuleMap] parent module map + * for the module name, used to resolve relative names. + * @param {Boolean} isNormalized: is the ID already normalized. + * This is true if this call is done for a define() module ID. + * @param {Boolean} applyMap: apply the map config to the ID. + * Should only be true if this map is for a dependency. + * + * @returns {Object} + */ + function makeModuleMap(name, parentModuleMap, isNormalized, applyMap) { + var url, pluginModule, suffix, nameParts, + prefix = null, + parentName = parentModuleMap ? parentModuleMap.name : null, + originalName = name, + isDefine = true, + normalizedName = ''; + + //If no name, then it means it is a require call, generate an + //internal name. + if (!name) { + isDefine = false; + name = '_@r' + (requireCounter += 1); + } + + nameParts = splitPrefix(name); + prefix = nameParts[0]; + name = nameParts[1]; + + if (prefix) { + prefix = normalize(prefix, parentName, applyMap); + pluginModule = getOwn(defined, prefix); + } + + //Account for relative paths if there is a base name. + if (name) { + if (prefix) { + if (pluginModule && pluginModule.normalize) { + //Plugin is loaded, use its normalize method. + normalizedName = pluginModule.normalize(name, function (name) { + return normalize(name, parentName, applyMap); + }); + } else { + // If nested plugin references, then do not try to + // normalize, as it will not normalize correctly. This + // places a restriction on resourceIds, and the longer + // term solution is not to normalize until plugins are + // loaded and all normalizations to allow for async + // loading of a loader plugin. But for now, fixes the + // common uses. Details in #1131 + normalizedName = name.indexOf('!') === -1 ? + normalize(name, parentName, applyMap) : + name; + } + } else { + //A regular module. + normalizedName = normalize(name, parentName, applyMap); + + //Normalized name may be a plugin ID due to map config + //application in normalize. The map config values must + //already be normalized, so do not need to redo that part. + nameParts = splitPrefix(normalizedName); + prefix = nameParts[0]; + normalizedName = nameParts[1]; + isNormalized = true; + + url = context.nameToUrl(normalizedName); + } + } + + //If the id is a plugin id that cannot be determined if it needs + //normalization, stamp it with a unique ID so two matching relative + //ids that may conflict can be separate. + suffix = prefix && !pluginModule && !isNormalized ? + '_unnormalized' + (unnormalizedCounter += 1) : + ''; + + return { + prefix: prefix, + name: normalizedName, + parentMap: parentModuleMap, + unnormalized: !!suffix, + url: url, + originalName: originalName, + isDefine: isDefine, + id: (prefix ? + prefix + '!' + normalizedName : + normalizedName) + suffix + }; + } + + function getModule(depMap) { + var id = depMap.id, + mod = getOwn(registry, id); + + if (!mod) { + mod = registry[id] = new context.Module(depMap); + } + + return mod; + } + + function on(depMap, name, fn) { + var id = depMap.id, + mod = getOwn(registry, id); + + if (hasProp(defined, id) && + (!mod || mod.defineEmitComplete)) { + if (name === 'defined') { + fn(defined[id]); + } + } else { + mod = getModule(depMap); + if (mod.error && name === 'error') { + fn(mod.error); + } else { + mod.on(name, fn); + } + } + } + + function onError(err, errback) { + var ids = err.requireModules, + notified = false; + + if (errback) { + errback(err); + } else { + each(ids, function (id) { + var mod = getOwn(registry, id); + if (mod) { + //Set error on module, so it skips timeout checks. + mod.error = err; + if (mod.events.error) { + notified = true; + mod.emit('error', err); + } + } + }); + + if (!notified) { + req.onError(err); + } + } + } + + /** + * Internal method to transfer globalQueue items to this context's + * defQueue. + */ + function takeGlobalQueue() { + //Push all the globalDefQueue items into the context's defQueue + if (globalDefQueue.length) { + //Array splice in the values since the context code has a + //local var ref to defQueue, so cannot just reassign the one + //on context. + apsp.apply(defQueue, + [defQueue.length, 0].concat(globalDefQueue)); + globalDefQueue = []; + } + } + + handlers = { + 'require': function (mod) { + if (mod.require) { + return mod.require; + } else { + return (mod.require = context.makeRequire(mod.map)); + } + }, + 'exports': function (mod) { + mod.usingExports = true; + if (mod.map.isDefine) { + if (mod.exports) { + return (defined[mod.map.id] = mod.exports); + } else { + return (mod.exports = defined[mod.map.id] = {}); + } + } + }, + 'module': function (mod) { + if (mod.module) { + return mod.module; + } else { + return (mod.module = { + id: mod.map.id, + uri: mod.map.url, + config: function () { + return getOwn(config.config, mod.map.id) || {}; + }, + exports: mod.exports || (mod.exports = {}) + }); + } + } + }; + + function cleanRegistry(id) { + //Clean up machinery used for waiting modules. + delete registry[id]; + delete enabledRegistry[id]; + } + + function breakCycle(mod, traced, processed) { + var id = mod.map.id; + + if (mod.error) { + mod.emit('error', mod.error); + } else { + traced[id] = true; + each(mod.depMaps, function (depMap, i) { + var depId = depMap.id, + dep = getOwn(registry, depId); + + //Only force things that have not completed + //being defined, so still in the registry, + //and only if it has not been matched up + //in the module already. + if (dep && !mod.depMatched[i] && !processed[depId]) { + if (getOwn(traced, depId)) { + mod.defineDep(i, defined[depId]); + mod.check(); //pass false? + } else { + breakCycle(dep, traced, processed); + } + } + }); + processed[id] = true; + } + } + + function checkLoaded() { + var err, usingPathFallback, + waitInterval = config.waitSeconds * 1000, + //It is possible to disable the wait interval by using waitSeconds of 0. + expired = waitInterval && (context.startTime + waitInterval) < new Date().getTime(), + noLoads = [], + reqCalls = [], + stillLoading = false, + needCycleCheck = true; + + //Do not bother if this call was a result of a cycle break. + if (inCheckLoaded) { + return; + } + + inCheckLoaded = true; + + //Figure out the state of all the modules. + eachProp(enabledRegistry, function (mod) { + var map = mod.map, + modId = map.id; + + //Skip things that are not enabled or in error state. + if (!mod.enabled) { + return; + } + + if (!map.isDefine) { + reqCalls.push(mod); + } + + if (!mod.error) { + //If the module should be executed, and it has not + //been inited and time is up, remember it. + if (!mod.inited && expired) { + if (hasPathFallback(modId)) { + usingPathFallback = true; + stillLoading = true; + } else { + noLoads.push(modId); + removeScript(modId); + } + } else if (!mod.inited && mod.fetched && map.isDefine) { + stillLoading = true; + if (!map.prefix) { + //No reason to keep looking for unfinished + //loading. If the only stillLoading is a + //plugin resource though, keep going, + //because it may be that a plugin resource + //is waiting on a non-plugin cycle. + return (needCycleCheck = false); + } + } + } + }); + + if (expired && noLoads.length) { + //If wait time expired, throw error of unloaded modules. + err = makeError('timeout', 'Load timeout for modules: ' + noLoads, null, noLoads); + err.contextName = context.contextName; + return onError(err); + } + + //Not expired, check for a cycle. + if (needCycleCheck) { + each(reqCalls, function (mod) { + breakCycle(mod, {}, {}); + }); + } + + //If still waiting on loads, and the waiting load is something + //other than a plugin resource, or there are still outstanding + //scripts, then just try back later. + if ((!expired || usingPathFallback) && stillLoading) { + //Something is still waiting to load. Wait for it, but only + //if a timeout is not already in effect. + if ((isBrowser || isWebWorker) && !checkLoadedTimeoutId) { + checkLoadedTimeoutId = setTimeout(function () { + checkLoadedTimeoutId = 0; + checkLoaded(); + }, 50); + } + } + + inCheckLoaded = false; + } + + Module = function (map) { + this.events = getOwn(undefEvents, map.id) || {}; + this.map = map; + this.shim = getOwn(config.shim, map.id); + this.depExports = []; + this.depMaps = []; + this.depMatched = []; + this.pluginMaps = {}; + this.depCount = 0; + + /* this.exports this.factory + this.depMaps = [], + this.enabled, this.fetched + */ + }; + + Module.prototype = { + init: function (depMaps, factory, errback, options) { + options = options || {}; + + //Do not do more inits if already done. Can happen if there + //are multiple define calls for the same module. That is not + //a normal, common case, but it is also not unexpected. + if (this.inited) { + return; + } + + this.factory = factory; + + if (errback) { + //Register for errors on this module. + this.on('error', errback); + } else if (this.events.error) { + //If no errback already, but there are error listeners + //on this module, set up an errback to pass to the deps. + errback = bind(this, function (err) { + this.emit('error', err); + }); + } + + //Do a copy of the dependency array, so that + //source inputs are not modified. For example + //"shim" deps are passed in here directly, and + //doing a direct modification of the depMaps array + //would affect that config. + this.depMaps = depMaps && depMaps.slice(0); + + this.errback = errback; + + //Indicate this module has be initialized + this.inited = true; + + this.ignore = options.ignore; + + //Could have option to init this module in enabled mode, + //or could have been previously marked as enabled. However, + //the dependencies are not known until init is called. So + //if enabled previously, now trigger dependencies as enabled. + if (options.enabled || this.enabled) { + //Enable this module and dependencies. + //Will call this.check() + this.enable(); + } else { + this.check(); + } + }, + + defineDep: function (i, depExports) { + //Because of cycles, defined callback for a given + //export can be called more than once. + if (!this.depMatched[i]) { + this.depMatched[i] = true; + this.depCount -= 1; + this.depExports[i] = depExports; + } + }, + + fetch: function () { + if (this.fetched) { + return; + } + this.fetched = true; + + context.startTime = (new Date()).getTime(); + + var map = this.map; + + //If the manager is for a plugin managed resource, + //ask the plugin to load it now. + if (this.shim) { + context.makeRequire(this.map, { + enableBuildCallback: true + })(this.shim.deps || [], bind(this, function () { + return map.prefix ? this.callPlugin() : this.load(); + })); + } else { + //Regular dependency. + return map.prefix ? this.callPlugin() : this.load(); + } + }, + + load: function () { + var url = this.map.url; + + //Regular dependency. + if (!urlFetched[url]) { + urlFetched[url] = true; + context.load(this.map.id, url); + } + }, + + /** + * Checks if the module is ready to define itself, and if so, + * define it. + */ + check: function () { + if (!this.enabled || this.enabling) { + return; + } + + var err, cjsModule, + id = this.map.id, + depExports = this.depExports, + exports = this.exports, + factory = this.factory; + + if (!this.inited) { + this.fetch(); + } else if (this.error) { + this.emit('error', this.error); + } else if (!this.defining) { + //The factory could trigger another require call + //that would result in checking this module to + //define itself again. If already in the process + //of doing that, skip this work. + this.defining = true; + + if (this.depCount < 1 && !this.defined) { + if (isFunction(factory)) { + //If there is an error listener, favor passing + //to that instead of throwing an error. However, + //only do it for define()'d modules. require + //errbacks should not be called for failures in + //their callbacks (#699). However if a global + //onError is set, use that. + if ((this.events.error && this.map.isDefine) || + req.onError !== defaultOnError) { + try { + exports = context.execCb(id, factory, depExports, exports); + } catch (e) { + err = e; + } + } else { + exports = context.execCb(id, factory, depExports, exports); + } + + // Favor return value over exports. If node/cjs in play, + // then will not have a return value anyway. Favor + // module.exports assignment over exports object. + if (this.map.isDefine && exports === undefined) { + cjsModule = this.module; + if (cjsModule) { + exports = cjsModule.exports; + } else if (this.usingExports) { + //exports already set the defined value. + exports = this.exports; + } + } + + if (err) { + err.requireMap = this.map; + err.requireModules = this.map.isDefine ? [this.map.id] : null; + err.requireType = this.map.isDefine ? 'define' : 'require'; + return onError((this.error = err)); + } + + } else { + //Just a literal value + exports = factory; + } + + this.exports = exports; + + if (this.map.isDefine && !this.ignore) { + defined[id] = exports; + + if (req.onResourceLoad) { + req.onResourceLoad(context, this.map, this.depMaps); + } + } + + //Clean up + cleanRegistry(id); + + this.defined = true; + } + + //Finished the define stage. Allow calling check again + //to allow define notifications below in the case of a + //cycle. + this.defining = false; + + if (this.defined && !this.defineEmitted) { + this.defineEmitted = true; + this.emit('defined', this.exports); + this.defineEmitComplete = true; + } + + } + }, + + callPlugin: function () { + var map = this.map, + id = map.id, + //Map already normalized the prefix. + pluginMap = makeModuleMap(map.prefix); + + //Mark this as a dependency for this plugin, so it + //can be traced for cycles. + this.depMaps.push(pluginMap); + + on(pluginMap, 'defined', bind(this, function (plugin) { + var load, normalizedMap, normalizedMod, + bundleId = getOwn(bundlesMap, this.map.id), + name = this.map.name, + parentName = this.map.parentMap ? this.map.parentMap.name : null, + localRequire = context.makeRequire(map.parentMap, { + enableBuildCallback: true + }); + + //If current map is not normalized, wait for that + //normalized name to load instead of continuing. + if (this.map.unnormalized) { + //Normalize the ID if the plugin allows it. + if (plugin.normalize) { + name = plugin.normalize(name, function (name) { + return normalize(name, parentName, true); + }) || ''; + } + + //prefix and name should already be normalized, no need + //for applying map config again either. + normalizedMap = makeModuleMap(map.prefix + '!' + name, + this.map.parentMap); + on(normalizedMap, + 'defined', bind(this, function (value) { + this.init([], function () { return value; }, null, { + enabled: true, + ignore: true + }); + })); + + normalizedMod = getOwn(registry, normalizedMap.id); + if (normalizedMod) { + //Mark this as a dependency for this plugin, so it + //can be traced for cycles. + this.depMaps.push(normalizedMap); + + if (this.events.error) { + normalizedMod.on('error', bind(this, function (err) { + this.emit('error', err); + })); + } + normalizedMod.enable(); + } + + return; + } + + //If a paths config, then just load that file instead to + //resolve the plugin, as it is built into that paths layer. + if (bundleId) { + this.map.url = context.nameToUrl(bundleId); + this.load(); + return; + } + + load = bind(this, function (value) { + this.init([], function () { return value; }, null, { + enabled: true + }); + }); + + load.error = bind(this, function (err) { + this.inited = true; + this.error = err; + err.requireModules = [id]; + + //Remove temp unnormalized modules for this module, + //since they will never be resolved otherwise now. + eachProp(registry, function (mod) { + if (mod.map.id.indexOf(id + '_unnormalized') === 0) { + cleanRegistry(mod.map.id); + } + }); + + onError(err); + }); + + //Allow plugins to load other code without having to know the + //context or how to 'complete' the load. + load.fromText = bind(this, function (text, textAlt) { + /*jslint evil: true */ + var moduleName = map.name, + moduleMap = makeModuleMap(moduleName), + hasInteractive = useInteractive; + + //As of 2.1.0, support just passing the text, to reinforce + //fromText only being called once per resource. Still + //support old style of passing moduleName but discard + //that moduleName in favor of the internal ref. + if (textAlt) { + text = textAlt; + } + + //Turn off interactive script matching for IE for any define + //calls in the text, then turn it back on at the end. + if (hasInteractive) { + useInteractive = false; + } + + //Prime the system by creating a module instance for + //it. + getModule(moduleMap); + + //Transfer any config to this other module. + if (hasProp(config.config, id)) { + config.config[moduleName] = config.config[id]; + } + + try { + req.exec(text); + } catch (e) { + return onError(makeError('fromtexteval', + 'fromText eval for ' + id + + ' failed: ' + e, + e, + [id])); + } + + if (hasInteractive) { + useInteractive = true; + } + + //Mark this as a dependency for the plugin + //resource + this.depMaps.push(moduleMap); + + //Support anonymous modules. + context.completeLoad(moduleName); + + //Bind the value of that module to the value for this + //resource ID. + localRequire([moduleName], load); + }); + + //Use parentName here since the plugin's name is not reliable, + //could be some weird string with no path that actually wants to + //reference the parentName's path. + plugin.load(map.name, localRequire, load, config); + })); + + context.enable(pluginMap, this); + this.pluginMaps[pluginMap.id] = pluginMap; + }, + + enable: function () { + enabledRegistry[this.map.id] = this; + this.enabled = true; + + //Set flag mentioning that the module is enabling, + //so that immediate calls to the defined callbacks + //for dependencies do not trigger inadvertent load + //with the depCount still being zero. + this.enabling = true; + + //Enable each dependency + each(this.depMaps, bind(this, function (depMap, i) { + var id, mod, handler; + + if (typeof depMap === 'string') { + //Dependency needs to be converted to a depMap + //and wired up to this module. + depMap = makeModuleMap(depMap, + (this.map.isDefine ? this.map : this.map.parentMap), + false, + !this.skipMap); + this.depMaps[i] = depMap; + + handler = getOwn(handlers, depMap.id); + + if (handler) { + this.depExports[i] = handler(this); + return; + } + + this.depCount += 1; + + on(depMap, 'defined', bind(this, function (depExports) { + this.defineDep(i, depExports); + this.check(); + })); + + if (this.errback) { + on(depMap, 'error', bind(this, this.errback)); + } + } + + id = depMap.id; + mod = registry[id]; + + //Skip special modules like 'require', 'exports', 'module' + //Also, don't call enable if it is already enabled, + //important in circular dependency cases. + if (!hasProp(handlers, id) && mod && !mod.enabled) { + context.enable(depMap, this); + } + })); + + //Enable each plugin that is used in + //a dependency + eachProp(this.pluginMaps, bind(this, function (pluginMap) { + var mod = getOwn(registry, pluginMap.id); + if (mod && !mod.enabled) { + context.enable(pluginMap, this); + } + })); + + this.enabling = false; + + this.check(); + }, + + on: function (name, cb) { + var cbs = this.events[name]; + if (!cbs) { + cbs = this.events[name] = []; + } + cbs.push(cb); + }, + + emit: function (name, evt) { + each(this.events[name], function (cb) { + cb(evt); + }); + if (name === 'error') { + //Now that the error handler was triggered, remove + //the listeners, since this broken Module instance + //can stay around for a while in the registry. + delete this.events[name]; + } + } + }; + + function callGetModule(args) { + //Skip modules already defined. + if (!hasProp(defined, args[0])) { + getModule(makeModuleMap(args[0], null, true)).init(args[1], args[2]); + } + } + + function removeListener(node, func, name, ieName) { + //Favor detachEvent because of IE9 + //issue, see attachEvent/addEventListener comment elsewhere + //in this file. + if (node.detachEvent && !isOpera) { + //Probably IE. If not it will throw an error, which will be + //useful to know. + if (ieName) { + node.detachEvent(ieName, func); + } + } else { + node.removeEventListener(name, func, false); + } + } + + /** + * Given an event from a script node, get the requirejs info from it, + * and then removes the event listeners on the node. + * @param {Event} evt + * @returns {Object} + */ + function getScriptData(evt) { + //Using currentTarget instead of target for Firefox 2.0's sake. Not + //all old browsers will be supported, but this one was easy enough + //to support and still makes sense. + var node = evt.currentTarget || evt.srcElement; + + //Remove the listeners once here. + removeListener(node, context.onScriptLoad, 'load', 'onreadystatechange'); + removeListener(node, context.onScriptError, 'error'); + + return { + node: node, + id: node && node.getAttribute('data-requiremodule') + }; + } + + function intakeDefines() { + var args; + + //Any defined modules in the global queue, intake them now. + takeGlobalQueue(); + + //Make sure any remaining defQueue items get properly processed. + while (defQueue.length) { + args = defQueue.shift(); + if (args[0] === null) { + return onError(makeError('mismatch', 'Mismatched anonymous define() module: ' + args[args.length - 1])); + } else { + //args are id, deps, factory. Should be normalized by the + //define() function. + callGetModule(args); + } + } + } + + context = { + config: config, + contextName: contextName, + registry: registry, + defined: defined, + urlFetched: urlFetched, + defQueue: defQueue, + Module: Module, + makeModuleMap: makeModuleMap, + nextTick: req.nextTick, + onError: onError, + + /** + * Set a configuration for the context. + * @param {Object} cfg config object to integrate. + */ + configure: function (cfg) { + //Make sure the baseUrl ends in a slash. + if (cfg.baseUrl) { + if (cfg.baseUrl.charAt(cfg.baseUrl.length - 1) !== '/') { + cfg.baseUrl += '/'; + } + } + + //Save off the paths since they require special processing, + //they are additive. + var shim = config.shim, + objs = { + paths: true, + bundles: true, + config: true, + map: true + }; + + eachProp(cfg, function (value, prop) { + if (objs[prop]) { + if (!config[prop]) { + config[prop] = {}; + } + mixin(config[prop], value, true, true); + } else { + config[prop] = value; + } + }); + + //Reverse map the bundles + if (cfg.bundles) { + eachProp(cfg.bundles, function (value, prop) { + each(value, function (v) { + if (v !== prop) { + bundlesMap[v] = prop; + } + }); + }); + } + + //Merge shim + if (cfg.shim) { + eachProp(cfg.shim, function (value, id) { + //Normalize the structure + if (isArray(value)) { + value = { + deps: value + }; + } + if ((value.exports || value.init) && !value.exportsFn) { + value.exportsFn = context.makeShimExports(value); + } + shim[id] = value; + }); + config.shim = shim; + } + + //Adjust packages if necessary. + if (cfg.packages) { + each(cfg.packages, function (pkgObj) { + var location, name; + + pkgObj = typeof pkgObj === 'string' ? { name: pkgObj } : pkgObj; + + name = pkgObj.name; + location = pkgObj.location; + if (location) { + config.paths[name] = pkgObj.location; + } + + //Save pointer to main module ID for pkg name. + //Remove leading dot in main, so main paths are normalized, + //and remove any trailing .js, since different package + //envs have different conventions: some use a module name, + //some use a file name. + config.pkgs[name] = pkgObj.name + '/' + (pkgObj.main || 'main') + .replace(currDirRegExp, '') + .replace(jsSuffixRegExp, ''); + }); + } + + //If there are any "waiting to execute" modules in the registry, + //update the maps for them, since their info, like URLs to load, + //may have changed. + eachProp(registry, function (mod, id) { + //If module already has init called, since it is too + //late to modify them, and ignore unnormalized ones + //since they are transient. + if (!mod.inited && !mod.map.unnormalized) { + mod.map = makeModuleMap(id); + } + }); + + //If a deps array or a config callback is specified, then call + //require with those args. This is useful when require is defined as a + //config object before require.js is loaded. + if (cfg.deps || cfg.callback) { + context.require(cfg.deps || [], cfg.callback); + } + }, + + makeShimExports: function (value) { + function fn() { + var ret; + if (value.init) { + ret = value.init.apply(global, arguments); + } + return ret || (value.exports && getGlobal(value.exports)); + } + return fn; + }, + + makeRequire: function (relMap, options) { + options = options || {}; + + function localRequire(deps, callback, errback) { + var id, map, requireMod; + + if (options.enableBuildCallback && callback && isFunction(callback)) { + callback.__requireJsBuild = true; + } + + if (typeof deps === 'string') { + if (isFunction(callback)) { + //Invalid call + return onError(makeError('requireargs', 'Invalid require call'), errback); + } + + //If require|exports|module are requested, get the + //value for them from the special handlers. Caveat: + //this only works while module is being defined. + if (relMap && hasProp(handlers, deps)) { + return handlers[deps](registry[relMap.id]); + } + + //Synchronous access to one module. If require.get is + //available (as in the Node adapter), prefer that. + if (req.get) { + return req.get(context, deps, relMap, localRequire); + } + + //Normalize module name, if it contains . or .. + map = makeModuleMap(deps, relMap, false, true); + id = map.id; + + if (!hasProp(defined, id)) { + return onError(makeError('notloaded', 'Module name "' + + id + + '" has not been loaded yet for context: ' + + contextName + + (relMap ? '' : '. Use require([])'))); + } + return defined[id]; + } + + //Grab defines waiting in the global queue. + intakeDefines(); + + //Mark all the dependencies as needing to be loaded. + context.nextTick(function () { + //Some defines could have been added since the + //require call, collect them. + intakeDefines(); + + requireMod = getModule(makeModuleMap(null, relMap)); + + //Store if map config should be applied to this require + //call for dependencies. + requireMod.skipMap = options.skipMap; + + requireMod.init(deps, callback, errback, { + enabled: true + }); + + checkLoaded(); + }); + + return localRequire; + } + + mixin(localRequire, { + isBrowser: isBrowser, + + /** + * Converts a module name + .extension into an URL path. + * *Requires* the use of a module name. It does not support using + * plain URLs like nameToUrl. + */ + toUrl: function (moduleNamePlusExt) { + var ext, + index = moduleNamePlusExt.lastIndexOf('.'), + segment = moduleNamePlusExt.split('/')[0], + isRelative = segment === '.' || segment === '..'; + + //Have a file extension alias, and it is not the + //dots from a relative path. + if (index !== -1 && (!isRelative || index > 1)) { + ext = moduleNamePlusExt.substring(index, moduleNamePlusExt.length); + moduleNamePlusExt = moduleNamePlusExt.substring(0, index); + } + + return context.nameToUrl(normalize(moduleNamePlusExt, + relMap && relMap.id, true), ext, true); + }, + + defined: function (id) { + return hasProp(defined, makeModuleMap(id, relMap, false, true).id); + }, + + specified: function (id) { + id = makeModuleMap(id, relMap, false, true).id; + return hasProp(defined, id) || hasProp(registry, id); + } + }); + + //Only allow undef on top level require calls + if (!relMap) { + localRequire.undef = function (id) { + //Bind any waiting define() calls to this context, + //fix for #408 + takeGlobalQueue(); + + var map = makeModuleMap(id, relMap, true), + mod = getOwn(registry, id); + + removeScript(id); + + delete defined[id]; + delete urlFetched[map.url]; + delete undefEvents[id]; + + //Clean queued defines too. Go backwards + //in array so that the splices do not + //mess up the iteration. + eachReverse(defQueue, function(args, i) { + if(args[0] === id) { + defQueue.splice(i, 1); + } + }); + + if (mod) { + //Hold on to listeners in case the + //module will be attempted to be reloaded + //using a different config. + if (mod.events.defined) { + undefEvents[id] = mod.events; + } + + cleanRegistry(id); + } + }; + } + + return localRequire; + }, + + /** + * Called to enable a module if it is still in the registry + * awaiting enablement. A second arg, parent, the parent module, + * is passed in for context, when this method is overridden by + * the optimizer. Not shown here to keep code compact. + */ + enable: function (depMap) { + var mod = getOwn(registry, depMap.id); + if (mod) { + getModule(depMap).enable(); + } + }, + + /** + * Internal method used by environment adapters to complete a load event. + * A load event could be a script load or just a load pass from a synchronous + * load call. + * @param {String} moduleName the name of the module to potentially complete. + */ + completeLoad: function (moduleName) { + var found, args, mod, + shim = getOwn(config.shim, moduleName) || {}, + shExports = shim.exports; + + takeGlobalQueue(); + + while (defQueue.length) { + args = defQueue.shift(); + if (args[0] === null) { + args[0] = moduleName; + //If already found an anonymous module and bound it + //to this name, then this is some other anon module + //waiting for its completeLoad to fire. + if (found) { + break; + } + found = true; + } else if (args[0] === moduleName) { + //Found matching define call for this script! + found = true; + } + + callGetModule(args); + } + + //Do this after the cycle of callGetModule in case the result + //of those calls/init calls changes the registry. + mod = getOwn(registry, moduleName); + + if (!found && !hasProp(defined, moduleName) && mod && !mod.inited) { + if (config.enforceDefine && (!shExports || !getGlobal(shExports))) { + if (hasPathFallback(moduleName)) { + return; + } else { + return onError(makeError('nodefine', + 'No define call for ' + moduleName, + null, + [moduleName])); + } + } else { + //A script that does not call define(), so just simulate + //the call for it. + callGetModule([moduleName, (shim.deps || []), shim.exportsFn]); + } + } + + checkLoaded(); + }, + + /** + * Converts a module name to a file path. Supports cases where + * moduleName may actually be just an URL. + * Note that it **does not** call normalize on the moduleName, + * it is assumed to have already been normalized. This is an + * internal API, not a public one. Use toUrl for the public API. + */ + nameToUrl: function (moduleName, ext, skipExt) { + var paths, syms, i, parentModule, url, + parentPath, bundleId, + pkgMain = getOwn(config.pkgs, moduleName); + + if (pkgMain) { + moduleName = pkgMain; + } + + bundleId = getOwn(bundlesMap, moduleName); + + if (bundleId) { + return context.nameToUrl(bundleId, ext, skipExt); + } + + //If a colon is in the URL, it indicates a protocol is used and it is just + //an URL to a file, or if it starts with a slash, contains a query arg (i.e. ?) + //or ends with .js, then assume the user meant to use an url and not a module id. + //The slash is important for protocol-less URLs as well as full paths. + if (req.jsExtRegExp.test(moduleName)) { + //Just a plain path, not module name lookup, so just return it. + //Add extension if it is included. This is a bit wonky, only non-.js things pass + //an extension, this method probably needs to be reworked. + url = moduleName + (ext || ''); + } else { + //A module that needs to be converted to a path. + paths = config.paths; + + syms = moduleName.split('/'); + //For each module name segment, see if there is a path + //registered for it. Start with most specific name + //and work up from it. + for (i = syms.length; i > 0; i -= 1) { + parentModule = syms.slice(0, i).join('/'); + + parentPath = getOwn(paths, parentModule); + if (parentPath) { + //If an array, it means there are a few choices, + //Choose the one that is desired + if (isArray(parentPath)) { + parentPath = parentPath[0]; + } + syms.splice(0, i, parentPath); + break; + } + } + + //Join the path parts together, then figure out if baseUrl is needed. + url = syms.join('/'); + url += (ext || (/^data\:|\?/.test(url) || skipExt ? '' : '.js')); + url = (url.charAt(0) === '/' || url.match(/^[\w\+\.\-]+:/) ? '' : config.baseUrl) + url; + } + + return config.urlArgs ? url + + ((url.indexOf('?') === -1 ? '?' : '&') + + config.urlArgs) : url; + }, + + //Delegates to req.load. Broken out as a separate function to + //allow overriding in the optimizer. + load: function (id, url) { + req.load(context, id, url); + }, + + /** + * Executes a module callback function. Broken out as a separate function + * solely to allow the build system to sequence the files in the built + * layer in the right sequence. + * + * @private + */ + execCb: function (name, callback, args, exports) { + return callback.apply(exports, args); + }, + + /** + * callback for script loads, used to check status of loading. + * + * @param {Event} evt the event from the browser for the script + * that was loaded. + */ + onScriptLoad: function (evt) { + //Using currentTarget instead of target for Firefox 2.0's sake. Not + //all old browsers will be supported, but this one was easy enough + //to support and still makes sense. + if (evt.type === 'load' || + (readyRegExp.test((evt.currentTarget || evt.srcElement).readyState))) { + //Reset interactive script so a script node is not held onto for + //to long. + interactiveScript = null; + + //Pull out the name of the module and the context. + var data = getScriptData(evt); + context.completeLoad(data.id); + } + }, + + /** + * Callback for script errors. + */ + onScriptError: function (evt) { + var data = getScriptData(evt); + if (!hasPathFallback(data.id)) { + return onError(makeError('scripterror', 'Script error for: ' + data.id, evt, [data.id])); + } + } + }; + + context.require = context.makeRequire(); + return context; + } + + /** + * Main entry point. + * + * If the only argument to require is a string, then the module that + * is represented by that string is fetched for the appropriate context. + * + * If the first argument is an array, then it will be treated as an array + * of dependency string names to fetch. An optional function callback can + * be specified to execute when all of those dependencies are available. + * + * Make a local req variable to help Caja compliance (it assumes things + * on a require that are not standardized), and to give a short + * name for minification/local scope use. + */ + req = requirejs = function (deps, callback, errback, optional) { + + //Find the right context, use default + var context, config, + contextName = defContextName; + + // Determine if have config object in the call. + if (!isArray(deps) && typeof deps !== 'string') { + // deps is a config object + config = deps; + if (isArray(callback)) { + // Adjust args if there are dependencies + deps = callback; + callback = errback; + errback = optional; + } else { + deps = []; + } + } + + if (config && config.context) { + contextName = config.context; + } + + context = getOwn(contexts, contextName); + if (!context) { + context = contexts[contextName] = req.s.newContext(contextName); + } + + if (config) { + context.configure(config); + } + + return context.require(deps, callback, errback); + }; + + /** + * Support require.config() to make it easier to cooperate with other + * AMD loaders on globally agreed names. + */ + req.config = function (config) { + return req(config); + }; + + /** + * Execute something after the current tick + * of the event loop. Override for other envs + * that have a better solution than setTimeout. + * @param {Function} fn function to execute later. + */ + req.nextTick = typeof setTimeout !== 'undefined' ? function (fn) { + setTimeout(fn, 4); + } : function (fn) { fn(); }; + + /** + * Export require as a global, but only if it does not already exist. + */ + if (!require) { + require = req; + } + + req.version = version; + + //Used to filter out dependencies that are already paths. + req.jsExtRegExp = /^\/|:|\?|\.js$/; + req.isBrowser = isBrowser; + s = req.s = { + contexts: contexts, + newContext: newContext + }; + + //Create default context. + req({}); + + //Exports some context-sensitive methods on global require. + each([ + 'toUrl', + 'undef', + 'defined', + 'specified' + ], function (prop) { + //Reference from contexts instead of early binding to default context, + //so that during builds, the latest instance of the default context + //with its config gets used. + req[prop] = function () { + var ctx = contexts[defContextName]; + return ctx.require[prop].apply(ctx, arguments); + }; + }); + + if (isBrowser) { + head = s.head = document.getElementsByTagName('head')[0]; + //If BASE tag is in play, using appendChild is a problem for IE6. + //When that browser dies, this can be removed. Details in this jQuery bug: + //http://dev.jquery.com/ticket/2709 + baseElement = document.getElementsByTagName('base')[0]; + if (baseElement) { + head = s.head = baseElement.parentNode; + } + } + + /** + * Any errors that require explicitly generates will be passed to this + * function. Intercept/override it if you want custom error handling. + * @param {Error} err the error object. + */ + req.onError = defaultOnError; + + /** + * Creates the node for the load command. Only used in browser envs. + */ + req.createNode = function (config, moduleName, url) { + var node = config.xhtml ? + document.createElementNS('http://www.w3.org/1999/xhtml', 'html:script') : + document.createElement('script'); + node.type = config.scriptType || 'text/javascript'; + node.charset = 'utf-8'; + node.async = true; + return node; + }; + + /** + * Does the request to load a module for the browser case. + * Make this a separate function to allow other environments + * to override it. + * + * @param {Object} context the require context to find state. + * @param {String} moduleName the name of the module. + * @param {Object} url the URL to the module. + */ + req.load = function (context, moduleName, url) { + var config = (context && context.config) || {}, + node; + if (isBrowser) { + //In the browser so use a script tag + node = req.createNode(config, moduleName, url); + + node.setAttribute('data-requirecontext', context.contextName); + node.setAttribute('data-requiremodule', moduleName); + + //Set up load listener. Test attachEvent first because IE9 has + //a subtle issue in its addEventListener and script onload firings + //that do not match the behavior of all other browsers with + //addEventListener support, which fire the onload event for a + //script right after the script execution. See: + //https://connect.microsoft.com/IE/feedback/details/648057/script-onload-event-is-not-fired-immediately-after-script-execution + //UNFORTUNATELY Opera implements attachEvent but does not follow the script + //script execution mode. + if (node.attachEvent && + //Check if node.attachEvent is artificially added by custom script or + //natively supported by browser + //read https://github.com/jrburke/requirejs/issues/187 + //if we can NOT find [native code] then it must NOT natively supported. + //in IE8, node.attachEvent does not have toString() + //Note the test for "[native code" with no closing brace, see: + //https://github.com/jrburke/requirejs/issues/273 + !(node.attachEvent.toString && node.attachEvent.toString().indexOf('[native code') < 0) && + !isOpera) { + //Probably IE. IE (at least 6-8) do not fire + //script onload right after executing the script, so + //we cannot tie the anonymous define call to a name. + //However, IE reports the script as being in 'interactive' + //readyState at the time of the define call. + useInteractive = true; + + node.attachEvent('onreadystatechange', context.onScriptLoad); + //It would be great to add an error handler here to catch + //404s in IE9+. However, onreadystatechange will fire before + //the error handler, so that does not help. If addEventListener + //is used, then IE will fire error before load, but we cannot + //use that pathway given the connect.microsoft.com issue + //mentioned above about not doing the 'script execute, + //then fire the script load event listener before execute + //next script' that other browsers do. + //Best hope: IE10 fixes the issues, + //and then destroys all installs of IE 6-9. + //node.attachEvent('onerror', context.onScriptError); + } else { + node.addEventListener('load', context.onScriptLoad, false); + node.addEventListener('error', context.onScriptError, false); + } + node.src = url; + + //For some cache cases in IE 6-8, the script executes before the end + //of the appendChild execution, so to tie an anonymous define + //call to the module name (which is stored on the node), hold on + //to a reference to this node, but clear after the DOM insertion. + currentlyAddingScript = node; + if (baseElement) { + head.insertBefore(node, baseElement); + } else { + head.appendChild(node); + } + currentlyAddingScript = null; + + return node; + } else if (isWebWorker) { + try { + //In a web worker, use importScripts. This is not a very + //efficient use of importScripts, importScripts will block until + //its script is downloaded and evaluated. However, if web workers + //are in play, the expectation that a build has been done so that + //only one script needs to be loaded anyway. This may need to be + //reevaluated if other use cases become common. + importScripts(url); + + //Account for anonymous modules + context.completeLoad(moduleName); + } catch (e) { + context.onError(makeError('importscripts', + 'importScripts failed for ' + + moduleName + ' at ' + url, + e, + [moduleName])); + } + } + }; + + function getInteractiveScript() { + if (interactiveScript && interactiveScript.readyState === 'interactive') { + return interactiveScript; + } + + eachReverse(scripts(), function (script) { + if (script.readyState === 'interactive') { + return (interactiveScript = script); + } + }); + return interactiveScript; + } + + //Look for a data-main script attribute, which could also adjust the baseUrl. + if (isBrowser && !cfg.skipDataMain) { + //Figure out baseUrl. Get it from the script tag with require.js in it. + eachReverse(scripts(), function (script) { + //Set the 'head' where we can append children by + //using the script's parent. + if (!head) { + head = script.parentNode; + } + + //Look for a data-main attribute to set main script for the page + //to load. If it is there, the path to data main becomes the + //baseUrl, if it is not already set. + dataMain = script.getAttribute('data-main'); + if (dataMain) { + //Preserve dataMain in case it is a path (i.e. contains '?') + mainScript = dataMain; + + //Set final baseUrl if there is not already an explicit one. + if (!cfg.baseUrl) { + //Pull off the directory of data-main for use as the + //baseUrl. + src = mainScript.split('/'); + mainScript = src.pop(); + subPath = src.length ? src.join('/') + '/' : './'; + + cfg.baseUrl = subPath; + } + + //Strip off any trailing .js since mainScript is now + //like a module name. + mainScript = mainScript.replace(jsSuffixRegExp, ''); + + //If mainScript is still a path, fall back to dataMain + if (req.jsExtRegExp.test(mainScript)) { + mainScript = dataMain; + } + + //Put the data-main script in the files to load. + cfg.deps = cfg.deps ? cfg.deps.concat(mainScript) : [mainScript]; + + return true; + } + }); + } + + /** + * The function that handles definitions of modules. Differs from + * require() in that a string for the module should be the first argument, + * and the function to execute after dependencies are loaded should + * return a value to define the module corresponding to the first argument's + * name. + */ + define = function (name, deps, callback) { + var node, context; + + //Allow for anonymous modules + if (typeof name !== 'string') { + //Adjust args appropriately + callback = deps; + deps = name; + name = null; + } + + //This module may not have dependencies + if (!isArray(deps)) { + callback = deps; + deps = null; + } + + //If no name, and callback is a function, then figure out if it a + //CommonJS thing with dependencies. + if (!deps && isFunction(callback)) { + deps = []; + //Remove comments from the callback string, + //look for require calls, and pull them into the dependencies, + //but only if there are function args. + if (callback.length) { + callback + .toString() + .replace(commentRegExp, '') + .replace(cjsRequireRegExp, function (match, dep) { + deps.push(dep); + }); + + //May be a CommonJS thing even without require calls, but still + //could use exports, and module. Avoid doing exports and module + //work though if it just needs require. + //REQUIRES the function to expect the CommonJS variables in the + //order listed below. + deps = (callback.length === 1 ? ['require'] : ['require', 'exports', 'module']).concat(deps); + } + } + + //If in IE 6-8 and hit an anonymous define() call, do the interactive + //work. + if (useInteractive) { + node = currentlyAddingScript || getInteractiveScript(); + if (node) { + if (!name) { + name = node.getAttribute('data-requiremodule'); + } + context = contexts[node.getAttribute('data-requirecontext')]; + } + } + + //Always save off evaluating the def call until the script onload handler. + //This allows multiple modules to be in a file without prematurely + //tracing dependencies, and allows for anonymous module support, + //where the module name is not known until the script onload event + //occurs. If no context, use the global queue, and get it processed + //in the onscript load callback. + (context ? context.defQueue : globalDefQueue).push([name, deps, callback]); + }; + + define.amd = { + jQuery: true + }; + + + /** + * Executes the text. Normally just uses eval, but can be modified + * to use a better, environment-specific call. Only used for transpiling + * loader plugins, not for plain JS modules. + * @param {String} text the text to execute/evaluate. + */ + req.exec = function (text) { + /*jslint evil: true */ + return eval(text); + }; + + //Set up with config info. + req(cfg); +}(this)); diff --git a/tests/jquery.js b/tests/jquery.js deleted file mode 100644 index 67511667d91..00000000000 --- a/tests/jquery.js +++ /dev/null @@ -1,24 +0,0 @@ -(function() { - -var current, version, url, - parts = document.location.search.slice( 1 ).split( "&" ), - length = parts.length, - i = 0; - -for ( ; i < length; i++ ) { - current = parts[ i ].split( "=" ); - if ( current[ 0 ] === "jquery" ) { - version = current[ 1 ]; - break; - } -} - -if ( version === "git" ) { - url = "http://code.jquery.com/jquery-git.js"; -} else { - url = "../../../external/jquery-" + ( version || "1.10.2" ) + "/jquery.js"; -} - -document.write( "" ); - -}() ); diff --git a/tests/unit/accordion/accordion.html b/tests/unit/accordion/accordion.html index 5a76ba566de..80b462da849 100644 --- a/tests/unit/accordion/accordion.html +++ b/tests/unit/accordion/accordion.html @@ -4,30 +4,12 @@ jQuery UI Accordion Test Suite - - - - - + - - - - - - - - + - - - + - - - - - - - - + @@ -122,5 +99,24 @@ + + + + + + + + diff --git a/tests/unit/draggable/draggable_common.js b/tests/unit/draggable/draggable_common.js index 5abd09e4935..e39209e4fea 100644 --- a/tests/unit/draggable/draggable_common.js +++ b/tests/unit/draggable/draggable_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "draggable", { +define([ + "helper/testsuite", + "ui/draggable" +], function( testHelper ) { + +testHelper.commonWidgetTests( "draggable", { defaults: { appendTo: "parent", axis: false, @@ -38,3 +43,5 @@ TestHelpers.commonWidgetTests( "draggable", { stop: null } }); + +}); diff --git a/tests/unit/draggable/draggable_core.js b/tests/unit/draggable/draggable_core.js index 81c31e22f1d..8f2e0808e1c 100644 --- a/tests/unit/draggable/draggable_core.js +++ b/tests/unit/draggable/draggable_core.js @@ -1,9 +1,16 @@ +define([ + "jquery", + "./draggable_test_helpers", + "helper/testsuite", + "jquery.simulate", + "ui/draggable", + "ui/resizable" +], function( $, draggableTestHelper, testHelper ) { + /* * draggable_core.js */ -(function( $ ) { - module( "draggable: core" ); test( "element types", function() { @@ -42,8 +49,8 @@ test( "element types", function() { // Support: FF, Chrome, and IE9, // there are some rounding errors in so we can't say equal, we have to settle for close enough - closeEnough( offsetBefore.left, offsetAfter.left - 50, 1, "dragged[50, 50] " + "<" + typeName + "> left" ); - closeEnough( offsetBefore.top, offsetAfter.top - 50, 1, "dragged[50, 50] " + "<" + typeName + "> top" ); + testHelper.closeEnough( offsetBefore.left, offsetAfter.left - 50, 1, "dragged[50, 50] " + "<" + typeName + "> left" ); + testHelper.closeEnough( offsetBefore.top, offsetAfter.top - 50, 1, "dragged[50, 50] " + "<" + typeName + "> top" ); el.draggable("destroy"); el.remove(); }); @@ -51,12 +58,12 @@ test( "element types", function() { test( "No options, relative", function() { expect( 2 ); - TestHelpers.draggable.shouldMove( $( "#draggable1" ).draggable(), "no options, relative" ); + draggableTestHelper.shouldMove( $( "#draggable1" ).draggable(), "no options, relative" ); }); test( "No options, absolute", function() { expect( 2 ); - TestHelpers.draggable.shouldMove( $( "#draggable2" ).draggable(), "no options, absolute" ); + draggableTestHelper.shouldMove( $( "#draggable2" ).draggable(), "no options, absolute" ); }); test( "resizable handle with complex markup (#8756 / #8757)", function() { @@ -99,7 +106,7 @@ test( "#8269: Removing draggable element on drop", function() { }); // Support: Opera 12.10, Safari 5.1, jQuery <1.8 - if ( TestHelpers.draggable.unreliableContains ) { + if ( draggableTestHelper.unreliableContains ) { ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); } else { @@ -125,7 +132,7 @@ test( "Stray mousemove after mousedown still drags", function() { $( document ).simulate( "mousemove", { button: -1 }); }); - TestHelpers.draggable.shouldMove( element, "element is draggable" ); + draggableTestHelper.shouldMove( element, "element is draggable" ); }); test( "#6258: not following mouse when scrolled and using overflow-y: scroll", function() { @@ -145,7 +152,7 @@ test( "#6258: not following mouse when scrolled and using overflow-y: scroll", f oldOverflowY = $( "html" ).css( "overflow-y" ), oldOverflowX = $( "html" ).css( "overflow-x" ); - TestHelpers.forceScrollableWindow(); + testHelper.forceScrollableWindow(); $( "html" ) .css( "overflow-y", "scroll" ) @@ -171,7 +178,7 @@ test( "#9315: jumps down with offset of scrollbar", function() { } }); - TestHelpers.forceScrollableWindow(); + testHelper.forceScrollableWindow(); $( "html" ).scrollTop( 300 ).scrollLeft( 300 ); @@ -205,7 +212,7 @@ test( "scroll offset with fixed ancestors", function() { } }); - TestHelpers.forceScrollableWindow(); + testHelper.forceScrollableWindow(); $( "#wrapper" ).css( "position", "fixed" ); $( "#wrapper2" ).css( "position", "absolute" ); @@ -276,8 +283,8 @@ test( "#5727: draggable from iframe", function() { equal( draggable1.closest( iframeBody ).length, 1 ); // TODO: fix draggable within an IFRAME to fire events on the element properly - // and these TestHelpers.draggable.shouldMove relies on events for testing - //TestHelpers.draggable.shouldMove( draggable1, "draggable from an iframe" ); + // and these draggableTestHelper.shouldMove relies on events for testing + //draggableTestHelper.shouldMove( draggable1, "draggable from an iframe" ); }); test( "#8399: A draggable should become the active element after you are finished interacting with it, but not before.", function() { @@ -289,7 +296,7 @@ test( "#8399: A draggable should become the active element after you are finishe notStrictEqual( document.activeElement, element.get( 0 ), "moving a draggable anchor did not make it the active element" ); }); - TestHelpers.draggable.move( element, 50, 50 ); + draggableTestHelper.move( element, 50, 50 ); strictEqual( document.activeElement, element.get( 0 ), "finishing moving a draggable anchor made it the active element" ); }); @@ -300,16 +307,16 @@ asyncTest( "blur behavior", function() { var element = $( "#draggable1" ).draggable(), focusElement = $( "
" ).appendTo( element ); - TestHelpers.onFocus( focusElement, function() { + testHelper.onFocus( focusElement, function() { strictEqual( document.activeElement, focusElement.get( 0 ), "test element is focused before mousing down on a draggable" ); - TestHelpers.draggable.move( focusElement, 1, 1 ); + draggableTestHelper.move( focusElement, 1, 1 ); // http://bugs.jqueryui.com/ticket/10527 // Draggable: Can't select option in modal dialog (IE8) strictEqual( document.activeElement, focusElement.get( 0 ), "test element is focused after mousing down on itself" ); - TestHelpers.draggable.move( element, 50, 50 ); + draggableTestHelper.move( element, 50, 50 ); // http://bugs.jqueryui.com/ticket/4261 // active element should blur when mousing down on a draggable @@ -362,16 +369,16 @@ test( "setting right/bottom css shouldn't cause resize", function() { element.draggable(); - TestHelpers.draggable.move( element, -50, -50 ); + draggableTestHelper.move( element, -50, -50 ); finalOffset = element.offset(); finalOffset.left += 50; finalOffset.top += 50; - closeEnough( element.width(), origWidth, 1, "element retains width" ); - closeEnough( element.height(), origHeight, 1, "element retains height" ); - closeEnough( finalOffset.top, origOffset.top, "element moves the correct vertical distance" ); - closeEnough( finalOffset.top, origOffset.top, "element moves the correct horizontal distance" ); + testHelper.closeEnough( element.width(), origWidth, 1, "element retains width" ); + testHelper.closeEnough( element.height(), origHeight, 1, "element retains height" ); + testHelper.closeEnough( finalOffset.top, origOffset.top, "element moves the correct vertical distance" ); + testHelper.closeEnough( finalOffset.top, origOffset.top, "element moves the correct horizontal distance" ); }); -})( jQuery ); +}); diff --git a/tests/unit/draggable/draggable_events.js b/tests/unit/draggable/draggable_events.js index dd5c315b0b4..66fa3b7eed6 100644 --- a/tests/unit/draggable/draggable_events.js +++ b/tests/unit/draggable/draggable_events.js @@ -1,7 +1,11 @@ +define([ + "jquery", + "ui/draggable" +], function( $ ) { + /* * draggable_events.js */ -(function( $ ) { var element; @@ -161,4 +165,4 @@ test( "position and offset in hash is consistent between start, drag, and stop", deepEqual( dragOffset, stopOffset, "drag offset equals stop offset" ); }); -})( jQuery ); +}); diff --git a/tests/unit/draggable/draggable_methods.js b/tests/unit/draggable/draggable_methods.js index 351d57764f0..c0311d700cb 100644 --- a/tests/unit/draggable/draggable_methods.js +++ b/tests/unit/draggable/draggable_methods.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "./draggable_test_helpers", + "ui/draggable" +], function( $, draggableTestHelper ) { + /* * draggable_methods.js */ -(function( $ ) { var element; @@ -54,19 +59,19 @@ test( "enable", function() { expect( 11 ); element.draggable({ disabled: true }); - TestHelpers.draggable.shouldNotDrag( element, ".draggable({ disabled: true })" ); + draggableTestHelper.shouldNotDrag( element, ".draggable({ disabled: true })" ); element.draggable("enable"); - TestHelpers.draggable.shouldMove( element, ".draggable('enable')" ); + draggableTestHelper.shouldMove( element, ".draggable('enable')" ); equal( element.draggable( "option", "disabled" ), false, "disabled option getter" ); element.draggable("destroy"); element.draggable({ disabled: true }); - TestHelpers.draggable.shouldNotDrag( element, ".draggable({ disabled: true })" ); + draggableTestHelper.shouldNotDrag( element, ".draggable({ disabled: true })" ); element.draggable( "option", "disabled", false ); equal(element.draggable( "option", "disabled" ), false, "disabled option setter" ); - TestHelpers.draggable.shouldMove( element, ".draggable('option', 'disabled', false)" ); + draggableTestHelper.shouldMove( element, ".draggable('option', 'disabled', false)" ); var expected = element.draggable(), actual = expected.draggable("enable"); @@ -77,19 +82,19 @@ test( "disable", function() { expect( 14 ); element = $( "#draggable2" ).draggable({ disabled: false }); - TestHelpers.draggable.shouldMove( element, ".draggable({ disabled: false })" ); + draggableTestHelper.shouldMove( element, ".draggable({ disabled: false })" ); element.draggable( "disable" ); - TestHelpers.draggable.shouldNotDrag( element, ".draggable('disable')" ); + draggableTestHelper.shouldNotDrag( element, ".draggable('disable')" ); equal( element.draggable( "option", "disabled" ), true, "disabled option getter" ); element.draggable( "destroy" ); element.draggable({ disabled: false }); - TestHelpers.draggable.shouldMove( element, ".draggable({ disabled: false })" ); + draggableTestHelper.shouldMove( element, ".draggable({ disabled: false })" ); element.draggable( "option", "disabled", true ); equal( element.draggable( "option", "disabled" ), true, "disabled option setter" ); - TestHelpers.draggable.shouldNotDrag( element, ".draggable('option', 'disabled', true)" ); + draggableTestHelper.shouldNotDrag( element, ".draggable('option', 'disabled', true)" ); ok( !element.draggable( "widget" ).hasClass( "ui-state-disabled" ), "element does not get ui-state-disabled" ); ok( !element.draggable( "widget" ).attr( "aria-disabled" ), "element does not get aria-disabled" ); @@ -100,4 +105,4 @@ test( "disable", function() { equal( actual, expected, "disable is chainable" ); }); -})( jQuery ); +}); diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js index e54ae1df46d..7e9de649690 100644 --- a/tests/unit/draggable/draggable_options.js +++ b/tests/unit/draggable/draggable_options.js @@ -1,4 +1,11 @@ -(function( $ ) { +define([ + "jquery", + "./draggable_test_helpers", + "helper/testsuite", + "ui/draggable", + "ui/droppable", + "ui/sortable" +], function( $, draggableTestHelper, testHelper ) { module( "draggable: options" ); @@ -6,10 +13,10 @@ module( "draggable: options" ); test( "{ appendTo: 'parent' }, default, no clone", function() { expect( 4 ); var element = $( "#draggable2" ).draggable({ appendTo: "parent" }); - TestHelpers.draggable.shouldMove( element, "absolute appendTo: parent" ); + draggableTestHelper.shouldMove( element, "absolute appendTo: parent" ); element = $( "#draggable1" ).draggable({ appendTo: "parent" }); - TestHelpers.draggable.shouldMove( element, "relative appendTo: parent" ); + draggableTestHelper.shouldMove( element, "relative appendTo: parent" ); }); // TODO: This doesn't actually test whether append happened, possibly remove @@ -17,20 +24,20 @@ test( "{ appendTo: Element }, no clone", function() { expect( 4 ); var element = $( "#draggable2" ).draggable({ appendTo: $( "#draggable2" ).parent()[ 0 ] }); - TestHelpers.draggable.shouldMove( element, "absolute appendTo: Element" ); + draggableTestHelper.shouldMove( element, "absolute appendTo: Element" ); element = $( "#draggable1" ).draggable({ appendTo: $( "#draggable2" ).parent()[ 0 ] }); - TestHelpers.draggable.shouldMove( element, "relative appendTo: Element" ); + draggableTestHelper.shouldMove( element, "relative appendTo: Element" ); }); // TODO: This doesn't actually test whether append happened, possibly remove test( "{ appendTo: Selector }, no clone", function() { expect( 4 ); var element = $( "#draggable2" ).draggable({ appendTo: "#main" }); - TestHelpers.draggable.shouldMove( element, "absolute appendTo: Selector" ); + draggableTestHelper.shouldMove( element, "absolute appendTo: Selector" ); element = $( "#draggable1" ).draggable({ appendTo: "#main" }); - TestHelpers.draggable.shouldMove( element, "relative appendTo: Selector" ); + draggableTestHelper.shouldMove( element, "relative appendTo: Selector" ); }); test( "{ appendTo: 'parent' }, default", function() { @@ -38,11 +45,11 @@ test( "{ appendTo: 'parent' }, default", function() { var element = $( "#draggable1" ).draggable(); - TestHelpers.draggable.trackAppendedParent( element ); + draggableTestHelper.trackAppendedParent( element ); equal( element.draggable( "option", "appendTo" ), "parent" ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] ); }); @@ -52,9 +59,9 @@ test( "{ appendTo: Element }", function() { var appendTo = $( "#draggable2" ).parent()[ 0 ], element = $( "#draggable1" ).draggable({ appendTo: appendTo }); - TestHelpers.draggable.trackAppendedParent( element ); + draggableTestHelper.trackAppendedParent( element ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_parent" ), appendTo ); }); @@ -64,9 +71,9 @@ test( "{ appendTo: jQuery }", function() { var appendTo = $( "#draggable2" ).parent(), element = $( "#draggable1" ).draggable({ appendTo: appendTo }); - TestHelpers.draggable.trackAppendedParent( element ); + draggableTestHelper.trackAppendedParent( element ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_parent" ), appendTo[ 0 ] ); }); @@ -76,9 +83,9 @@ test( "{ appendTo: Selector }", function() { var appendTo = "#main", element = $( "#draggable1" ).draggable({ appendTo: appendTo }); - TestHelpers.draggable.trackAppendedParent( element ); + draggableTestHelper.trackAppendedParent( element ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_parent" ), $(appendTo)[ 0 ] ); }); @@ -87,34 +94,34 @@ test( "appendTo, default, switching after initialization", function() { var element = $( "#draggable1" ).draggable({ helper: "clone" }); - TestHelpers.draggable.trackAppendedParent( element ); + draggableTestHelper.trackAppendedParent( element ); // Move and make sure element was appended to fixture - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_parent" ), $( "#main" )[ 0 ] ); // Move and make sure element was appended to main element.draggable( "option", "appendTo", $( "#qunit-fixture" ) ); - TestHelpers.draggable.move( element, 2, 2 ); + draggableTestHelper.move( element, 2, 2 ); equal( element.data( "last_dragged_parent" ), $( "#qunit-fixture" )[ 0 ] ); }); test( "{ axis: false }, default", function() { expect( 2 ); var element = $( "#draggable2" ).draggable({ axis: false }); - TestHelpers.draggable.shouldMove( element, "axis: false" ); + draggableTestHelper.shouldMove( element, "axis: false" ); }); test( "{ axis: 'x' }", function() { expect( 2 ); var element = $( "#draggable2" ).draggable({ axis: "x" }); - TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 0, "axis: x" ); + draggableTestHelper.testDrag( element, element, 50, 50, 50, 0, "axis: x" ); }); test( "{ axis: 'y' }", function() { expect( 2 ); var element = $( "#draggable2" ).draggable({ axis: "y" }); - TestHelpers.draggable.testDrag( element, element, 50, 50, 0, 50, "axis: y" ); + draggableTestHelper.testDrag( element, element, 50, 50, 0, 50, "axis: y" ); }); test( "{ axis: ? }, unexpected", function() { @@ -132,7 +139,7 @@ test( "{ axis: ? }, unexpected", function() { $.each(unexpected, function(key, val) { element = $( "#draggable2" ).draggable({ axis: val }); - TestHelpers.draggable.shouldMove( element, "axis: " + key ); + draggableTestHelper.shouldMove( element, "axis: " + key ); element.draggable( "destroy" ); }); }); @@ -143,15 +150,15 @@ test( "axis, default, switching after initialization", function() { var element = $( "#draggable1" ).draggable({ axis: false }); // Any Direction - TestHelpers.draggable.shouldMove( element, "axis: default" ); + draggableTestHelper.shouldMove( element, "axis: default" ); // Only horizontal element.draggable( "option", "axis", "x" ); - TestHelpers.draggable.testDrag( element, element, 50, 50, 50, 0, "axis: x as option" ); + draggableTestHelper.testDrag( element, element, 50, 50, 50, 0, "axis: x as option" ); // Vertical only element.draggable( "option", "axis", "y" ); - TestHelpers.draggable.testDrag( element, element, 50, 50, 0, 50, "axis: y as option" ); + draggableTestHelper.testDrag( element, element, 50, 50, 0, 50, "axis: y as option" ); }); @@ -161,12 +168,12 @@ test( "{ cancel: 'input,textarea,button,select,option' }, default", function() { $( "
" ).appendTo( "#qunit-fixture" ); var element = $( "#draggable-option-cancel-default" ).draggable({ cancel: "input,textarea,button,select,option" }); - TestHelpers.draggable.shouldMove( element, "cancel: default, element dragged" ); + draggableTestHelper.shouldMove( element, "cancel: default, element dragged" ); element.draggable( "destroy" ); element = $( "#draggable-option-cancel-default" ).draggable({ cancel: "input,textarea,button,select,option" }); - TestHelpers.draggable.shouldNotDrag( element, "cancel: default, input dragged", "#draggable-option-cancel-default input" ); + draggableTestHelper.shouldNotDrag( element, "cancel: default, input dragged", "#draggable-option-cancel-default input" ); element.draggable( "destroy" ); }); @@ -174,12 +181,12 @@ test( "{ cancel: 'span' }", function() { expect( 4 ); var element = $( "#draggable2" ).draggable(); - TestHelpers.draggable.shouldMove( element, "cancel: default, span dragged", "#draggable2 span" ); + draggableTestHelper.shouldMove( element, "cancel: default, span dragged", "#draggable2 span" ); element.draggable( "destroy" ); element = $( "#draggable2" ).draggable({ cancel: "span" }); - TestHelpers.draggable.shouldNotDrag( element, "cancel: span, span dragged", "#draggable2 span" ); + draggableTestHelper.shouldNotDrag( element, "cancel: span, span dragged", "#draggable2 span" ); }); test( "{ cancel: ? }, unexpected", function() { @@ -197,7 +204,7 @@ test( "{ cancel: ? }, unexpected", function() { $.each( unexpected, function( key, val ) { element = $( "#draggable2" ).draggable({ cancel: val }); - TestHelpers.draggable.shouldMove( element, "cancel: " + key ); + draggableTestHelper.shouldMove( element, "cancel: " + key ); element.draggable( "destroy" ); }); }); @@ -215,15 +222,15 @@ test( "{ cancel: Selectors }, matching parent selector", function() { $( "#wrapping a" ).append( element ); - TestHelpers.draggable.shouldMove( element, "drag span child", "#draggable2 span" ); - TestHelpers.draggable.shouldNotDrag( $( "#draggable2 span a" ), "drag span a" ); - TestHelpers.draggable.shouldNotDrag( $( "#wrapping a" ), "drag wrapping a" ); + draggableTestHelper.shouldMove( element, "drag span child", "#draggable2 span" ); + draggableTestHelper.shouldNotDrag( $( "#draggable2 span a" ), "drag span a" ); + draggableTestHelper.shouldNotDrag( $( "#wrapping a" ), "drag wrapping a" ); $( "#draggable2" ).draggable( "option", "cancel", "span > a" ); $( "#draggable2" ).find( "a" ).append( "" ); - TestHelpers.draggable.shouldMove( element, "drag span child", $( "#draggable2 span a" ).last() ); - TestHelpers.draggable.shouldNotDrag( $( "#draggable2 span a" ).first(), "drag span a first child" ); + draggableTestHelper.shouldMove( element, "drag span child", $( "#draggable2 span a" ).last() ); + draggableTestHelper.shouldNotDrag( $( "#draggable2 span a" ).first(), "drag span a first child" ); }); */ @@ -235,13 +242,13 @@ test( "cancelement, default, switching after initialization", function() { var input = $( "#draggable-option-cancel-default input" ), element = $( "#draggable-option-cancel-default" ).draggable(); - TestHelpers.draggable.shouldNotDrag( element, "cancel: default, input dragged", input ); + draggableTestHelper.shouldNotDrag( element, "cancel: default, input dragged", input ); element.draggable( "option", "cancel", "textarea" ); - TestHelpers.draggable.shouldMove( element, "cancel: textarea, input dragged", input ); + draggableTestHelper.shouldMove( element, "cancel: textarea, input dragged", input ); element.draggable( "option", "cancel", "input" ); - TestHelpers.draggable.shouldNotDrag( element, "cancel: input, input dragged", input ); + draggableTestHelper.shouldNotDrag( element, "cancel: input, input dragged", input ); }); test( "connectToSortable, dragging out of a sortable", function() { @@ -413,8 +420,8 @@ test( "{ containment: Element }", function() { p = element.parent(), po = p.offset(), expected = { - left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ), - top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" ) + left: po.left + draggableTestHelper.border( p, "left" ) + draggableTestHelper.margin( element, "left" ), + top: po.top + draggableTestHelper.border( p, "top" ) + draggableTestHelper.margin( element, "top" ) }; element.simulate( "drag", { @@ -433,8 +440,8 @@ test( "{ containment: Selector }", function() { p = element.parent(), po = p.offset(), expected = { - left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ), - top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" ) + left: po.left + draggableTestHelper.border( p, "left" ) + draggableTestHelper.margin( element, "left" ), + top: po.top + draggableTestHelper.border( p, "top" ) + draggableTestHelper.margin( element, "top" ) }; element.simulate( "drag", { @@ -453,7 +460,7 @@ test( "{ containment: [x1, y1, x2, y2] }", function() { element.draggable( "option", "containment", [ eo.left, eo.top, eo.left + element.width() + 5, eo.top + element.height() + 5 ] ); - TestHelpers.draggable.testDrag( element, element, -100, -100, 0, 0, "containment: [x1, y1, x2, y2]" ); + draggableTestHelper.testDrag( element, element, -100, -100, 0, 0, "containment: [x1, y1, x2, y2]" ); }); test( "{ containment: 'parent' }, relative", function() { @@ -464,8 +471,8 @@ test( "{ containment: 'parent' }, relative", function() { p = element.parent(), po = p.offset(), expected = { - left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ), - top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" ) + left: po.left + draggableTestHelper.border( p, "left" ) + draggableTestHelper.margin( element, "left" ), + top: po.top + draggableTestHelper.border( p, "top" ) + draggableTestHelper.margin( element, "top" ) }; element.simulate( "drag", { @@ -484,8 +491,8 @@ test( "{ containment: 'parent' }, absolute", function() { p = element.parent(), po = p.offset(), expected = { - left: po.left + TestHelpers.draggable.border( p, "left" ) + TestHelpers.draggable.margin( element, "left" ), - top: po.top + TestHelpers.draggable.border( p, "top" ) + TestHelpers.draggable.margin( element, "top" ) + left: po.left + draggableTestHelper.border( p, "left" ) + draggableTestHelper.margin( element, "left" ), + top: po.top + draggableTestHelper.border( p, "top" ) + draggableTestHelper.margin( element, "top" ) }; element.simulate( "drag", { @@ -508,8 +515,8 @@ test( "containment, account for border", function() { }), parentBottom = parent.offset().top + parent.outerHeight(), parentRight = parent.offset().left + parent.outerWidth(), - parentBorderBottom = TestHelpers.draggable.border( parent, "bottom" ), - parentBorderRight = TestHelpers.draggable.border( parent, "right" ); + parentBorderBottom = draggableTestHelper.border( parent, "bottom" ), + parentBorderRight = draggableTestHelper.border( parent, "right" ); el.css({ height: "5px", @@ -521,9 +528,9 @@ test( "containment, account for border", function() { dy: 100 }); - closeEnough( el.offset().top, parentBottom - parentBorderBottom - el.height(), 1, + testHelper.closeEnough( el.offset().top, parentBottom - parentBorderBottom - el.height(), 1, "The draggable should be on top of its parent's bottom border" ); - closeEnough( el.offset().left, parentRight - parentBorderRight - el.width(), 1, + testHelper.closeEnough( el.offset().left, parentRight - parentBorderRight - el.width(), 1, "The draggable should be to the right of its parent's right border" ); }); @@ -564,16 +571,16 @@ test( "containment, default, switching after initialization", function() { po = element.parent().offset(), containment = [ po.left - 100, po.top - 100, po.left + 500, po.top + 500 ]; - TestHelpers.draggable.testDrag( element, element, -100, -100, -100, -100, "containment: default" ); + draggableTestHelper.testDrag( element, element, -100, -100, -100, -100, "containment: default" ); element.draggable( "option", "containment", "parent" ).css({ top: 0, left: 0 }); - TestHelpers.draggable.testDrag( element, element, -100, -100, 0, 0, "containment: parent as option" ); + draggableTestHelper.testDrag( element, element, -100, -100, 0, 0, "containment: parent as option" ); element.draggable( "option", "containment", containment ).css({ top: 0, left: 0 }); - TestHelpers.draggable.testDrag( element, element, -100, -100, -100, -100, "containment: array as option" ); + draggableTestHelper.testDrag( element, element, -100, -100, -100, -100, "containment: array as option" ); element.draggable( "option", "containment", false ); - TestHelpers.draggable.testDrag( element, element, -100, -100, -100, -100, "containment: false as option" ); + draggableTestHelper.testDrag( element, element, -100, -100, -100, -100, "containment: false as option" ); }); test( "{ cursor: 'auto' }, default", function() { @@ -651,7 +658,7 @@ test( "#6889: Cursor doesn't revert to pre-dragging state after revert action wh }), expected = getCursor(); - if ( TestHelpers.draggable.unreliableContains ) { + if ( draggableTestHelper.unreliableContains ) { ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); } else { @@ -667,17 +674,17 @@ test( "cursor, default, switching after initialization", function() { var element = $( "#draggable1" ).draggable(); - TestHelpers.draggable.trackMouseCss( element ); + draggableTestHelper.trackMouseCss( element ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_cursor" ), "auto" ); element.draggable( "option", "cursor", "move" ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_cursor" ), "move" ); element.draggable( "option", "cursor", "ns-resize" ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.data( "last_dragged_cursor" ), "ns-resize" ); }); @@ -704,11 +711,13 @@ test( "cursorAt", function() { equal( ui.position.left - ui.originalPosition.left, deltaX, testName + " " + position + " left" ); equal( ui.position.top - ui.originalPosition.top, deltaY, testName + " " + position + " top" ); } else if ( testData.cursorAt.right ) { - equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x - TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" ); - equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y - TestHelpers.draggable.unreliableOffset, testName + " " + position + " top" ); + equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x - + draggableTestHelper.unreliableOffset, testName + " " + position + " left" ); + equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y - + draggableTestHelper.unreliableOffset, testName + " " + position + " top" ); } else { - equal( event.clientX - ui.offset.left, testData.x + TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" ); - equal( event.clientY - ui.offset.top, testData.y + TestHelpers.draggable.unreliableOffset, testName + " " + position + " top" ); + equal( event.clientX - ui.offset.left, testData.x + draggableTestHelper.unreliableOffset, testName + " " + position + " left" ); + equal( event.clientY - ui.offset.top, testData.y + draggableTestHelper.unreliableOffset, testName + " " + position + " top" ); } } }); @@ -746,11 +755,13 @@ test( "cursorAt, switching after initialization", function() { equal( ui.position.left - ui.originalPosition.left, deltaX, testName + " " + position + " left" ); equal( ui.position.top - ui.originalPosition.top, deltaY, testName + " " + position + " top" ); } else if ( testData.cursorAt.right ) { - equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x - TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" ); - equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y - TestHelpers.draggable.unreliableOffset, testName + " " + position + " top" ); + equal( ui.helper.width() - ( event.clientX - ui.offset.left ), testData.x - + draggableTestHelper.unreliableOffset, testName + " " + position + " left" ); + equal( ui.helper.height() - ( event.clientY - ui.offset.top ), testData.y - + draggableTestHelper.unreliableOffset, testName + " " + position + " top" ); } else { - equal( event.clientX - ui.offset.left, testData.x + TestHelpers.draggable.unreliableOffset, testName + " " + position + " left" ); - equal( event.clientY - ui.offset.top, testData.y + TestHelpers.draggable.unreliableOffset, testName + " " + position + " top" ); + equal( event.clientX - ui.offset.left, testData.x + draggableTestHelper.unreliableOffset, testName + " " + position + " left" ); + equal( event.clientY - ui.offset.top, testData.y + draggableTestHelper.unreliableOffset, testName + " " + position + " top" ); } } }); @@ -772,29 +783,29 @@ test( "disabled", function() { var element = $( "#draggable1" ).draggable(); - TestHelpers.draggable.shouldMove( element, "disabled: default" ); + draggableTestHelper.shouldMove( element, "disabled: default" ); element.draggable( "option", "disabled", true ); - TestHelpers.draggable.shouldNotDrag( element, "option: disabled true" ); + draggableTestHelper.shouldNotDrag( element, "option: disabled true" ); element.draggable( "option", "disabled", false ); - TestHelpers.draggable.shouldMove( element, "option: disabled false" ); + draggableTestHelper.shouldMove( element, "option: disabled false" ); }); test( "{ grid: [50, 50] }, relative", function() { expect( 4 ); var element = $( "#draggable1" ).draggable({ grid: [ 50, 50 ] }); - TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] relative" ); - TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] relative" ); + draggableTestHelper.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] relative" ); + draggableTestHelper.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] relative" ); }); test( "{ grid: [50, 50] }, absolute", function() { expect( 4 ); var element = $( "#draggable2" ).draggable({ grid: [ 50, 50 ] }); - TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] absolute" ); - TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] absolute" ); + draggableTestHelper.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] absolute" ); + draggableTestHelper.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] absolute" ); }); test( "grid, switching after initialization", function() { @@ -803,13 +814,13 @@ test( "grid, switching after initialization", function() { var element = $( "#draggable1" ).draggable(); // Forward - TestHelpers.draggable.testDrag( element, element, 24, 24, 24, 24, "grid: default" ); - TestHelpers.draggable.testDrag( element, element, 0, 0, 0, 0, "grid: default" ); + draggableTestHelper.testDrag( element, element, 24, 24, 24, 24, "grid: default" ); + draggableTestHelper.testDrag( element, element, 0, 0, 0, 0, "grid: default" ); element.draggable( "option", "grid", [ 50, 50 ] ); - TestHelpers.draggable.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] as option" ); - TestHelpers.draggable.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] as option" ); + draggableTestHelper.testDrag( element, element, 24, 24, 0, 0, "grid: [50, 50] as option" ); + draggableTestHelper.testDrag( element, element, 26, 25, 50, 50, "grid: [50, 50] as option" ); }); test( "{ handle: 'span' }", function() { @@ -817,9 +828,9 @@ test( "{ handle: 'span' }", function() { var element = $( "#draggable2" ).draggable({ handle: "span" }); - TestHelpers.draggable.shouldMove( element, "handle: span", "#draggable2 span"); - TestHelpers.draggable.shouldMove( element, "handle: span child", "#draggable2 span em" ); - TestHelpers.draggable.shouldNotDrag( element, "handle: span element" ); + draggableTestHelper.shouldMove( element, "handle: span", "#draggable2 span"); + draggableTestHelper.shouldMove( element, "handle: span child", "#draggable2 span em" ); + draggableTestHelper.shouldNotDrag( element, "handle: span element" ); }); test( "handle, default, switching after initialization", function() { @@ -827,31 +838,31 @@ test( "handle, default, switching after initialization", function() { var element = $( "#draggable2" ).draggable(); - TestHelpers.draggable.shouldMove( element, "handle: default, element dragged" ); - TestHelpers.draggable.shouldMove( element, "handle: default, span dragged", "#draggable2 span" ); + draggableTestHelper.shouldMove( element, "handle: default, element dragged" ); + draggableTestHelper.shouldMove( element, "handle: default, span dragged", "#draggable2 span" ); // Switch element.draggable( "option", "handle", "span" ); - TestHelpers.draggable.shouldNotDrag( element, "handle: span as option, element dragged" ); - TestHelpers.draggable.shouldMove( element, "handle: span as option, span dragged", "#draggable2 span" ); + draggableTestHelper.shouldNotDrag( element, "handle: span as option, element dragged" ); + draggableTestHelper.shouldMove( element, "handle: span as option, span dragged", "#draggable2 span" ); // And back element.draggable( "option", "handle", false ); - TestHelpers.draggable.shouldMove( element, "handle: false as option, element dragged" ); - TestHelpers.draggable.shouldMove( element, "handle: false as option, span dragged", "#draggable2 span" ); + draggableTestHelper.shouldMove( element, "handle: false as option, element dragged" ); + draggableTestHelper.shouldMove( element, "handle: false as option, span dragged", "#draggable2 span" ); }); test( "helper, default, switching after initialization", function() { expect( 6 ); var element = $( "#draggable1" ).draggable(); - TestHelpers.draggable.shouldMove( element, "helper: default" ); + draggableTestHelper.shouldMove( element, "helper: default" ); element.draggable( "option", "helper", "clone" ); - TestHelpers.draggable.shouldMove( element, "helper: clone" ); + draggableTestHelper.shouldMove( element, "helper: clone" ); element.draggable( "option", "helper", "original" ); - TestHelpers.draggable.shouldMove( element, "helper: original" ); + draggableTestHelper.shouldMove( element, "helper: original" ); }); // http://bugs.jqueryui.com/ticket/9446 @@ -865,7 +876,7 @@ test( "helper, function returning original element", function() { } }); - TestHelpers.draggable.testDragHelperOffset( element, 100, 100, 100, 100, "original element is draggable" ); + draggableTestHelper.testDragHelperOffset( element, 100, 100, 100, 100, "original element is draggable" ); element.simulate( "drag", { dx: 100, @@ -884,25 +895,25 @@ function testHelperPosition( scrollPositions, position, helper, scrollElements, }); if ( scrollElements.length === 1 && scrollElements[ 0 ] === "#scrollParent" ) { - TestHelpers.draggable.setScrollable( "#main", false ); - TestHelpers.draggable.setScrollable( "#scrollParent", true ); + draggableTestHelper.setScrollable( "#main", false ); + draggableTestHelper.setScrollable( "#scrollParent", true ); } for ( j = 0; j < scrollPositions.length; j++ ) { for ( i = 0; i < scrollElements.length; i++ ) { - TestHelpers.draggable.setScroll( scrollElements[ i ] ); + draggableTestHelper.setScroll( scrollElements[ i ] ); } - TestHelpers.draggable.testScroll( element, scrollPositions[ j ] ); + draggableTestHelper.testScroll( element, scrollPositions[ j ] ); for ( i = 0; i < scrollElements.length; i++ ) { - TestHelpers.draggable.restoreScroll( scrollElements[ i ] ); + draggableTestHelper.restoreScroll( scrollElements[ i ] ); } } if ( scrollElements.length === 1 && scrollElements[ 1 ] === "#scrollParent" ) { - TestHelpers.draggable.setScrollable( "#main", true ); - TestHelpers.draggable.setScrollable( "#scrollParent", false ); + draggableTestHelper.setScrollable( "#main", true ); + draggableTestHelper.setScrollable( "#scrollParent", false ); } }); } @@ -960,15 +971,15 @@ test( "opacity, default, switching after initialization", function() { } }); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( opacity, 1 ); element.draggable( "option", "opacity", 0.5 ); - TestHelpers.draggable.move( element, 2, 1 ); + draggableTestHelper.move( element, 2, 1 ); equal( opacity, 0.5 ); element.draggable( "option", "opacity", false ); - TestHelpers.draggable.move( element, 3, 1 ); + draggableTestHelper.move( element, 3, 1 ); equal( opacity, 1 ); }); @@ -979,13 +990,13 @@ asyncTest( "revert and revertDuration", function() { revert: true, revertDuration: 0 }); - TestHelpers.draggable.shouldMovePositionButNotOffset( element, "revert: true, revertDuration: 0 should revert immediately" ); + draggableTestHelper.shouldMovePositionButNotOffset( element, "revert: true, revertDuration: 0 should revert immediately" ); $( "#draggable2" ).draggable( "option", "revert", "invalid" ); - TestHelpers.draggable.shouldMovePositionButNotOffset( element, "revert: invalid, revertDuration: 0 should revert immediately" ); + draggableTestHelper.shouldMovePositionButNotOffset( element, "revert: invalid, revertDuration: 0 should revert immediately" ); $( "#draggable2" ).draggable( "option", "revert", false ); - TestHelpers.draggable.shouldMove( element, "revert: false should allow movement" ); + draggableTestHelper.shouldMove( element, "revert: false should allow movement" ); $( "#draggable2" ).draggable( "option", { revert: true, @@ -996,7 +1007,7 @@ asyncTest( "revert and revertDuration", function() { }); // animation are async, so test for it asynchronously - TestHelpers.draggable.move( element, 50, 50 ); + draggableTestHelper.move( element, 50, 50 ); setTimeout( function() { ok( $( "#draggable2" ).is( ":animated" ), "revert: true with revertDuration should animate" ); }); @@ -1012,7 +1023,7 @@ test( "revert: valid", function() { $( "#droppable" ).droppable(); - TestHelpers.draggable.shouldMovePositionButNotOffset( element, "revert: valid reverts when dropped on a droppable" ); + draggableTestHelper.shouldMovePositionButNotOffset( element, "revert: valid reverts when dropped on a droppable" ); }); test( "scope", function() { @@ -1026,17 +1037,17 @@ test( "scope", function() { $( "#droppable" ).droppable({ scope: "tasks" }); - TestHelpers.draggable.shouldMovePositionButNotOffset( element, "revert: valid reverts when dropped on a droppable in scope" ); + draggableTestHelper.shouldMovePositionButNotOffset( element, "revert: valid reverts when dropped on a droppable in scope" ); $( "#droppable" ).droppable( "destroy" ).droppable({ scope: "nottasks" }); - TestHelpers.draggable.shouldMove( element, "revert: valid reverts when dropped on a droppable out of scope" ); + draggableTestHelper.shouldMove( element, "revert: valid reverts when dropped on a droppable out of scope" ); }); test( "scroll, scrollSensitivity, and scrollSpeed", function() { expect( 2 ); - TestHelpers.draggable.setScrollable( "#main", false ); + draggableTestHelper.setScrollable( "#main", false ); var currentScrollTop, viewportHeight = $( window ).height(), @@ -1078,7 +1089,7 @@ test( "scroll, scrollSensitivity, and scrollSpeed", function() { moves: 1 }); - TestHelpers.draggable.restoreScroll( document ); + draggableTestHelper.restoreScroll( document ); }); test( "scroll ignores containers that are overflow: hidden", function() { @@ -1117,7 +1128,7 @@ test( "scroll ignores containers that are overflow: hidden", function() { test( "#6817: auto scroll goes double distance when dragging", function() { expect( 2 ); - TestHelpers.draggable.restoreScroll( document ); + draggableTestHelper.restoreScroll( document ); var offsetBefore, distance = 10, @@ -1127,7 +1138,7 @@ test( "#6817: auto scroll goes double distance when dragging", function() { stop: function( e, ui ) { equal( ui.offset.top, newY, "offset of item matches pointer position after scroll" ); // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal - closeEnough( ui.offset.top - offsetBefore.top, distance, 1, "offset of item only moves expected distance after scroll" ); + testHelper.closeEnough( ui.offset.top - offsetBefore.top, distance, 1, "offset of item only moves expected distance after scroll" ); } }), scrollSensitivity = element.draggable( "option", "scrollSensitivity" ), @@ -1148,7 +1159,7 @@ test( "#6817: auto scroll goes double distance when dragging", function() { moves: 1 }); - TestHelpers.draggable.restoreScroll( document ); + draggableTestHelper.restoreScroll( document ); }); test( "snap, snapMode, and snapTolerance", function() { @@ -1184,8 +1195,8 @@ test( "snap, snapMode, and snapTolerance", function() { }); // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal - closeEnough( element.offset().left, newX, 1, "doesn't snap outside the snapTolerance" ); - closeEnough( element.offset().top, newY, 1, "doesn't snap outside the snapTolerance" ); + testHelper.closeEnough( element.offset().left, newX, 1, "doesn't snap outside the snapTolerance" ); + testHelper.closeEnough( element.offset().top, newY, 1, "doesn't snap outside the snapTolerance" ); newX += 3; @@ -1310,13 +1321,13 @@ test( "#8459: element can snap to an element that was removed during drag", func }); // Support: Opera 12.10, Safari 5.1, jQuery <1.8 - if ( TestHelpers.draggable.unreliableContains ) { + if ( draggableTestHelper.unreliableContains ) { ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" ); } else { // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal - closeEnough( element.offset().left, newX, 1, "doesn't snap to a removed element" ); - closeEnough( element.offset().top, newY, 1, "doesn't snap to a removed element" ); + testHelper.closeEnough( element.offset().left, newX, 1, "doesn't snap to a removed element" ); + testHelper.closeEnough( element.offset().top, newY, 1, "doesn't snap to a removed element" ); } }); @@ -1367,10 +1378,10 @@ test( "stack", function() { stack: "#draggable1, #draggable2" }); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( element.css( "zIndex" ), "2", "stack increments zIndex correctly" ); - TestHelpers.draggable.move( element2, 1, 1 ); + draggableTestHelper.move( element2, 1, 1 ); equal( element2.css( "zIndex" ), "3", "stack increments zIndex correctly" ); }); @@ -1408,15 +1419,15 @@ test( "zIndex, default, switching after initialization", function() { element.css( "z-index", 1 ); - TestHelpers.draggable.move( element, 1, 1 ); + draggableTestHelper.move( element, 1, 1 ); equal( zindex, 1 ); element.draggable( "option", "zIndex", 5 ); - TestHelpers.draggable.move( element, 2, 1 ); + draggableTestHelper.move( element, 2, 1 ); equal( zindex, 5 ); element.draggable( "option", "zIndex", false ); - TestHelpers.draggable.move( element, 3, 1 ); + draggableTestHelper.move( element, 3, 1 ); equal( zindex, 1 ); }); @@ -1463,4 +1474,4 @@ test( "iframeFix", function() { }); }); -})( jQuery ); +}); diff --git a/tests/unit/draggable/draggable_test_helpers.js b/tests/unit/draggable/draggable_test_helpers.js index 1b004e7cfab..2e4d5af7245 100644 --- a/tests/unit/draggable/draggable_test_helpers.js +++ b/tests/unit/draggable/draggable_test_helpers.js @@ -1,4 +1,55 @@ -TestHelpers.draggable = { +define([ + "jquery", + "helper/testsuite" +], function( $, testHelper ) { + +function testDragPosition( el, dx, dy, expectedDX, expectedDY, msg ) { + msg = msg ? msg + "." : ""; + + $( el ).one( "dragstop", function( event, ui ) { + var positionExpected = { left: ui.originalPosition.left + expectedDX, top: ui.originalPosition.top + expectedDY }; + deepEqual( ui.position, positionExpected, "position dragged[" + dx + ", " + dy + "] " + msg ); + }); +} + +function testDragOffset( el, dx, dy, expectedDX, expectedDY, msg ) { + msg = msg ? msg + "." : ""; + + var offsetBefore = el.offset(), + offsetExpected = { left: offsetBefore.left + expectedDX, top: offsetBefore.top + expectedDY }; + + $( el ).one( "dragstop", function( event, ui ) { + deepEqual( ui.offset, offsetExpected, "offset dragged[" + dx + ", " + dy + "] " + msg ); + }); +} + +function testDragHelperOffset( el, dx, dy, expectedDX, expectedDY, msg ) { + msg = msg ? msg + "." : ""; + + var offsetBefore = el.offset(), + offsetExpected = { left: offsetBefore.left + expectedDX, top: offsetBefore.top + expectedDY }; + + $( el ).one( "dragstop", function( event, ui ) { + deepEqual( ui.helper.offset(), offsetExpected, "offset dragged[" + dx + ", " + dy + "] " + msg ); + }); +} + +function testDrag( el, handle, dx, dy, expectedDX, expectedDY, msg ) { + testDragPosition( el, dx, dy, expectedDX, expectedDY, msg ); + testDragOffset( el, dx, dy, expectedDX, expectedDY, msg ); + + $( handle ).simulate( "drag", { + dx: dx, + dy: dy + }); +} + +function shouldMove( el, msg, handle ) { + handle = handle || el; + testDrag( el, handle, 100, 100, 100, 100, msg ); +} + +return { // TODO: remove the unreliable offset hacks unreliableOffset: $.ui.ie && ( !document.documentMode || document.documentMode < 8 ) ? 2 : 0, // Support: Opera 12.10, Safari 5.1, jQuery <1.8 @@ -6,60 +57,24 @@ TestHelpers.draggable = { var element = $( "
" ); return $.contains( element[ 0 ].ownerDocument, element[ 0 ] ); })(), - testDragPosition: function( el, dx, dy, expectedDX, expectedDY, msg ) { - msg = msg ? msg + "." : ""; - - $( el ).one( "dragstop", function( event, ui ) { - var positionExpected = { left: ui.originalPosition.left + expectedDX, top: ui.originalPosition.top + expectedDY }; - deepEqual( ui.position, positionExpected, "position dragged[" + dx + ", " + dy + "] " + msg ); - }); - }, - testDragOffset: function( el, dx, dy, expectedDX, expectedDY, msg ) { - msg = msg ? msg + "." : ""; - - var offsetBefore = el.offset(), - offsetExpected = { left: offsetBefore.left + expectedDX, top: offsetBefore.top + expectedDY }; - - $( el ).one( "dragstop", function( event, ui ) { - deepEqual( ui.offset, offsetExpected, "offset dragged[" + dx + ", " + dy + "] " + msg ); - }); - }, - testDragHelperOffset: function( el, dx, dy, expectedDX, expectedDY, msg ) { - msg = msg ? msg + "." : ""; - - var offsetBefore = el.offset(), - offsetExpected = { left: offsetBefore.left + expectedDX, top: offsetBefore.top + expectedDY }; - - $( el ).one( "dragstop", function( event, ui ) { - deepEqual( ui.helper.offset(), offsetExpected, "offset dragged[" + dx + ", " + dy + "] " + msg ); - }); - }, - testDrag: function( el, handle, dx, dy, expectedDX, expectedDY, msg ) { - TestHelpers.draggable.testDragPosition( el, dx, dy, expectedDX, expectedDY, msg ); - TestHelpers.draggable.testDragOffset( el, dx, dy, expectedDX, expectedDY, msg ); - - $( handle ).simulate( "drag", { - dx: dx, - dy: dy - }); - }, + testDragPosition: testDragPosition, + testDragOffset: testDragOffset, + testDragHelperOffset: testDragHelperOffset, + testDrag: testDrag, shouldMovePositionButNotOffset: function( el, msg, handle ) { handle = handle || el; - TestHelpers.draggable.testDragPosition( el, 100, 100, 100, 100, msg ); - TestHelpers.draggable.testDragHelperOffset( el, 100, 100, 0, 0, msg ); + testDragPosition( el, 100, 100, 100, 100, msg ); + testDragHelperOffset( el, 100, 100, 0, 0, msg ); $( handle ).simulate( "drag", { dx: 100, dy: 100 }); }, - shouldMove: function( el, msg, handle ) { - handle = handle || el; - TestHelpers.draggable.testDrag( el, handle, 100, 100, 100, 100, msg ); - }, + shouldMove: shouldMove, shouldNotMove: function( el, msg, handle ) { handle = handle || el; - TestHelpers.draggable.testDrag( el, handle, 100, 100, 0, 0, msg ); + testDrag( el, handle, 100, 100, 0, 0, msg ); }, shouldNotDrag: function( el, msg, handle ) { handle = handle || el; @@ -93,7 +108,7 @@ TestHelpers.draggable = { testScroll: function( el, position ) { var oldPosition = $( "#main" ).css( "position" ); $( "#main" ).css({ position: position, top: "0px", left: "0px" }); - TestHelpers.draggable.shouldMove( el, position + " parent" ); + shouldMove( el, position + " parent" ); $( "#main" ).css( "position", oldPosition ); }, restoreScroll: function( what ) { @@ -129,3 +144,5 @@ TestHelpers.draggable = { }); } }; + +}); diff --git a/tests/unit/droppable/droppable.html b/tests/unit/droppable/droppable.html index 03ff888e87b..3e6584fe5a7 100644 --- a/tests/unit/droppable/droppable.html +++ b/tests/unit/droppable/droppable.html @@ -4,32 +4,12 @@ jQuery UI Droppable Test Suite - - - - - + - - - - - - - - + @@ -42,5 +22,24 @@
 
+ + + + + + + + diff --git a/tests/unit/droppable/droppable_common.js b/tests/unit/droppable/droppable_common.js index c112def3cf4..3e526175260 100644 --- a/tests/unit/droppable/droppable_common.js +++ b/tests/unit/droppable/droppable_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "droppable", { +define([ + "helper/testsuite", + "ui/droppable" +], function( testHelper ) { + +testHelper.commonWidgetTests( "droppable", { defaults: { accept: "*", activeClass: false, @@ -18,3 +23,5 @@ TestHelpers.commonWidgetTests( "droppable", { over: null } }); + +}); diff --git a/tests/unit/droppable/droppable_core.js b/tests/unit/droppable/droppable_core.js index 53b08fd8565..7ab92409cb6 100644 --- a/tests/unit/droppable/droppable_core.js +++ b/tests/unit/droppable/droppable_core.js @@ -1,9 +1,13 @@ +define([ + "jquery", + "./droppable_test_helpers", + "ui/droppable" +], function( $, droppableTestHelpers ) { + /* * droppable_core.js */ -(function($) { - module("droppable: core"); test("element types", function() { @@ -20,10 +24,10 @@ test("element types", function() { (typeName === "table" && el.append("content")); el.droppable(); - TestHelpers.droppable.shouldDrop(); + droppableTestHelpers.shouldDrop(); el.droppable("destroy"); el.remove(); }); }); -})(jQuery); +}); diff --git a/tests/unit/droppable/droppable_events.js b/tests/unit/droppable/droppable_events.js index 4b8fe5acd2c..5e0bc6bfffe 100644 --- a/tests/unit/droppable/droppable_events.js +++ b/tests/unit/droppable/droppable_events.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "ui/droppable", + "jquery.simulate" +], function( $ ) { module( "droppable: events" ); @@ -60,4 +64,4 @@ test("drop", function() { }); */ -})( jQuery ); +}); diff --git a/tests/unit/droppable/droppable_methods.js b/tests/unit/droppable/droppable_methods.js index a8a2a078c2e..667bb02f3a3 100644 --- a/tests/unit/droppable/droppable_methods.js +++ b/tests/unit/droppable/droppable_methods.js @@ -1,7 +1,8 @@ -/* - * droppable_methods.js - */ -(function($) { +define([ + "jquery", + "./droppable_test_helpers", + "ui/droppable" +], function( $, droppableTestHelper ) { module("droppable: methods"); @@ -47,16 +48,16 @@ test("enable", function() { var el, expected, actual; el = $("#droppable1").droppable({ disabled: true }); - TestHelpers.droppable.shouldNotDrop(); + droppableTestHelper.shouldNotDrop(); el.droppable("enable"); - TestHelpers.droppable.shouldDrop(); + droppableTestHelper.shouldDrop(); equal(el.droppable("option", "disabled"), false, "disabled option getter"); el.droppable("destroy"); el.droppable({ disabled: true }); - TestHelpers.droppable.shouldNotDrop(); + droppableTestHelper.shouldNotDrop(); el.droppable("option", "disabled", false); equal(el.droppable("option", "disabled"), false, "disabled option setter"); - TestHelpers.droppable.shouldDrop(); + droppableTestHelper.shouldDrop(); expected = $("
").droppable(), actual = expected.droppable("enable"); @@ -69,23 +70,23 @@ test( "disable", function() { var actual, expected, element = $( "#droppable1" ).droppable({ disabled: false }); - TestHelpers.droppable.shouldDrop(); + droppableTestHelper.shouldDrop(); element.droppable( "disable" ); - TestHelpers.droppable.shouldNotDrop(); + droppableTestHelper.shouldNotDrop(); equal( element.droppable( "option", "disabled" ), true, "disabled option getter" ); element.droppable( "destroy" ); element.droppable({ disabled: false }); - TestHelpers.droppable.shouldDrop(); + droppableTestHelper.shouldDrop(); element.droppable( "option", "disabled", true ); ok( !element.droppable( "widget" ).hasClass( "ui-state-disabled" ), "element does not get ui-state-disabled" ); ok( !element.droppable( "widget" ).attr( "aria-disabled" ), "element does not get aria-disabled" ); ok( element.droppable( "widget" ).hasClass( "ui-droppable-disabled" ), "element gets ui-droppable-disabled" ); equal( element.droppable( "option", "disabled" ), true, "disabled option setter" ); - TestHelpers.droppable.shouldNotDrop(); + droppableTestHelper.shouldNotDrop(); expected = $( "
" ).droppable(); actual = expected.droppable( "disable" ); equal( actual, expected, "disable is chainable" ); }); -})( jQuery ); +}); diff --git a/tests/unit/droppable/droppable_options.js b/tests/unit/droppable/droppable_options.js index 047858abf11..84778d3a60c 100644 --- a/tests/unit/droppable/droppable_options.js +++ b/tests/unit/droppable/droppable_options.js @@ -1,7 +1,11 @@ +define([ + "jquery", + "ui/droppable" +], function( $ ) { + /* * droppable_options.js */ -(function($) { module( "droppable: options" ); @@ -201,4 +205,4 @@ test( "tolerance, touch", function() { ok(false, 'missing test - untested code is broken code'); }); */ -})(jQuery); +}); diff --git a/tests/unit/droppable/droppable_test_helpers.js b/tests/unit/droppable/droppable_test_helpers.js index ffd37452999..627afcf8a89 100644 --- a/tests/unit/droppable/droppable_test_helpers.js +++ b/tests/unit/droppable/droppable_test_helpers.js @@ -1,4 +1,6 @@ -TestHelpers.droppable = { +define(function() { + +return { shouldDrop: function() { // todo: actually implement this ok(true, "missing test - untested code is broken code"); @@ -7,4 +9,6 @@ TestHelpers.droppable = { // todo: actually implement this ok(true, "missing test - untested code is broken code"); } -}; \ No newline at end of file +}; + +}); diff --git a/tests/unit/effects/effects.html b/tests/unit/effects/effects.html index d6cfdb7973b..a07be49ed8b 100644 --- a/tests/unit/effects/effects.html +++ b/tests/unit/effects/effects.html @@ -4,38 +4,12 @@ jQuery UI Effects Test Suite - - - - - + - - - - + #qunit-fixture { width: 1000px; @@ -111,5 +85,21 @@

Slide with relative width

+ + + + + + + + diff --git a/tests/unit/effects/effects_core.js b/tests/unit/effects/effects_core.js index 05db494e6af..381a8a73beb 100644 --- a/tests/unit/effects/effects_core.js +++ b/tests/unit/effects/effects_core.js @@ -1,4 +1,23 @@ -(function($) { +define([ + "jquery", + "helper/testsuite", + "ui/effect-blind", + "ui/effect-bounce", + "ui/effect-clip", + "ui/effect-drop", + "ui/effect-explode", + "ui/effect-fade", + "ui/effect-fold", + "ui/effect-highlight", + "ui/effect", + "ui/effect-puff", + "ui/effect-pulsate", + "ui/effect-scale", + "ui/effect-shake", + "ui/effect-size", + "ui/effect-slide", + "ui/effect-transfer" +], function( $, testHelper ) { function present( value, array, message ) { QUnit.push( jQuery.inArray( value, array ) !== -1 , value, array, message ); @@ -217,7 +236,7 @@ asyncTest( "animateClass: css and class changes during animation are not lost (# $.each( $.effects.effect, function( effect ) { module( "effects." + effect ); - TestHelpers.testJshint( "effect-" + effect ); + testHelper.testJshint( "effect-" + effect ); if ( effect === "transfer" ) { return; @@ -268,4 +287,4 @@ $.each( $.effects.effect, function( effect ) { }); }); -})(jQuery); +}); diff --git a/tests/unit/effects/effects_scale.js b/tests/unit/effects/effects_scale.js index 6abbcb5382f..c30f043847a 100644 --- a/tests/unit/effects/effects_scale.js +++ b/tests/unit/effects/effects_scale.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "ui/effect-scale" +], function( $ ) { + module( "effect.scale: Scale" ); function run( position, v, h, vo, ho ) { @@ -65,4 +69,4 @@ $(function() { } }); -})( jQuery ); +}); diff --git a/tests/unit/helper/css.js b/tests/unit/helper/css.js new file mode 100644 index 00000000000..523e5566ef2 --- /dev/null +++ b/tests/unit/helper/css.js @@ -0,0 +1,18 @@ +(function() { + +if( !window.helper ) { + window.helper = {}; +} + +function includeStyle( url ) { + document.write( "" ); +} + +window.helper.loadCss = function( styles ) { + var i; + for( i = 0; i < styles.length; i++ ) { + includeStyle( "themes/base/" + styles[ i ] + ".css" ); + } +}; + +}()); diff --git a/tests/unit/helper/jquery.js b/tests/unit/helper/jquery.js new file mode 100644 index 00000000000..d98f7f96759 --- /dev/null +++ b/tests/unit/helper/jquery.js @@ -0,0 +1,30 @@ +(function() { + +if( !window.helper ) { + window.helper = {}; +} + +window.helper.jqueryUrl = function() { + + var current, length, url, version, + i = 0, + parts = document.location.search.slice( 1 ).split( "&" ); + + for ( length = parts.length ; i < length; i++ ) { + current = parts[ i ].split( "=" ); + if ( current[ 0 ] === "jquery" ) { + version = current[ 1 ]; + break; + } + } + + if ( version === "git" ) { + url = "http://code.jquery.com/jquery-git.js"; + } else { + url = "../../../external/jquery-" + ( version || "1.10.2" ) + "/jquery"; + } + + return url; +}; + +}()); diff --git a/tests/unit/helper/require.js b/tests/unit/helper/require.js new file mode 100644 index 00000000000..c299db81a88 --- /dev/null +++ b/tests/unit/helper/require.js @@ -0,0 +1,28 @@ + +// Common require.config. +require.config({ + paths: { + "helper": "../helper", + "jquery": window.helper.jqueryUrl(), + "jquery.simulate": "../../../external/jquery-simulate/jquery.simulate", + "jshint": "../../../external/jshint/jshint", + "ui": "../../../ui" + }, + shim: { + "jquery.simulate": [ "jquery" ] + } +}); + +function require_series( dependencies, callback ) { + var i = 0; + function next() { + if ( i >= dependencies.length ) { + return callback(); + } + require([ dependencies[ i ] ], function() { + i++; + next(); + }); + } + next(); +} diff --git a/tests/unit/testsuite.js b/tests/unit/helper/testsuite.js similarity index 71% rename from tests/unit/testsuite.js rename to tests/unit/helper/testsuite.js index 18353c702df..aa5877ffeca 100644 --- a/tests/unit/testsuite.js +++ b/tests/unit/helper/testsuite.js @@ -1,129 +1,10 @@ -(function( $ ) { +define([ + "jquery", + "ui/core" +], function( $ ) { -var reset, jshintLoaded; - -window.TestHelpers = {}; - -function includeStyle( url ) { - document.write( "" ); -} - -function includeScript( url ) { - document.write( "" ); -} - -function url( value ) { - return value + ( /\?/.test( value ) ? "&" : "?" ) + new Date().getTime() + "" + - parseInt( Math.random() * 100000, 10 ); -} - -reset = QUnit.reset; -QUnit.reset = function() { - - // Ensure jQuery events and data on the fixture are properly removed - jQuery( "#qunit-fixture" ).empty(); - - // Let QUnit reset the fixture - reset.apply( this, arguments ); -}; - - -QUnit.config.requireExpects = true; - -/* -// TODO: Add back the ability to test against minified files -// see QUnit.urlParams.min usage below -QUnit.config.urlConfig.push({ - id: "min", - label: "Minified source", - tooltip: "Load minified source files instead of the regular unminified ones." -}); -*/ - -TestHelpers.loadResources = QUnit.urlParams.min ? - function() { - includeStyle( "dist/jquery-ui.min.css" ); - includeScript( "dist/jquery-ui.min.js" ); - } : - function( resources ) { - $.each( resources.css || [], function( i, resource ) { - includeStyle( "themes/base/" + resource + ".css" ); - }); - $.each( resources.js || [], function( i, resource ) { - includeScript( resource ); - }); - }; - -QUnit.config.urlConfig.push({ - id: "nojshint", - label: "Skip JSHint", - tooltip: "Skip running JSHint, e.g., within TestSwarm, where Jenkins runs it already" -}); - -QUnit.config.urlConfig.push({ - id: "jquery", - label: "jQuery version", - value: [ - "1.6", "1.6.1", "1.6.2", "1.6.3", "1.6.4", "1.7", "1.7.1", "1.7.2", - "1.8.0", "1.8.1", "1.8.2", "1.8.3", "1.9.0", "1.9.1", "1.10.0", - "1.10.1", "1.10.2", "2.0.0", "2.0.1", "2.0.2", "2.0.3", "git" - ], - tooltip: "Which jQuery Core version to test against" -}); - -jshintLoaded = false; -TestHelpers.testJshint = function( module ) { - - // Function.prototype.bind check is needed because JSHint doesn't work in ES3 browsers anymore - // https://github.com/jshint/jshint/issues/1384 - if ( QUnit.urlParams.nojshint || !Function.prototype.bind ) { - return; - } - - if ( !jshintLoaded ) { - includeScript( "external/jshint/jshint.js" ); - jshintLoaded = true; - } - - asyncTest( "JSHint", function() { - expect( 1 ); - - $.when( - $.ajax({ - url: url( "../../../ui/.jshintrc" ), - dataType: "json" - }), - $.ajax({ - url: url( "../../../ui/" + module + ".js" ), - dataType: "text" - }) - ).done(function( hintArgs, srcArgs ) { - var globals, passed, errors, - jshintrc = hintArgs[ 0 ], - source = srcArgs[ 0 ]; - - globals = jshintrc.globals || {}; - delete jshintrc.globals; - passed = JSHINT( source, jshintrc, globals ); - errors = $.map( JSHINT.errors, function( error ) { - - // JSHINT may report null if there are too many errors - if ( !error ) { - return; - } - - return "[L" + error.line + ":C" + error.character + "] " + - error.reason + "\n" + error.evidence + "\n"; - }).join( "\n" ); - ok( passed, errors ); - start(); - }) - .fail(function() { - ok( false, "error loading source" ); - start(); - }); - }); -}; +var reset, jshintLoaded, + exports = {}; function testWidgetDefaults( widget, defaults ) { var pluginDefaults = $.ui[ widget ].prototype.options; @@ -185,54 +66,69 @@ function testBasicUsage( widget ) { }); } -TestHelpers.commonWidgetTests = function( widget, settings ) { - module( widget + ": common widget" ); +// Asset revisioning through time and random hashing +function revStamp( value ) { + return value + (/\?/.test(value) ? "&" : "?") + new Date().getTime() + "" + + parseInt(Math.random() * 100000, 10); +} - TestHelpers.testJshint( widget ); - testWidgetDefaults( widget, settings.defaults ); - testWidgetOverrides( widget ); - testBasicUsage( widget ); - test( "version", function() { - expect( 1 ); - ok( "version" in $.ui[ widget ].prototype, "version property exists" ); - }); +reset = QUnit.reset; +QUnit.reset = function() { + // Ensure jQuery events and data on the fixture are properly removed + jQuery( "#qunit-fixture" ).empty(); + // Let QUnit reset the fixture + reset.apply( this, arguments ); }; -TestHelpers.onFocus = function( element, onFocus ) { - var fn = function( event ) { - if ( !event.originalEvent ) { - return; - } - element.unbind( "focus", fn ); - onFocus(); - }; +QUnit.config.requireExpects = true; - element.bind( "focus", fn )[ 0 ].focus(); -}; +QUnit.config.urlConfig.push({ + id: "nojshint", + label: "Skip JSHint", + tooltip: "Skip running JSHint, e.g., within TestSwarm, where Jenkins runs it already" +}); -TestHelpers.forceScrollableWindow = function( appendTo ) { +QUnit.config.urlConfig.push({ + id: "jquery", + label: "jQuery version", + value: [ + "1.6", "1.6.1", "1.6.2", "1.6.3", "1.6.4", "1.7", "1.7.1", "1.7.2", + "1.8.0", "1.8.1", "1.8.2", "1.8.3", "1.9.0", "1.9.1", "1.10.0", + "1.10.1", "1.10.2", "2.0.0", "2.0.1", "2.0.2", "2.0.3", "git" + ], + tooltip: "Which jQuery Core version to test against" +}); - // The main testable area is 10000x10000 so to enforce scrolling, - // this DIV must be greater than 10000 to work - return $( "
" ).css({ - height: "11000px", - width: "11000px" - }).appendTo( appendTo || "#qunit-fixture" ); -}; +/** + * Exports + */ // Taken from https://github.com/jquery/qunit/tree/master/addons/close-enough -window.closeEnough = function( actual, expected, maxDifference, message ) { +exports.closeEnough = function( actual, expected, maxDifference, message ) { var passes = ( actual === expected ) || Math.abs( actual - expected ) <= maxDifference; QUnit.push( passes, actual, expected, message ); }; -/* +exports.commonWidgetTests = function( widget, settings ) { + module( widget + ": common widget" ); + + exports.testJshint( widget ); + testWidgetDefaults( widget, settings.defaults ); + testWidgetOverrides( widget ); + testBasicUsage( widget ); + test( "version", function() { + expect( 1 ); + ok( "version" in $.ui[ widget ].prototype, "version property exists" ); + }); +}; + +/** * Experimental assertion for comparing DOM objects. * * Serializes an element and some properties and attributes and its children if any, * otherwise the text. Then compares the result using deepEqual(). */ -window.domEqual = function( selector, modifier, message ) { +exports.domEqual = function( selector, modifier, message ) { var expected, actual, properties = [ "disabled", @@ -277,7 +173,6 @@ window.domEqual = function( selector, modifier, message ) { styles[ $.camelCase( key ) ] = style[ key ]; } } - // support: Opera, IE <9 } else { for ( key in style ) { @@ -339,4 +234,76 @@ window.domEqual = function( selector, modifier, message ) { } }; -}( jQuery )); +exports.forceScrollableWindow = function( appendTo ) { + + // The main testable area is 10000x10000 so to enforce scrolling, + // this DIV must be greater than 10000 to work + return $( "
" ).css({ + height: "11000px", + width: "11000px" + }).appendTo( appendTo || "#qunit-fixture" ); +}; + +exports.onFocus = function( element, onFocus ) { + var fn = function( event ){ + if( !event.originalEvent ) { + return; + } + element.unbind( "focus", fn ); + onFocus(); + }; + + element.bind( "focus", fn )[ 0 ].focus(); +}; + +exports.testJshint = function( module ) { + // Function.prototype.bind check is needed because JSHint doesn't work in ES3 browsers anymore + // https://github.com/jshint/jshint/issues/1384 + if ( QUnit.urlParams.nojshint || !Function.prototype.bind ) { + return; + } + + asyncTest( "JSHint", function() { + require([ "jshint" ], function() { + expect( 1 ); + + $.when( + $.ajax({ + url: revStamp( "../../../ui/.jshintrc" ), + dataType: "json" + }), + $.ajax({ + url: revStamp( "../../../ui/" + module + ".js" ), + dataType: "text" + }) + ).done(function( hintArgs, srcArgs ) { + var globals, passed, errors, + jshintrc = hintArgs[ 0 ], + source = srcArgs[ 0 ]; + + globals = jshintrc.globals || {}; + delete jshintrc.globals; + passed = JSHINT( source, jshintrc, globals ); + errors = $.map( JSHINT.errors, function( error ) { + // JSHINT may report null if there are too many errors + if ( !error ) { + return; + } + + return "[L" + error.line + ":C" + error.character + "] " + + error.reason + "\n" + error.evidence + "\n"; + }).join( "\n" ); + ok( passed, errors ); + start(); + }) + .fail(function( hintError, srcError ) { + ok( false, "error loading source: " + ( hintError || srcError ).statusText ); + start(); + }); + }); + }); +}; + +return exports; + +}); diff --git a/tests/unit/menu/menu.html b/tests/unit/menu/menu.html index 2aa286c69fb..93311fd8909 100644 --- a/tests/unit/menu/menu.html +++ b/tests/unit/menu/menu.html @@ -4,30 +4,12 @@ jQuery UI Menu Test Suite - - - - - + - - - - - - - - + .ui-menu { font-size: 15px; @@ -316,5 +298,25 @@
+ + + + + + + + diff --git a/tests/unit/menu/menu_common.js b/tests/unit/menu/menu_common.js index 099dd091e03..7e3eb6d46ab 100644 --- a/tests/unit/menu/menu_common.js +++ b/tests/unit/menu/menu_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "menu", { +define([ + "helper/testsuite", + "ui/menu" +], function( testHelper ) { + +testHelper.commonWidgetTests( "menu", { defaults: { disabled: false, icons: { @@ -19,3 +24,5 @@ TestHelpers.commonWidgetTests( "menu", { select: null } }); + +}); diff --git a/tests/unit/menu/menu_core.js b/tests/unit/menu/menu_core.js index df039c9e49d..1b9ae1ddcce 100644 --- a/tests/unit/menu/menu_core.js +++ b/tests/unit/menu/menu_core.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "./menu_test_helpers", + "ui/menu" +], function( $, menuTestHelper ) { module( "menu: core" ); @@ -37,7 +41,7 @@ asyncTest( "#9044: Autofocus issue with dialog opened from menu widget", functio $( "#test9044" ).focus(); }); - TestHelpers.menu.click( element, "3" ); + menuTestHelper.click( element, "3" ); setTimeout( function() { equal( document.activeElement.id, "test9044", "Focus was swallowed by menu" ); $( "#test9044" ).remove(); @@ -58,4 +62,4 @@ asyncTest( "#9532: Need a way in Menu to keep ui-state-active class on selected }, 500 ); }); -})( jQuery ); +}); diff --git a/tests/unit/menu/menu_events.js b/tests/unit/menu/menu_events.js index 0a02cc01b5f..8ea5c56983a 100644 --- a/tests/unit/menu/menu_events.js +++ b/tests/unit/menu/menu_events.js @@ -1,12 +1,17 @@ -(function( $ ) { +define([ + "jquery", + "./menu_test_helpers", + "jquery.simulate", + "ui/menu" +], function( $, menuTestHelper ) { -var log = TestHelpers.menu.log, - logOutput = TestHelpers.menu.logOutput, - click = TestHelpers.menu.click; +var log = menuTestHelper.log, + logOutput = menuTestHelper.logOutput, + click = menuTestHelper.click; module( "menu: events", { setup: function() { - TestHelpers.menu.clearLog(); + menuTestHelper.clearLog(); } }); @@ -661,4 +666,4 @@ asyncTest( "#10571: When typing in a menu, only menu-items should be focused", f }); }); -})( jQuery ); +}); diff --git a/tests/unit/menu/menu_methods.js b/tests/unit/menu/menu_methods.js index 5a633998d82..11d64ea0e3a 100644 --- a/tests/unit/menu/menu_methods.js +++ b/tests/unit/menu/menu_methods.js @@ -1,27 +1,32 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "./menu_test_helpers", + "ui/menu" +], function( $, testHelper, menuTestHelper ) { -var log = TestHelpers.menu.log, - logOutput = TestHelpers.menu.logOutput, - click = TestHelpers.menu.click; +var log = menuTestHelper.log, + logOutput = menuTestHelper.logOutput, + click = menuTestHelper.click; module( "menu: methods", { setup: function() { - TestHelpers.menu.clearLog(); + menuTestHelper.clearLog(); } }); test( "destroy", function() { expect( 4 ); - domEqual( "#menu1", function() { + testHelper.domEqual( "#menu1", function() { $( "#menu1" ).menu().menu( "destroy" ); }); - domEqual( "#menu2", function() { + testHelper.domEqual( "#menu2", function() { $( "#menu2" ).menu().menu( "destroy" ); }); - domEqual( "#menu5", function() { + testHelper.domEqual( "#menu5", function() { $( "#menu5").menu().menu( "destroy" ); }); - domEqual( "#menu6", function() { + testHelper.domEqual( "#menu6", function() { $( "#menu6" ).menu().menu( "destroy" ); }); }); @@ -114,4 +119,4 @@ test( "widget", function() { // TODO: test select method -})( jQuery ); +}); diff --git a/tests/unit/menu/menu_options.js b/tests/unit/menu/menu_options.js index c7667334583..d5aa625d9cb 100644 --- a/tests/unit/menu/menu_options.js +++ b/tests/unit/menu/menu_options.js @@ -1,12 +1,16 @@ -(function( $ ) { +define([ + "jquery", + "./menu_test_helpers", + "ui/menu" +], function( $, menuTestHelper ) { -var log = TestHelpers.menu.log, - logOutput = TestHelpers.menu.logOutput, - click = TestHelpers.menu.click; +var log = menuTestHelper.log, + logOutput = menuTestHelper.logOutput, + click = menuTestHelper.click; module( "menu: options", { setup: function() { - TestHelpers.menu.clearLog(); + menuTestHelper.clearLog(); } }); @@ -106,4 +110,4 @@ test( "{ role: null }", function() { }); }); -})( jQuery ); +}); diff --git a/tests/unit/menu/menu_test_helpers.js b/tests/unit/menu/menu_test_helpers.js index 77737c39045..1841dd086d2 100644 --- a/tests/unit/menu/menu_test_helpers.js +++ b/tests/unit/menu/menu_test_helpers.js @@ -1,9 +1,9 @@ -(function() { +define(function() { var lastItem, log = []; -TestHelpers.menu = { +return { log: function( message, clear ) { if ( clear ) { log.length = 0; @@ -28,4 +28,4 @@ TestHelpers.menu = { } }; -})(); +}); diff --git a/tests/unit/position/position.html b/tests/unit/position/position.html index 0cad7d322a7..979d8e128bc 100644 --- a/tests/unit/position/position.html +++ b/tests/unit/position/position.html @@ -4,20 +4,7 @@ jQuery UI Position Test Suite - - - - - - - - -
@@ -49,5 +36,19 @@
+ + + + + + + diff --git a/tests/unit/position/position_core.js b/tests/unit/position/position_core.js index 5b1872af983..a7567fd3f9c 100644 --- a/tests/unit/position/position_core.js +++ b/tests/unit/position/position_core.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "ui/position" +], function( $, testHelper ) { var win = $( window ), scrollTopSupport = function() { @@ -16,7 +20,7 @@ module( "position", { } }); -TestHelpers.testJshint( "position" ); +testHelper.testJshint( "position" ); test( "my, at, of", function() { expect( 4 ); @@ -726,4 +730,4 @@ test( "bug #5280: consistent results (avoid fractional values)", function() { deepEqual( offset1, offset2 ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/progressbar/progressbar.html b/tests/unit/progressbar/progressbar.html index 56a02bcd4d6..f7330473f27 100644 --- a/tests/unit/progressbar/progressbar.html +++ b/tests/unit/progressbar/progressbar.html @@ -4,29 +4,12 @@ jQuery UI Progressbar Test Suite - - - - - + - - - - - - - + @@ -36,5 +19,24 @@
+ + + + + + + + diff --git a/tests/unit/progressbar/progressbar_common.js b/tests/unit/progressbar/progressbar_common.js index 0768576f568..d624d1ce7e8 100644 --- a/tests/unit/progressbar/progressbar_common.js +++ b/tests/unit/progressbar/progressbar_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "progressbar", { +define([ + "helper/testsuite", + "ui/progressbar" +], function( testHelper ) { + +testHelper.commonWidgetTests( "progressbar", { defaults: { disabled: false, max: 100, @@ -10,3 +15,5 @@ TestHelpers.commonWidgetTests( "progressbar", { create: null } }); + +}); diff --git a/tests/unit/progressbar/progressbar_core.js b/tests/unit/progressbar/progressbar_core.js index 8f4a138ce9c..4ce80d8a6ef 100644 --- a/tests/unit/progressbar/progressbar_core.js +++ b/tests/unit/progressbar/progressbar_core.js @@ -1,3 +1,8 @@ +define([ + "jquery", + "ui/progressbar" +], function( $ ) { + module( "progressbar: core" ); test( "markup structure", function() { @@ -51,3 +56,5 @@ test( "accessibility", function() { equal( element.attr( "aria-valuemax" ), 150, "aria-valuemax" ); strictEqual( element.attr( "aria-valuenow" ), undefined, "aria-valuenow" ); }); + +}); diff --git a/tests/unit/progressbar/progressbar_events.js b/tests/unit/progressbar/progressbar_events.js index 95effda7511..4552fc3fb9a 100644 --- a/tests/unit/progressbar/progressbar_events.js +++ b/tests/unit/progressbar/progressbar_events.js @@ -1,3 +1,8 @@ +define([ + "jquery", + "ui/progressbar" +], function( $ ) { + module( "progressbar: events" ); test( "create", function() { @@ -49,3 +54,5 @@ test( "complete", function() { value = 100; element.progressbar( "value", value ); }); + +}); diff --git a/tests/unit/progressbar/progressbar_methods.js b/tests/unit/progressbar/progressbar_methods.js index 004eacf90d2..b7a80d4d472 100644 --- a/tests/unit/progressbar/progressbar_methods.js +++ b/tests/unit/progressbar/progressbar_methods.js @@ -1,8 +1,14 @@ +define([ + "jquery", + "helper/testsuite", + "ui/progressbar" +], function( $, testHelper ) { + module( "progressbar: methods" ); test( "destroy", function() { expect( 1 ); - domEqual( "#progressbar", function() { + testHelper.domEqual( "#progressbar", function() { $( "#progressbar" ).progressbar().progressbar( "destroy" ); }); }); @@ -33,3 +39,5 @@ test( "widget", function() { equal( widgetElement.length, 1, "one element" ); strictEqual( widgetElement[ 0 ], element[ 0 ], "same element" ); }); + +}); diff --git a/tests/unit/progressbar/progressbar_options.js b/tests/unit/progressbar/progressbar_options.js index bc0b5d00d62..f97865b1603 100644 --- a/tests/unit/progressbar/progressbar_options.js +++ b/tests/unit/progressbar/progressbar_options.js @@ -1,3 +1,8 @@ +define([ + "jquery", + "ui/progressbar" +], function( $ ) { + module( "progressbar: options" ); test( "{ value: 0 }, default", function() { @@ -70,3 +75,5 @@ test( "change max below value", function() { equal( $( "#progressbar" ).progressbar( "value" ), 5, "value constrained at max" ); }); + +}); diff --git a/tests/unit/resizable/resizable.html b/tests/unit/resizable/resizable.html index 5668c909a22..6d3b1332d40 100644 --- a/tests/unit/resizable/resizable.html +++ b/tests/unit/resizable/resizable.html @@ -4,31 +4,12 @@ jQuery UI Resizable Test Suite - - - - - + - - - - - - - - + #container { @@ -79,5 +60,24 @@ solid gray + + + + + + + + diff --git a/tests/unit/resizable/resizable_common.js b/tests/unit/resizable/resizable_common.js index c261ac5b9c8..a85ae421d65 100644 --- a/tests/unit/resizable/resizable_common.js +++ b/tests/unit/resizable/resizable_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "resizable", { +define([ + "helper/testsuite", + "ui/resizable" +], function( testHelper ) { + +testHelper.commonWidgetTests( "resizable", { defaults: { alsoResize: false, animate: false, @@ -28,3 +33,5 @@ TestHelpers.commonWidgetTests( "resizable", { stop: null } }); + +}); diff --git a/tests/unit/resizable/resizable_core.js b/tests/unit/resizable/resizable_core.js index f43cde6237c..a2c5a2980f6 100644 --- a/tests/unit/resizable/resizable_core.js +++ b/tests/unit/resizable/resizable_core.js @@ -1,9 +1,13 @@ +define([ + "jquery", + "./resizable_test_helpers", + "ui/resizable" +], function( $, resizableTestHelper ) { + /* * resizable_core.js */ -(function($) { - module("resizable: core"); /* @@ -30,10 +34,10 @@ test("n", function() { var handle = ".ui-resizable-n", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, 0, -50); + resizableTestHelper.drag(handle, 0, -50); equal( target.height(), 150, "compare height" ); - TestHelpers.resizable.drag(handle, 0, 50); + resizableTestHelper.drag(handle, 0, 50); equal( target.height(), 100, "compare height" ); equal( target[0].style.left, "", "left should not be modified" ); @@ -45,10 +49,10 @@ test("s", function() { var handle = ".ui-resizable-s", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, 0, 50); + resizableTestHelper.drag(handle, 0, 50); equal( target.height(), 150, "compare height" ); - TestHelpers.resizable.drag(handle, 0, -50); + resizableTestHelper.drag(handle, 0, -50); equal( target.height(), 100, "compare height" ); equal( target[0].style.top, "", "top should not be modified" ); @@ -61,10 +65,10 @@ test("e", function() { var handle = ".ui-resizable-e", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, 50); + resizableTestHelper.drag(handle, 50); equal( target.width(), 150, "compare width"); - TestHelpers.resizable.drag(handle, -50); + resizableTestHelper.drag(handle, -50); equal( target.width(), 100, "compare width" ); equal( target[0].style.height, "", "height should not be modified" ); @@ -77,10 +81,10 @@ test("w", function() { var handle = ".ui-resizable-w", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, -50); + resizableTestHelper.drag(handle, -50); equal( target.width(), 150, "compare width" ); - TestHelpers.resizable.drag(handle, 50); + resizableTestHelper.drag(handle, 50); equal( target.width(), 100, "compare width" ); equal( target[0].style.height, "", "height should not be modified" ); @@ -92,11 +96,11 @@ test("ne", function() { var handle = ".ui-resizable-ne", target = $("#resizable1").css({ overflow: "hidden" }).resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, -50, -50); + resizableTestHelper.drag(handle, -50, -50); equal( target.width(), 50, "compare width" ); equal( target.height(), 150, "compare height" ); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal( target.width(), 100, "compare width" ); equal( target.height(), 100, "compare height" ); @@ -108,11 +112,11 @@ test("se", function() { var handle = ".ui-resizable-se", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal( target.width(), 150, "compare width" ); equal( target.height(), 150, "compare height" ); - TestHelpers.resizable.drag(handle, -50, -50); + resizableTestHelper.drag(handle, -50, -50); equal( target.width(), 100, "compare width" ); equal( target.height(), 100, "compare height" ); @@ -125,11 +129,11 @@ test("sw", function() { var handle = ".ui-resizable-sw", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, -50, -50); + resizableTestHelper.drag(handle, -50, -50); equal( target.width(), 150, "compare width" ); equal( target.height(), 50, "compare height" ); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal( target.width(), 100, "compare width" ); equal( target.height(), 100, "compare height" ); @@ -141,11 +145,11 @@ test("nw", function() { var handle = ".ui-resizable-nw", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, -50, -50); + resizableTestHelper.drag(handle, -50, -50); equal( target.width(), 150, "compare width" ); equal( target.height(), 150, "compare height" ); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal( target.width(), 100, "compare width" ); equal( target.height(), 100, "compare height" ); }); @@ -163,10 +167,10 @@ test("handle with complex markup (#8756)", function() { var handle = ".ui-resizable-w div", target = $("#resizable1").resizable({ handles: "all" }); - TestHelpers.resizable.drag(handle, -50); + resizableTestHelper.drag(handle, -50); equal( target.width(), 150, "compare width" ); - TestHelpers.resizable.drag(handle, 50); + resizableTestHelper.drag(handle, 50); equal( target.width(), 100, "compare width" ); }); @@ -186,7 +190,7 @@ test("resizable accounts for scroll position correctly (#3815)", function() { left = el.css("left"); top = el.css("top"); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); deepEqual( el.position(), position, "position stays the same when resized" ); equal( el.css("left"), left, "css('left') stays the same when resized" ); equal( el.css("top"), top, "css('top') stays the same when resized" ); @@ -203,7 +207,7 @@ test( "resizable stores correct size when using helper and grid (#9547)", functi grid: [ 10, 10 ] }); - TestHelpers.resizable.drag( handle, 1, 1 ); + resizableTestHelper.drag( handle, 1, 1 ); equal( target.width(), 100, "compare width" ); equal( target.height(), 100, "compare height" ); }); @@ -227,18 +231,18 @@ test( "nested resizable", function() { innerHandle = $( "#inner > .ui-resizable-e" ); outerHandle = $( "#outer > .ui-resizable-e" ); - TestHelpers.resizable.drag( innerHandle, 10 ); + resizableTestHelper.drag( innerHandle, 10 ); equal( inner.width(), 40, "compare width of inner element" ); - TestHelpers.resizable.drag( innerHandle, -10 ); + resizableTestHelper.drag( innerHandle, -10 ); equal( inner.width(), 30, "compare width of inner element" ); - TestHelpers.resizable.drag( outerHandle, 10 ); + resizableTestHelper.drag( outerHandle, 10 ); equal( outer.width(), 60, "compare width of outer element" ); - TestHelpers.resizable.drag( outerHandle, -10 ); + resizableTestHelper.drag( outerHandle, -10 ); equal( outer.width(), 50, "compare width of outer element" ); inner.remove(); outer.remove(); }); -})(jQuery); +}); diff --git a/tests/unit/resizable/resizable_events.js b/tests/unit/resizable/resizable_events.js index d3851f72ed6..81a7b87e412 100644 --- a/tests/unit/resizable/resizable_events.js +++ b/tests/unit/resizable/resizable_events.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "./resizable_test_helpers", + "ui/resizable" +], function( $, resizableTestHelper ) { + /* * resizable_events.js */ -(function($) { module("resizable: events"); @@ -23,7 +28,7 @@ test("start", function() { } }); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal(count, 1, "start callback should happen exactly once"); @@ -54,7 +59,7 @@ test( "resize", function() { } }); - TestHelpers.resizable.drag( handle, 50, 50 ); + resizableTestHelper.drag( handle, 50, 50 ); equal( count, 2, "resize callback should happen exactly once per size adjustment" ); @@ -82,7 +87,7 @@ test( "resize (min/max dimensions)", function() { } }); - TestHelpers.resizable.drag( handle, -200, -200 ); + resizableTestHelper.drag( handle, -200, -200 ); equal( count, 1, "resize callback should happen exactly once per size adjustment" ); @@ -112,10 +117,10 @@ test( "resize (containment)", function() { }); // Prove you can't resize outside containment by dragging southeast corner southeast - TestHelpers.resizable.drag( handle, 100, 100 ); + resizableTestHelper.drag( handle, 100, 100 ); // Prove you can't resize outside containment by dragging southeast corner northwest - TestHelpers.resizable.drag( handle, -200, -200 ); + resizableTestHelper.drag( handle, -200, -200 ); equal( count, 1, "resize callback should happen exactly once per size adjustment" ); @@ -140,7 +145,7 @@ test("resize (grid)", function() { } }); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal(count, 1, "resize callback should happen exactly once per grid-unit size adjustment"); @@ -159,7 +164,7 @@ test( "resize, custom adjustment", function() { } }); - TestHelpers.resizable.drag( handle, 50, 50 ); + resizableTestHelper.drag( handle, 50, 50 ); equal( element.width(), 100, "resize event can control width" ); equal( element.height(), 200, "resize event can control height" ); @@ -185,7 +190,7 @@ test("stop", function() { } }); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal(count, 1, "stop callback should happen exactly once"); @@ -230,7 +235,7 @@ test( "resize (containment) works with parent with negative offset", function() widthBefore = target.width(); - TestHelpers.resizable.drag( handle, increaseWidthBy, 0 ); + resizableTestHelper.drag( handle, increaseWidthBy, 0 ); widthAfter = target.width(); @@ -238,4 +243,4 @@ test( "resize (containment) works with parent with negative offset", function() }); -})(jQuery); +}); diff --git a/tests/unit/resizable/resizable_methods.js b/tests/unit/resizable/resizable_methods.js index abb8951f930..a5a92d56081 100644 --- a/tests/unit/resizable/resizable_methods.js +++ b/tests/unit/resizable/resizable_methods.js @@ -1,7 +1,11 @@ +define([ + "jquery", + "ui/resizable" +], function( $ ) { + /* * resizable_methods.js */ -(function($) { module( "resizable: methods" ); @@ -18,4 +22,4 @@ test( "disable", function() { equal( chainable, element, "disable is chainable" ); }); -})(jQuery); +}); diff --git a/tests/unit/resizable/resizable_options.js b/tests/unit/resizable/resizable_options.js index c46801bc54d..422f3003f14 100644 --- a/tests/unit/resizable/resizable_options.js +++ b/tests/unit/resizable/resizable_options.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "./resizable_test_helpers", + "ui/resizable" +], function( $, resizableTestHelper ) { + /* * resizable_options.js */ -(function($) { module("resizable: options"); @@ -19,7 +24,7 @@ test( "alsoResize", function() { }), handle = ".ui-resizable-e"; - TestHelpers.resizable.drag( handle, 80 ); + resizableTestHelper.drag( handle, 80 ); equal( element.width(), 180, "resizable width" ); equal( other.width(), 130, "alsoResize width" ); }); @@ -30,11 +35,11 @@ test("aspectRatio: 'preserve' (e)", function() { var handle = ".ui-resizable-e", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, 80); + resizableTestHelper.drag(handle, 80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, -130); + resizableTestHelper.drag(handle, -130); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -44,11 +49,11 @@ test("aspectRatio: 'preserve' (w)", function() { var handle = ".ui-resizable-w", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, -80); + resizableTestHelper.drag(handle, -80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, 130); + resizableTestHelper.drag(handle, 130); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -58,11 +63,11 @@ test("aspectRatio: 'preserve' (n)", function() { var handle = ".ui-resizable-n", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, 0, -80); + resizableTestHelper.drag(handle, 0, -80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, 0, 80); + resizableTestHelper.drag(handle, 0, 80); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -72,11 +77,11 @@ test("aspectRatio: 'preserve' (s)", function() { var handle = ".ui-resizable-s", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, 0, 80); + resizableTestHelper.drag(handle, 0, 80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, 0, -80); + resizableTestHelper.drag(handle, 0, -80); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -86,11 +91,11 @@ test("aspectRatio: 'preserve' (se)", function() { var handle = ".ui-resizable-se", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, 80, 80); + resizableTestHelper.drag(handle, 80, 80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, -80, -80); + resizableTestHelper.drag(handle, -80, -80); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -100,11 +105,11 @@ test("aspectRatio: 'preserve' (sw)", function() { var handle = ".ui-resizable-sw", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, -80, 80); + resizableTestHelper.drag(handle, -80, 80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, 80, -80); + resizableTestHelper.drag(handle, 80, -80); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -114,11 +119,11 @@ test("aspectRatio: 'preserve' (ne)", function() { var handle = ".ui-resizable-ne", target = $("#resizable1").resizable({ aspectRatio: "preserve", handles: "all", minWidth: 70, minHeight: 50, maxWidth: 150, maxHeight: 130 }); - TestHelpers.resizable.drag(handle, 80, -80); + resizableTestHelper.drag(handle, 80, -80); equal( target.width(), 130, "compare maxWidth"); equal( target.height(), 130, "compare maxHeight"); - TestHelpers.resizable.drag(handle, -80, 80); + resizableTestHelper.drag(handle, -80, 80); equal( target.width(), 70, "compare minWidth"); equal( target.height(), 70, "compare minHeight"); }); @@ -138,7 +143,7 @@ test( "aspectRatio: Resizing can move objects", function() { $( "#container" ).css({ width: 200, height: 300 }); $( "#resizable1" ).css({ width: 100, height: 100, left: 75, top: 200 }); - TestHelpers.resizable.drag( handleW, -20 ); + resizableTestHelper.drag( handleW, -20 ); equal( target.width(), 100, "compare width - no size change" ); equal( target.height(), 100, "compare height - no size change" ); equal( target.position().left, 75, "compare left - no movement" ); @@ -147,7 +152,7 @@ test( "aspectRatio: Resizing can move objects", function() { $( "#container" ).css({ width: 200, height: 300, position: "absolute", left: 100, top: 100 }); $( "#resizable1" ).css({ width: 100, height: 100, left: 0, top: 0 }); - TestHelpers.resizable.drag( handleNW, -20, -20 ); + resizableTestHelper.drag( handleNW, -20, -20 ); equal( target.width(), 100, "compare width - no size change" ); equal( target.height(), 100, "compare height - no size change" ); equal( target.position().left, 0, "compare left - no movement" ); @@ -161,11 +166,11 @@ test( "containment", function() { containment: "#container" }); - TestHelpers.resizable.drag( ".ui-resizable-se", 20, 30 ); + resizableTestHelper.drag( ".ui-resizable-se", 20, 30 ); equal( element.width(), 120, "unconstrained width within container" ); equal( element.height(), 130, "unconstrained height within container" ); - TestHelpers.resizable.drag( ".ui-resizable-se", 400, 400 ); + resizableTestHelper.drag( ".ui-resizable-se", 400, 400 ); equal( element.width(), 300, "constrained width at containment edge" ); equal( element.height(), 200, "constrained height at containment edge" ); }); @@ -180,10 +185,10 @@ test( "containment - not immediate parent", function() { handles: "all" }); - TestHelpers.resizable.drag( ".ui-resizable-e", 300, 0 ); + resizableTestHelper.drag( ".ui-resizable-e", 300, 0 ); equal( element.width(), 400, "Relative, contained within container width" ); - TestHelpers.resizable.drag( ".ui-resizable-s", 0, 300 ); + resizableTestHelper.drag( ".ui-resizable-s", 0, 300 ); equal( element.height(), 400, "Relative, contained within container height" ); $( "#child" ).css( { left: 50, top: 50 } ); @@ -195,10 +200,10 @@ test( "containment - not immediate parent", function() { handles: "all" }); - TestHelpers.resizable.drag( ".ui-resizable-e", 400, 0 ); + resizableTestHelper.drag( ".ui-resizable-e", 400, 0 ); equal( element.width(), 300, "Relative with Left, contained within container width" ); - TestHelpers.resizable.drag( ".ui-resizable-s", 0, 400 ); + resizableTestHelper.drag( ".ui-resizable-s", 0, 400 ); equal( element.height(), 300, "Relative with Top, contained within container height" ); }); @@ -212,10 +217,10 @@ test( "containment - immediate parent", function() { handles: "all" }); - TestHelpers.resizable.drag( ".ui-resizable-e", 400, 0 ); + resizableTestHelper.drag( ".ui-resizable-e", 400, 0 ); equal( element.width(), 300, "Relative, contained within container width" ); - TestHelpers.resizable.drag( ".ui-resizable-s", 0, 400 ); + resizableTestHelper.drag( ".ui-resizable-s", 0, 400 ); equal( element.height(), 300, "Relative, contained within container height" ); $( "#child" ).css( { left: 50, top: 50 } ); @@ -227,10 +232,10 @@ test( "containment - immediate parent", function() { handles: "all" }); - TestHelpers.resizable.drag( ".ui-resizable-e", 400, 0 ); + resizableTestHelper.drag( ".ui-resizable-e", 400, 0 ); equal( element.width(), 250, "Relative with Left, contained within container width" ); - TestHelpers.resizable.drag( ".ui-resizable-s", 0, 400 ); + resizableTestHelper.drag( ".ui-resizable-s", 0, 400 ); equal( element.height(), 250, "Relative with Top, contained within container height" ); }); @@ -239,11 +244,11 @@ test("grid", function() { var handle = ".ui-resizable-se", target = $("#resizable1").resizable({ handles: "all", grid: [0, 20] }); - TestHelpers.resizable.drag(handle, 3, 9); + resizableTestHelper.drag(handle, 3, 9); equal( target.width(), 103, "compare width"); equal( target.height(), 100, "compare height"); - TestHelpers.resizable.drag(handle, 15, 11); + resizableTestHelper.drag(handle, 15, 11); equal( target.width(), 118, "compare width"); equal( target.height(), 120, "compare height"); }); @@ -253,11 +258,11 @@ test("grid (min/max dimensions)", function() { var handle = ".ui-resizable-se", target = $("#resizable1").resizable({ handles: "all", grid: 20, minWidth: 65, minHeight: 65, maxWidth: 135, maxHeight: 135 }); - TestHelpers.resizable.drag(handle, 50, 50); + resizableTestHelper.drag(handle, 50, 50); equal( target.width(), 120, "grid should respect maxWidth"); equal( target.height(), 120, "grid should respect maxHeight"); - TestHelpers.resizable.drag(handle, -100, -100); + resizableTestHelper.drag(handle, -100, -100); equal( target.width(), 80, "grid should respect minWidth"); equal( target.height(), 80, "grid should respect minHeight"); }); @@ -267,11 +272,11 @@ test("grid (wrapped)", function() { var handle = ".ui-resizable-se", target = $("#resizable2").resizable({ handles: "all", grid: [0, 20] }); - TestHelpers.resizable.drag(handle, 3, 9); + resizableTestHelper.drag(handle, 3, 9); equal( target.width(), 103, "compare width"); equal( target.height(), 100, "compare height"); - TestHelpers.resizable.drag(handle, 15, 11); + resizableTestHelper.drag(handle, 15, 11); equal( target.width(), 118, "compare width"); equal( target.height(), 120, "compare height"); }); @@ -286,13 +291,13 @@ test( "grid - Resizable: can be moved when grid option is set (#9611)", function grid: 50 }); - TestHelpers.resizable.drag( handle, 50, 50 ); + resizableTestHelper.drag( handle, 50, 50 ); equal( target.width(), 50, "compare width" ); equal( target.height(), 50, "compare height" ); oldPosition = target.position(); - TestHelpers.resizable.drag( handle, 50, 50 ); + resizableTestHelper.drag( handle, 50, 50 ); equal( target.width(), 50, "compare width" ); equal( target.height(), 50, "compare height" ); equal( target.position().top, oldPosition.top, "compare top" ); @@ -314,7 +319,7 @@ test( "grid - maintains grid with padding and border when approaching no dimensi grid: 50 }); - TestHelpers.resizable.drag( handle, 50, 50 ); + resizableTestHelper.drag( handle, 50, 50 ); equal( target.outerWidth(), 50, "compare width" ); equal( target.outerHeight(), 50, "compare height" ); }); @@ -324,11 +329,11 @@ test("ui-resizable-se { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1 var handle = ".ui-resizable-se", target = $("#resizable1").resizable({ handles: "all", minWidth: 60, minHeight: 60, maxWidth: 100, maxHeight: 100 }); - TestHelpers.resizable.drag(handle, -50, -50); + resizableTestHelper.drag(handle, -50, -50); equal( target.width(), 60, "compare minWidth" ); equal( target.height(), 60, "compare minHeight" ); - TestHelpers.resizable.drag(handle, 70, 70); + resizableTestHelper.drag(handle, 70, 70); equal( target.width(), 100, "compare maxWidth" ); equal( target.height(), 100, "compare maxHeight" ); }); @@ -338,11 +343,11 @@ test("ui-resizable-sw { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1 var handle = ".ui-resizable-sw", target = $("#resizable1").resizable({ handles: "all", minWidth: 60, minHeight: 60, maxWidth: 100, maxHeight: 100 }); - TestHelpers.resizable.drag(handle, 50, -50); + resizableTestHelper.drag(handle, 50, -50); equal( target.width(), 60, "compare minWidth" ); equal( target.height(), 60, "compare minHeight" ); - TestHelpers.resizable.drag(handle, -70, 70); + resizableTestHelper.drag(handle, -70, 70); equal( target.width(), 100, "compare maxWidth" ); equal( target.height(), 100, "compare maxHeight" ); }); @@ -352,11 +357,11 @@ test("ui-resizable-ne { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1 var handle = ".ui-resizable-ne", target = $("#resizable1").resizable({ handles: "all", minWidth: 60, minHeight: 60, maxWidth: 100, maxHeight: 100 }); - TestHelpers.resizable.drag(handle, -50, 50); + resizableTestHelper.drag(handle, -50, 50); equal( target.width(), 60, "compare minWidth" ); equal( target.height(), 60, "compare minHeight" ); - TestHelpers.resizable.drag(handle, 70, -70); + resizableTestHelper.drag(handle, 70, -70); equal( target.width(), 100, "compare maxWidth" ); equal( target.height(), 100, "compare maxHeight" ); }); @@ -366,11 +371,11 @@ test("ui-resizable-nw { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1 var handle = ".ui-resizable-nw", target = $("#resizable1").resizable({ handles: "all", minWidth: 60, minHeight: 60, maxWidth: 100, maxHeight: 100 }); - TestHelpers.resizable.drag(handle, 70, 70); + resizableTestHelper.drag(handle, 70, 70); equal( target.width(), 60, "compare minWidth" ); equal( target.height(), 60, "compare minHeight" ); - TestHelpers.resizable.drag(handle, -70, -70); + resizableTestHelper.drag(handle, -70, -70); equal( target.width(), 100, "compare maxWidth" ); equal( target.height(), 100, "compare maxHeight" ); }); @@ -397,11 +402,11 @@ test( "alsoResize + containment", function() { containment: "#container" }); - TestHelpers.resizable.drag( ".ui-resizable-se", 400, 400 ); + resizableTestHelper.drag( ".ui-resizable-se", 400, 400 ); equal( element.width(), 300, "resizable constrained width at containment edge" ); equal( element.height(), 200, "resizable constrained height at containment edge" ); equal( other.width(), 250, "alsoResize constrained width at containment edge" ); equal( other.height(), 150, "alsoResize constrained height at containment edge" ); }); -})(jQuery); +}); diff --git a/tests/unit/resizable/resizable_test_helpers.js b/tests/unit/resizable/resizable_test_helpers.js index 7ab5aa1a545..706893a61ee 100644 --- a/tests/unit/resizable/resizable_test_helpers.js +++ b/tests/unit/resizable/resizable_test_helpers.js @@ -1,4 +1,9 @@ -TestHelpers.resizable = { +define([ + "jquery", + "jquery.simulate" +], function( $ ) { + +return { drag: function( el, dx, dy ) { // this mouseover is to work around a limitation in resizable // TODO: fix resizable so handle doesn't require mouseover in order to be used @@ -8,4 +13,6 @@ TestHelpers.resizable = { dy: dy }); } -}; \ No newline at end of file +}; + +}); diff --git a/tests/unit/selectable/selectable.html b/tests/unit/selectable/selectable.html index cc4edf90969..7555e8c3f51 100644 --- a/tests/unit/selectable/selectable.html +++ b/tests/unit/selectable/selectable.html @@ -4,30 +4,12 @@ jQuery UI Selectable Test Suite - - - - - + - - - - - - - + @@ -43,5 +25,24 @@ + + + + + + + + diff --git a/tests/unit/selectable/selectable_common.js b/tests/unit/selectable/selectable_common.js index d00a47be593..648c83a5b2c 100644 --- a/tests/unit/selectable/selectable_common.js +++ b/tests/unit/selectable/selectable_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests("selectable", { +define([ + "helper/testsuite", + "ui/selectable" +], function( testHelper ) { + +testHelper.commonWidgetTests("selectable", { defaults: { appendTo: "body", autoRefresh: true, @@ -19,3 +24,5 @@ TestHelpers.commonWidgetTests("selectable", { unselecting: null } }); + +}); diff --git a/tests/unit/selectable/selectable_events.js b/tests/unit/selectable/selectable_events.js index 2cb99f49c12..ec7033782d8 100644 --- a/tests/unit/selectable/selectable_events.js +++ b/tests/unit/selectable/selectable_events.js @@ -1,7 +1,13 @@ +define([ + "jquery", + "helper/testsuite", + "ui/selectable", + "jquery.simulate" +], function( $, testHelper ) { + /* * selectable_events.js */ -(function( $ ) { module("selectable: events"); @@ -40,7 +46,7 @@ test( "mousedown: initial position of helper", function() { var helperOffset, element = $( "#selectable1" ).selectable(), - contentToForceScroll = TestHelpers.forceScrollableWindow( "body" ); + contentToForceScroll = testHelper.forceScrollableWindow( "body" ); $( window ).scrollTop( 100 ).scrollLeft( 100 ); @@ -62,4 +68,4 @@ test( "mousedown: initial position of helper", function() { $( window ).scrollTop( 0 ).scrollLeft( 0 ); }); -})( jQuery ); +}); diff --git a/tests/unit/selectable/selectable_methods.js b/tests/unit/selectable/selectable_methods.js index 179d4ecd6b6..4f4da1809b7 100644 --- a/tests/unit/selectable/selectable_methods.js +++ b/tests/unit/selectable/selectable_methods.js @@ -1,7 +1,11 @@ +define([ + "jquery", + "ui/selectable" +], function( $ ) { + /* * selectable_methods.js */ -(function($) { module("selectable: methods"); @@ -107,4 +111,4 @@ test( "disable", function() { equal( chainable, element, "disable is chainable" ); }); -})(jQuery); +}); diff --git a/tests/unit/selectable/selectable_options.js b/tests/unit/selectable/selectable_options.js index 973247f5695..3a5035c808d 100644 --- a/tests/unit/selectable/selectable_options.js +++ b/tests/unit/selectable/selectable_options.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "ui/selectable", + "jquery.simulate" +], function( $ ) { + /* * selectable_options.js */ -(function($) { module("selectable: options"); @@ -62,4 +67,4 @@ test("filter", function() { el.selectable("destroy"); }); -})(jQuery); +}); diff --git a/tests/unit/selectmenu/selectmenu.html b/tests/unit/selectmenu/selectmenu.html index 618eea3d00a..99c9acd7d38 100644 --- a/tests/unit/selectmenu/selectmenu.html +++ b/tests/unit/selectmenu/selectmenu.html @@ -4,31 +4,12 @@ jQuery UI Selectmenu Test Suite - - - - - + - - - - - - - + @@ -83,5 +64,24 @@ + + + + + + + + diff --git a/tests/unit/selectmenu/selectmenu_common.js b/tests/unit/selectmenu/selectmenu_common.js index 10073fe97be..044e0407527 100644 --- a/tests/unit/selectmenu/selectmenu_common.js +++ b/tests/unit/selectmenu/selectmenu_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "selectmenu", { +define([ + "helper/testsuite", + "ui/selectmenu" +], function( testHelper ) { + +testHelper.commonWidgetTests( "selectmenu", { defaults: { appendTo: null, disabled: null, @@ -21,3 +26,5 @@ TestHelpers.commonWidgetTests( "selectmenu", { select: null } }); + +}); diff --git a/tests/unit/selectmenu/selectmenu_core.js b/tests/unit/selectmenu/selectmenu_core.js index e9312d12fc1..ab9eea45cf9 100644 --- a/tests/unit/selectmenu/selectmenu_core.js +++ b/tests/unit/selectmenu/selectmenu_core.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "ui/selectmenu", + "jquery.simulate" +], function( $ ) { module( "selectmenu: core" ); @@ -271,4 +275,4 @@ $.each([ }); }); -})( jQuery ); +}); diff --git a/tests/unit/selectmenu/selectmenu_events.js b/tests/unit/selectmenu/selectmenu_events.js index 3ab6e93bc68..ebd864125d0 100644 --- a/tests/unit/selectmenu/selectmenu_events.js +++ b/tests/unit/selectmenu/selectmenu_events.js @@ -1,4 +1,8 @@ -(function ( $ ) { +define([ + "jquery", + "ui/selectmenu", + "jquery.simulate" +], function( $ ) { module( "selectmenu: events", { setup: function() { @@ -130,4 +134,4 @@ asyncTest( "select", function() { }); }); -})( jQuery ); +}); diff --git a/tests/unit/selectmenu/selectmenu_methods.js b/tests/unit/selectmenu/selectmenu_methods.js index ca1b8c6a34c..3b226cf931e 100644 --- a/tests/unit/selectmenu/selectmenu_methods.js +++ b/tests/unit/selectmenu/selectmenu_methods.js @@ -1,10 +1,15 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "ui/selectmenu", + "jquery.simulate" +], function( $, testHelper ) { module( "selectmenu: methods" ); test( "destroy", function() { expect( 1 ); - domEqual( "#speed", function() { + testHelper.domEqual( "#speed", function() { $( "#speed" ).selectmenu().selectmenu( "destroy" ); }); }); @@ -168,4 +173,4 @@ test( "widget and menuWidget", function() { ok( menu.is( "ul.ui-menu" ), "Menu Widget: element and class" ); }); -})( jQuery ); +}); diff --git a/tests/unit/selectmenu/selectmenu_options.js b/tests/unit/selectmenu/selectmenu_options.js index ce66e822b17..4b663cc3c25 100644 --- a/tests/unit/selectmenu/selectmenu_options.js +++ b/tests/unit/selectmenu/selectmenu_options.js @@ -1,4 +1,7 @@ -(function ( $ ) { +define([ + "jquery", + "ui/selectmenu" +], function( $ ) { module( "selectmenu: options" ); @@ -122,4 +125,4 @@ test( "width", function() { equal( button.outerWidth(), 300, "button width fills container" ); }); -})( jQuery ); +}); diff --git a/tests/unit/slider/slider.html b/tests/unit/slider/slider.html index 0142b5e6bdd..24335d15c04 100644 --- a/tests/unit/slider/slider.html +++ b/tests/unit/slider/slider.html @@ -4,30 +4,12 @@ jQuery UI Slider Test Suite - - - - - + - - - - - - - + @@ -40,5 +22,24 @@ + + + + + + + + diff --git a/tests/unit/slider/slider_common.js b/tests/unit/slider/slider_common.js index 6d7278de6a2..6af5f8767a2 100644 --- a/tests/unit/slider/slider_common.js +++ b/tests/unit/slider/slider_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "slider", { +define([ + "helper/testsuite", + "ui/slider" +], function( testHelper ) { + +testHelper.commonWidgetTests( "slider", { defaults: { animate: false, cancel: "input,textarea,button,select,option", @@ -21,3 +26,5 @@ TestHelpers.commonWidgetTests( "slider", { stop: null } }); + +}); diff --git a/tests/unit/slider/slider_core.js b/tests/unit/slider/slider_core.js index 3649bded038..40b77a1e825 100644 --- a/tests/unit/slider/slider_core.js +++ b/tests/unit/slider/slider_core.js @@ -1,4 +1,9 @@ -(function( $ ) { +define([ + "jquery", + "jquery.simulate", + "ui/slider" +], function( $ ) { + // // Slider Test Helper Functions // @@ -294,4 +299,4 @@ test( "keydown LEFT on handle decreases value by step, not less than min", funct element.slider( "destroy" ); }); -})( jQuery ); +}); diff --git a/tests/unit/slider/slider_events.js b/tests/unit/slider/slider_events.js index ec93a056536..4ab220db1ca 100644 --- a/tests/unit/slider/slider_events.js +++ b/tests/unit/slider/slider_events.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "jquery.simulate", + "ui/slider" +], function( $ ) { module( "slider: events" ); @@ -149,4 +153,4 @@ test( "mouse based interaction part two: when handles overlap", function() { }); -})( jQuery ); +}); diff --git a/tests/unit/slider/slider_methods.js b/tests/unit/slider/slider_methods.js index c5c7e1254aa..8844261efc2 100644 --- a/tests/unit/slider/slider_methods.js +++ b/tests/unit/slider/slider_methods.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "ui/slider" +], function( $, testHelper ) { module( "slider: methods" ); @@ -25,7 +29,7 @@ test( "init", function() { test( "destroy", function() { expect( 1 ); - domEqual( "#slider1", function() { + testHelper.domEqual( "#slider1", function() { $( "#slider1" ).slider().slider( "destroy" ); }); }); @@ -94,4 +98,4 @@ test( "value", function() { // ok(false, "missing test - untested code is broken code." ); //}); -})( jQuery ); +}); diff --git a/tests/unit/slider/slider_options.js b/tests/unit/slider/slider_options.js index 587f847cfa3..3f7ddae70cd 100644 --- a/tests/unit/slider/slider_options.js +++ b/tests/unit/slider/slider_options.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "jquery.simulate", + "ui/slider" +], function( $ ) { var element, options; @@ -316,4 +320,4 @@ test( "range", function() { element.slider( "destroy" ); }); -})( jQuery ); +}); diff --git a/tests/unit/sortable/sortable.html b/tests/unit/sortable/sortable.html index 3edc999b76e..02e01beb210 100644 --- a/tests/unit/sortable/sortable.html +++ b/tests/unit/sortable/sortable.html @@ -4,32 +4,12 @@ jQuery UI Sortable Test Suite - - - - - + - - - - - - - - + #sortable, #sortable2 { position:relative; @@ -100,5 +80,24 @@ + + + + + + + + diff --git a/tests/unit/sortable/sortable_common.js b/tests/unit/sortable/sortable_common.js index 86850a658c3..bad6f76755e 100644 --- a/tests/unit/sortable/sortable_common.js +++ b/tests/unit/sortable/sortable_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "sortable", { +define([ + "helper/testsuite", + "ui/sortable" +], function( testHelper ) { + +testHelper.commonWidgetTests( "sortable", { defaults: { appendTo: "parent", axis: false, @@ -43,3 +48,5 @@ TestHelpers.commonWidgetTests( "sortable", { update: null } }); + +}); diff --git a/tests/unit/sortable/sortable_core.js b/tests/unit/sortable/sortable_core.js index 54069a9bfbd..14c5044ccf3 100644 --- a/tests/unit/sortable/sortable_core.js +++ b/tests/unit/sortable/sortable_core.js @@ -1,9 +1,13 @@ +define([ + "jquery", + "./sortable_test_helpers", + "ui/sortable" +], function( $, sortableTestHelper ) { + /* * sortable_core.js */ -(function( $ ) { - module( "sortable: core" ); test( "#9314: Sortable: Items cannot be dragged directly into bottom position", function() { @@ -13,7 +17,7 @@ test( "#9314: Sortable: Items cannot be dragged directly into bottom position", connectWith: ".connectWith" }); - TestHelpers.sortable.sort( $( "li", el[ 1 ] )[ 0 ], 0, -12, 5, "Dragging the sortable into connected sortable" ); + sortableTestHelper.sort( $( "li", el[ 1 ] )[ 0 ], 0, -12, 5, "Dragging the sortable into connected sortable" ); }); test( "ui-sortable-handle applied to appropriate element", function() { @@ -37,4 +41,4 @@ test( "ui-sortable-handle applied to appropriate element", function() { equal( el.find( ".ui-sortable-handle" ).length, 0, "class name removed on destroy" ); }); -})( jQuery ); +}); diff --git a/tests/unit/sortable/sortable_events.js b/tests/unit/sortable/sortable_events.js index 1bf72cc6a8c..0b430bca423 100644 --- a/tests/unit/sortable/sortable_events.js +++ b/tests/unit/sortable/sortable_events.js @@ -1,7 +1,14 @@ +define([ + "jquery", + "./sortable_test_helpers", + "jquery.simulate", + "ui/draggable", + "ui/sortable" +], function( $, sortableTestHelper ) { + /* * sortable_events.js */ -(function($) { module("sortable: events"); @@ -172,7 +179,7 @@ test("#3019: Stop fires too early", function() { } }); - TestHelpers.sortable.sort($("li", el)[0], 0, 44, 2, "Dragging the sortable"); + sortableTestHelper.sort($("li", el)[0], 0, 44, 2, "Dragging the sortable"); equal(helper, null, "helper should be false"); }); @@ -375,4 +382,4 @@ test("deactivate", function() { }); */ -})(jQuery); +}); diff --git a/tests/unit/sortable/sortable_methods.js b/tests/unit/sortable/sortable_methods.js index f3fe240e701..0cb825ebcfc 100644 --- a/tests/unit/sortable/sortable_methods.js +++ b/tests/unit/sortable/sortable_methods.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "./sortable_test_helpers", + "ui/sortable" +], function( $, sortableTestHelper ) { + /* * sortable_methods.js */ -(function($) { module("sortable: methods"); @@ -47,7 +52,7 @@ test("enable", function() { el = $("#sortable").sortable({ disabled: true }); - TestHelpers.sortable.sort($("li", el)[0], 0, 44, 0, ".sortable({ disabled: true })"); + sortableTestHelper.sort($("li", el)[0], 0, 44, 0, ".sortable({ disabled: true })"); el.sortable("enable"); equal(el.sortable("option", "disabled"), false, "disabled option getter"); @@ -57,7 +62,7 @@ test("enable", function() { el.sortable("option", "disabled", false); equal(el.sortable("option", "disabled"), false, "disabled option setter"); - TestHelpers.sortable.sort($("li", el)[0], 0, 44, 2, ".sortable('option', 'disabled', false)"); + sortableTestHelper.sort($("li", el)[0], 0, 44, 2, ".sortable('option', 'disabled', false)"); expected = $("
").sortable(), actual = expected.sortable("enable"); @@ -70,15 +75,15 @@ test( "disable", function() { var chainable, element = $( "#sortable" ).sortable({ disabled: false }); - TestHelpers.sortable.sort( $( "li", element )[ 0 ], 0, 44, 2, ".sortable({ disabled: false })" ); + sortableTestHelper.sort( $( "li", element )[ 0 ], 0, 44, 2, ".sortable({ disabled: false })" ); chainable = element.sortable( "disable" ); - TestHelpers.sortable.sort( $( "li", element )[ 0 ], 0, 44, 0, "disabled.sortable getter" ); + sortableTestHelper.sort( $( "li", element )[ 0 ], 0, 44, 0, "disabled.sortable getter" ); element.sortable( "destroy" ); element.sortable({ disabled: false }); - TestHelpers.sortable.sort( $( "li", element )[ 0 ], 0, 44, 2, ".sortable({ disabled: false })" ); + sortableTestHelper.sort( $( "li", element )[ 0 ], 0, 44, 2, ".sortable({ disabled: false })" ); element.sortable( "option", "disabled", true); equal( element.sortable( "option", "disabled" ), true, "disabled option setter" ); @@ -86,8 +91,8 @@ test( "disable", function() { ok( !element.sortable( "widget" ).attr( "aria-disabled" ), "element does not get aria-disabled" ); ok( element.sortable( "widget" ).hasClass( "ui-sortable-disabled" ), "element gets ui-sortable-disabled" ); - TestHelpers.sortable.sort($( "li", element )[ 0 ], 0, 44, 0, ".sortable('option', 'disabled', true)" ); + sortableTestHelper.sort($( "li", element )[ 0 ], 0, 44, 0, ".sortable('option', 'disabled', true)" ); equal( chainable, element, "disable is chainable" ); }); -})(jQuery); +}); diff --git a/tests/unit/sortable/sortable_options.js b/tests/unit/sortable/sortable_options.js index f2beb4dbcd6..af66435597c 100644 --- a/tests/unit/sortable/sortable_options.js +++ b/tests/unit/sortable/sortable_options.js @@ -1,7 +1,12 @@ +define([ + "jquery", + "jquery.simulate", + "ui/sortable" +], function( $ ) { + /* * sortable_options.js */ -(function($) { module("sortable: options"); @@ -457,4 +462,4 @@ test("{ zIndex: false }", function() { ok(false, "missing test - untested code is broken code."); }); */ -})(jQuery); +}); diff --git a/tests/unit/sortable/sortable_test_helpers.js b/tests/unit/sortable/sortable_test_helpers.js index 12e4829ea2c..0d1ef8b6a63 100644 --- a/tests/unit/sortable/sortable_test_helpers.js +++ b/tests/unit/sortable/sortable_test_helpers.js @@ -1,4 +1,9 @@ -TestHelpers.sortable = { +define([ + "jquery", + "jquery.simulate" +], function( $ ) { + +return { sort: function( handle, dx, dy, index, msg ) { $( handle ).simulate( "drag", { dx: dx, @@ -6,4 +11,6 @@ TestHelpers.sortable = { }); equal( $( handle ).parent().children().index( handle ), index, msg ); } -}; \ No newline at end of file +}; + +}); diff --git a/tests/unit/spinner/spinner.html b/tests/unit/spinner/spinner.html index 58988d74235..dc73625095b 100644 --- a/tests/unit/spinner/spinner.html +++ b/tests/unit/spinner/spinner.html @@ -4,33 +4,12 @@ jQuery UI Spinner Test Suite - - - - - - - + - - - - - - - - + @@ -41,5 +20,27 @@ + + + + + + + + + + diff --git a/tests/unit/spinner/spinner_common.js b/tests/unit/spinner/spinner_common.js index b494e3ca34d..8396bb3e679 100644 --- a/tests/unit/spinner/spinner_common.js +++ b/tests/unit/spinner/spinner_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "spinner", { +define([ + "helper/testsuite", + "ui/spinner" +], function( testHelper ) { + +testHelper.commonWidgetTests( "spinner", { defaults: { culture: null, disabled: false, @@ -21,3 +26,5 @@ TestHelpers.commonWidgetTests( "spinner", { stop: null } }); + +}); diff --git a/tests/unit/spinner/spinner_core.js b/tests/unit/spinner/spinner_core.js index 03088fecdca..c39e1d4eb86 100644 --- a/tests/unit/spinner/spinner_core.js +++ b/tests/unit/spinner/spinner_core.js @@ -1,6 +1,10 @@ -(function( $ ) { +define([ + "jquery", + "./spinner_test_helpers", + "ui/spinner" +], function( $, spinnerTestHelper ) { -var simulateKeyDownUp = TestHelpers.spinner.simulateKeyDownUp; +var simulateKeyDownUp = spinnerTestHelper.simulateKeyDownUp; module( "spinner: core" ); @@ -235,4 +239,4 @@ test( "precision", function() { equal( element.val(), "0.05", "precision from min" ); }); -})( jQuery ); +}); diff --git a/tests/unit/spinner/spinner_events.js b/tests/unit/spinner/spinner_events.js index 007d34cf770..0568cd6e5ad 100644 --- a/tests/unit/spinner/spinner_events.js +++ b/tests/unit/spinner/spinner_events.js @@ -1,6 +1,10 @@ -(function( $ ) { +define([ + "jquery", + "./spinner_test_helpers", + "ui/spinner" +], function( $, spinnerTestHelper ) { -var simulateKeyDownUp = TestHelpers.spinner.simulateKeyDownUp; +var simulateKeyDownUp = spinnerTestHelper.simulateKeyDownUp; module( "spinner: events" ); @@ -256,4 +260,4 @@ asyncTest( "change", function() { setTimeout( step1 ); }); -})( jQuery ); +}); diff --git a/tests/unit/spinner/spinner_methods.js b/tests/unit/spinner/spinner_methods.js index cc4bc5a16ca..c918f14e3f0 100644 --- a/tests/unit/spinner/spinner_methods.js +++ b/tests/unit/spinner/spinner_methods.js @@ -1,12 +1,17 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "./spinner_test_helpers", + "ui/spinner" +], function( $, testHelper, spinnerTestHelper ) { -var simulateKeyDownUp = TestHelpers.spinner.simulateKeyDownUp; +var simulateKeyDownUp = spinnerTestHelper.simulateKeyDownUp; module( "spinner: methods" ); test( "destroy", function() { expect( 1 ); - domEqual( "#spin", function() { + testHelper.domEqual( "#spin", function() { $( "#spin" ).spinner().spinner( "destroy" ); }); }); @@ -205,4 +210,4 @@ test( "widget", function() { strictEqual( widgetElement[ 0 ], element.parent()[ 0 ], "parent element" ); }); -})( jQuery ); +}); diff --git a/tests/unit/spinner/spinner_options.js b/tests/unit/spinner/spinner_options.js index 6f3650f923a..08a40e5f6c6 100644 --- a/tests/unit/spinner/spinner_options.js +++ b/tests/unit/spinner/spinner_options.js @@ -1,4 +1,7 @@ -(function( $ ) { +define([ + "jquery", + "ui/spinner" +], function( $ ) { module( "spinner: options" ); @@ -259,4 +262,4 @@ test( "step, string", function() { equal( element.val(), "$0.70", "stepUp" ); }); -})( jQuery ); +}); diff --git a/tests/unit/spinner/spinner_test_helpers.js b/tests/unit/spinner/spinner_test_helpers.js index 2021e8f7ae9..dcaa5b1c5f1 100644 --- a/tests/unit/spinner/spinner_test_helpers.js +++ b/tests/unit/spinner/spinner_test_helpers.js @@ -1,4 +1,9 @@ -TestHelpers.spinner = { +define([ + "jquery", + "jquery.simulate" +], function( $ ) { + +return { simulateKeyDownUp: function( element, keyCode, shift ) { element .simulate( "keydown", { keyCode: keyCode, shiftKey: shift || false } ) @@ -6,3 +11,4 @@ TestHelpers.spinner = { } }; +}); diff --git a/tests/unit/tabs/tabs.html b/tests/unit/tabs/tabs.html index 778639eb347..1397e10be0c 100644 --- a/tests/unit/tabs/tabs.html +++ b/tests/unit/tabs/tabs.html @@ -4,30 +4,12 @@ jQuery UI Tabs Test Suite - - - - - + - - - - - - - - + #tabs8, #tabs8 * { margin: 0; @@ -147,5 +129,25 @@ + + + + + + + + diff --git a/tests/unit/tabs/tabs_common.js b/tests/unit/tabs/tabs_common.js index a589515f806..8624742904d 100644 --- a/tests/unit/tabs/tabs_common.js +++ b/tests/unit/tabs/tabs_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "tabs", { +define([ + "helper/testsuite", + "ui/tabs" +], function( testHelper ) { + +testHelper.commonWidgetTests( "tabs", { defaults: { active: null, collapsible: false, @@ -16,3 +21,5 @@ TestHelpers.commonWidgetTests( "tabs", { load: null } }); + +}); diff --git a/tests/unit/tabs/tabs_core.js b/tests/unit/tabs/tabs_core.js index 2e094e11b0a..1407237213a 100644 --- a/tests/unit/tabs/tabs_core.js +++ b/tests/unit/tabs/tabs_core.js @@ -1,6 +1,11 @@ -(function( $ ) { +define([ + "jquery", + "./tabs_test_helpers", + "jquery.simulate", + "ui/tabs" +], function( $, tabsTestHelper ) { -var state = TestHelpers.tabs.state; +var state = tabsTestHelper.state; module( "tabs: core" ); @@ -612,4 +617,4 @@ test( "#4033 - IE expands hash to full url and misinterprets tab as ajax", funct state( element, 1 ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/tabs/tabs_events.js b/tests/unit/tabs/tabs_events.js index f9b1755f778..d1c565e5224 100644 --- a/tests/unit/tabs/tabs_events.js +++ b/tests/unit/tabs/tabs_events.js @@ -1,6 +1,10 @@ -(function( $ ) { +define([ + "jquery", + "./tabs_test_helpers", + "ui/tabs" +], function( $, tabsTestHelper ) { -var state = TestHelpers.tabs.state; +var state = tabsTestHelper.state; module( "tabs: events" ); @@ -315,4 +319,4 @@ asyncTest( "load", function() { } }); -}( jQuery ) ); +}); diff --git a/tests/unit/tabs/tabs_methods.js b/tests/unit/tabs/tabs_methods.js index a3d663f272d..60083b3bea6 100644 --- a/tests/unit/tabs/tabs_methods.js +++ b/tests/unit/tabs/tabs_methods.js @@ -1,16 +1,21 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "./tabs_test_helpers", + "ui/tabs" +], function( $, testHelper, tabsTestHelper ) { -var disabled = TestHelpers.tabs.disabled, - state = TestHelpers.tabs.state; +var disabled = tabsTestHelper.disabled, + state = tabsTestHelper.state; module( "tabs: methods" ); test( "destroy", function() { expect( 2 ); - domEqual( "#tabs1", function() { + testHelper.domEqual( "#tabs1", function() { $( "#tabs1" ).tabs().tabs( "destroy" ); }); - domEqual( "#tabs2", function() { + testHelper.domEqual( "#tabs2", function() { $( "#tabs2" ).tabs().tabs( "destroy" ); }); }); @@ -267,4 +272,4 @@ test( "widget", function() { strictEqual( widgetElement[ 0 ], element[ 0 ], "same element" ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/tabs/tabs_options.js b/tests/unit/tabs/tabs_options.js index a9632481e8f..71c36aaaaa8 100644 --- a/tests/unit/tabs/tabs_options.js +++ b/tests/unit/tabs/tabs_options.js @@ -1,8 +1,12 @@ -(function( $ ) { +define([ + "jquery", + "./tabs_test_helpers", + "ui/tabs" +], function( $, tabsTestHelper ) { -var disabled = TestHelpers.tabs.disabled, - equalHeight = TestHelpers.tabs.equalHeight, - state = TestHelpers.tabs.state; +var disabled = tabsTestHelper.disabled, + equalHeight = tabsTestHelper.equalHeight, + state = tabsTestHelper.state; module( "tabs: options" ); @@ -363,4 +367,4 @@ asyncTest( "hide and show - animation", function() { }); -}( jQuery ) ); +}); diff --git a/tests/unit/tabs/tabs_test_helpers.js b/tests/unit/tabs/tabs_test_helpers.js index 0c930918615..9ccc12b5ec2 100644 --- a/tests/unit/tabs/tabs_test_helpers.js +++ b/tests/unit/tabs/tabs_test_helpers.js @@ -1,4 +1,6 @@ -TestHelpers.tabs = { +define(function() { + +return { disabled: function( tabs, state ) { var expected, actual, internalState = tabs.tabs( "option", "disabled" ); @@ -65,3 +67,4 @@ TestHelpers.tabs = { } }; +}); diff --git a/tests/unit/tooltip/tooltip.html b/tests/unit/tooltip/tooltip.html index 18bd88c2c98..936453ac06f 100644 --- a/tests/unit/tooltip/tooltip.html +++ b/tests/unit/tooltip/tooltip.html @@ -4,30 +4,12 @@ jQuery UI Tooltip Test Suite - - - - - + - - - - - - - + @@ -47,5 +29,24 @@ + + + + + + + + diff --git a/tests/unit/tooltip/tooltip_common.js b/tests/unit/tooltip/tooltip_common.js index 2d6ea91eccd..9460bf207f1 100644 --- a/tests/unit/tooltip/tooltip_common.js +++ b/tests/unit/tooltip/tooltip_common.js @@ -1,4 +1,9 @@ -TestHelpers.commonWidgetTests( "tooltip", { +define([ + "helper/testsuite", + "ui/tooltip" +], function( testHelper ) { + +testHelper.commonWidgetTests( "tooltip", { defaults: { content: function() {}, disabled: false, @@ -19,3 +24,5 @@ TestHelpers.commonWidgetTests( "tooltip", { open: null } }); + +}); diff --git a/tests/unit/tooltip/tooltip_core.js b/tests/unit/tooltip/tooltip_core.js index 760ffeed2eb..8608cdcbed4 100644 --- a/tests/unit/tooltip/tooltip_core.js +++ b/tests/unit/tooltip/tooltip_core.js @@ -1,4 +1,7 @@ -(function( $ ) { +define([ + "jquery", + "ui/tooltip" +], function( $ ) { module( "tooltip: core" ); @@ -174,4 +177,4 @@ asyncTest( "destroy during hide animation; only one close event", function() { }); }); -}( jQuery ) ); +}); diff --git a/tests/unit/tooltip/tooltip_events.js b/tests/unit/tooltip/tooltip_events.js index de16471aedf..0dbddd934f9 100644 --- a/tests/unit/tooltip/tooltip_events.js +++ b/tests/unit/tooltip/tooltip_events.js @@ -1,4 +1,7 @@ -(function( $ ) { +define([ + "jquery", + "ui/tooltip" +], function( $ ) { module( "tooltip: events" ); @@ -54,4 +57,4 @@ test( "focus events", function() { element.trigger( "focusout" ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/tooltip/tooltip_methods.js b/tests/unit/tooltip/tooltip_methods.js index 8d1b1bd5b7d..86f5b377e21 100644 --- a/tests/unit/tooltip/tooltip_methods.js +++ b/tests/unit/tooltip/tooltip_methods.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "ui/tooltip" +], function( $, testHelper ) { module( "tooltip: methods" ); @@ -6,12 +10,12 @@ test( "destroy", function() { expect( 3 ); var element = $( "#tooltipped1" ); - domEqual( "#tooltipped1", function() { + testHelper.domEqual( "#tooltipped1", function() { element.tooltip().tooltip( "destroy" ); }); // make sure that open tooltips are removed on destroy - domEqual( "#tooltipped1", function() { + testHelper.domEqual( "#tooltipped1", function() { element .tooltip() .tooltip( "open", $.Event( "mouseover", { target: element[0] }) ) @@ -137,4 +141,4 @@ test( "preserve changes to title attributes on close and destroy", function() { } ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/tooltip/tooltip_options.js b/tests/unit/tooltip/tooltip_options.js index 17f0a423775..7633f969d2c 100644 --- a/tests/unit/tooltip/tooltip_options.js +++ b/tests/unit/tooltip/tooltip_options.js @@ -1,4 +1,7 @@ -(function( $ ) { +define([ + "jquery", + "ui/tooltip" +], function( $ ) { module( "tooltip: options" ); @@ -171,4 +174,4 @@ test( "track and programmatic focus", function() { equal( "inputtitle", $( ".ui-tooltip" ).text() ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/widget/widget.html b/tests/unit/widget/widget.html index 2b764abab9e..1b3572179ad 100644 --- a/tests/unit/widget/widget.html +++ b/tests/unit/widget/widget.html @@ -4,25 +4,12 @@ jQuery UI Widget Test Suite - - - - - + - - - - - + @@ -36,5 +23,22 @@ + + + + + + + + diff --git a/tests/unit/widget/widget_animation.js b/tests/unit/widget/widget_animation.js index 8ef55aa115c..d1f2be72cc4 100644 --- a/tests/unit/widget/widget_animation.js +++ b/tests/unit/widget/widget_animation.js @@ -1,3 +1,7 @@ +define([ + "jquery", + "ui/widget" +], function( $ ) { module( "widget animation", (function() { var show = $.fn.show, @@ -255,3 +259,5 @@ asyncTest( "show: object(effect)", function() { next(); }); }); + +}); diff --git a/tests/unit/widget/widget_core.js b/tests/unit/widget/widget_core.js index feafca9428c..31d06032f1f 100644 --- a/tests/unit/widget/widget_core.js +++ b/tests/unit/widget/widget_core.js @@ -1,4 +1,8 @@ -(function( $ ) { +define([ + "jquery", + "helper/testsuite", + "ui/widget" +], function( $, testHelper ) { module( "widget factory", { teardown: function() { @@ -9,7 +13,7 @@ module( "widget factory", { } }); -TestHelpers.testJshint( "widget" ); +testHelper.testJshint( "widget" ); test( "widget creation", function() { expect( 5 ); @@ -1504,4 +1508,4 @@ test( "$.widget.bridge() - widgetFullName", function() { equal( typeof instance, "object", "instance stored in .data(widgetFullName)" ); }); -}( jQuery ) ); +}); diff --git a/tests/unit/widget/widget_extend.js b/tests/unit/widget/widget_extend.js index 14f9a46e0d2..36c465778ec 100644 --- a/tests/unit/widget/widget_extend.js +++ b/tests/unit/widget/widget_extend.js @@ -1,3 +1,8 @@ +define([ + "jquery", + "ui/widget" +], function( $ ) { + test( "$.widget.extend()", function() { expect( 27 ); @@ -103,3 +108,5 @@ test( "$.widget.extend()", function() { input.key[0] = 10; deepEqual( input, output, "don't clone arrays" ); }); + +});