Skip to content

Commit f278f50

Browse files
committed
cleaned some tests comments
1 parent 2e7107c commit f278f50

File tree

4 files changed

+19
-7
lines changed

4 files changed

+19
-7
lines changed

tests/unit/css.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@ define([
33
"intern/chai!assert"
44
], function (registerSuite, assert) {
55

6-
// Note that as this test is running, other unrelated styles are likely being loaded
7-
// from other test suites that load widgets that use theme!. Make test resilient to this.
8-
//for(var i = 0; i < 100000000; i++){var test = "toto"+i;}
9-
106
var container;
117

128
var cssRequire = require.config({
139
context: "css",
10+
//note: BaseUrl is relative to requirejs-dplugins/node_modules/intern/ so baseUrl needs 3 "../"
11+
// to be able to access requirejs-dplugins sibling directories.
12+
// The last /requirejs-dplugins is here to allow to use css! directly instead of
13+
// requirejs-dplugins/css!
1414
baseUrl: "../../../requirejs-dplugins",
1515
paths: {dojo: "../dojo"}
1616
});

tests/unit/has.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,10 @@ define([
99
setup: function () {
1010
moduleRequire = require.config({
1111
context: "module",
12-
baseUrl: "../../../requirejs-dplugins", // note: "../../.." make "grunt intern:remote" fail
12+
//note: BaseUrl is relative to requirejs-dplugins/node_modules/intern/
13+
// and the extra ../requirejs-dplugins is necessary to access requirejs-dplugins
14+
// sibling directories.
15+
baseUrl: "../../../requirejs-dplugins",
1316
config: {
1417
has: {
1518
"config-feature": true,
@@ -99,6 +102,10 @@ define([
99102
setup: function () {
100103
pluginRequire = require.config({
101104
context: "plugin",
105+
//note: BaseUrl is relative to requirejs-dplugins/node_modules/intern/ so baseUrl needs 3 "../"
106+
// to be able to access requirejs-dplugins sibling directories.
107+
// The last /requirejs-dplugins is here to allow to use has! directly instead of
108+
// requirejs-dplugins/has!
102109
baseUrl: "../../../requirejs-dplugins",
103110
packages: [{
104111
name: "modules",

tests/unit/i18n.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,11 @@ define([
1414
context = getNewContext();
1515
require.config({
1616
context: context,
17-
baseUrl: "../../../requirejs-dplugins", // note: "../../.." make "grunt intern:remote" fail
17+
//note: BaseUrl is relative to requirejs-dplugins/node_modules/intern/ so baseUrl needs 3 "../"
18+
// to be able to access requirejs-dplugins sibling directories.
19+
// The last /requirejs-dplugins is here to allow to use i18n! directly instead of
20+
// requirejs-dplugins/i18n!
21+
baseUrl: "../../../requirejs-dplugins",
1822
packages: [{
1923
name: "myapp",
2024
location: "tests/unit/resources/app/dist/myapp"

tests/unit/jquery.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
1-
/* global context:true, onlyLayer:true */
21
define([
32
"intern!object",
43
"intern/chai!assert"
54
], function (registerSuite, assert) {
65
var index = 0;
76
function getContextRequire() {
87
return require.config({
8+
//note: BaseUrl is relative to requirejs-dplugins/node_modules/intern/ so baseUrl needs 3 "../"
9+
// to be able to access requirejs-dplugins sibling directories.
910
baseUrl: "../../..",
1011
context: "jquery" + index++
1112
});

0 commit comments

Comments
 (0)