Skip to content

Commit 3720df5

Browse files
SergioCrisostomoSergioCrisostomo
committed
add DOMReady specs
1 parent 0dbb4d5 commit 3720df5

10 files changed

+236
-25
lines changed

‎Gruntfile.js

+8
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,12 @@
11
"use strict";
2+
var fs = require('fs');
3+
var build = (function(){
4+
// travis testing
5+
if (process.env && process.env.BUILD) return process.env.BUILD == 'default' ? 'all' : 'nocompat';
6+
// local testing
7+
else return process.argv[2] == null || process.argv[2] == 'all' ? 'all' : 'nocompat';
8+
})();
9+
var htmlFileServer = require('./Tests/httpserver.js')(build);
210

311
module.exports = function(grunt) {
412

‎Specs/Utilities/DOMReady.js

+90-24
Original file line numberDiff line numberDiff line change
@@ -6,30 +6,96 @@ provides: ~
66
...
77
*/
88

9-
/* todo
10-
document.addListener = function(type, fn){
11-
if (this.addEventListener) this.addEventListener(type, fn, false);
12-
else this.attachEvent('on' + type, fn);
13-
return this;
14-
};
15-
16-
document.removeListener = function(type, fn){
17-
if (this.removeEventListener) this.removeEventListener(type, fn, false);
18-
else this.detachEvent('on' + type, fn);
19-
return this;
20-
};
21-
22-
23-
window.fireEvent =
24-
document.fireEvent = function(type){
25-
if (type == 'domready')
26-
for (var i = 0; i < domreadyCallbacks.length; ++i){
9+
describe("DOMReady", function(){
10+
11+
var win, frame, cb, ready;
12+
function checkStatus(){
13+
ready = win && win.callbackFired;
14+
if (ready) cb();
15+
return ready;
2716
}
28-
domreadyCallbacks[i]();
29-
};
3017

31-
window.addEvent = function(){};
18+
function newFrame(url){
19+
var iframe = new IFrame({
20+
src: 'specsserver/' + url
21+
});
22+
document.getElement('body').adopt(iframe);
23+
return iframe;
24+
}
25+
26+
beforeEach(function(){
27+
cb = jasmine.createSpy('DOMReady!');
28+
});
29+
30+
afterEach(function(){
31+
frame.destroy();
32+
win = cb = frame = ready = null;
33+
});
34+
35+
it('should fire DOMReady, after flushing, when the DOM is ready', function(){
36+
frame = newFrame('DOMReady.flushA.html~DOMReady.flushB.html');
37+
frame.addEvent('load', function(){
38+
win = frame.contentWindow;
39+
expect(win.moments[0]).toEqual('loading');
40+
expect(win.moments[1]).toEqual('loading');
41+
expect(win.moments[2] == 'interactive' || win.moments[2] == 'complete').toBeTruthy();
42+
});
43+
44+
waitsFor(function(){
45+
return checkStatus();
46+
}, "the iframe to load", 8000);
47+
runs(function(){
48+
expect(cb).toHaveBeenCalled();
49+
});
50+
});
51+
52+
it('should fire DOMReady when the DOM is ready', function(){
53+
frame = newFrame('DOMReady.head.html');
54+
frame.addEvent('load', function(){
55+
win = frame.contentWindow;
56+
});
57+
waitsFor(function(){
58+
return checkStatus();
59+
}, "the iframe to load", 1500);
60+
runs(function(){
61+
expect(cb).toHaveBeenCalled();
62+
});
63+
});
64+
65+
it('should fire DOMReady when a new `addEvent("domready"` is added', function(){
66+
frame = newFrame('DOMReady.onAdd.html');
67+
frame.addEvent('load', function(){
68+
win = frame.contentWindow;
69+
win.addEvent('domready', win.callback);
70+
});
71+
waitsFor(function(){
72+
return checkStatus();
73+
}, "the iframe to load", 1500);
74+
runs(function(){
75+
expect(cb).toHaveBeenCalled();
76+
});
77+
});
78+
79+
it('should fire when MooTools was loaded into a already-ready page', function(){
80+
frame = newFrame('DOMReady.delayed.html');
81+
var ready;
82+
frame.addEvent('load', function(){
83+
win = frame.contentWindow;
84+
expect(win.MooTools).toBeFalsy(); // because MooTools should not be loaded yet
85+
var i = setInterval(function(){
86+
if (win.addEvent && win.callback){
87+
win.addEvent('domready', win.callback);
88+
if (ready) clearInterval(i);
89+
}
90+
}, 50);
91+
});
92+
waitsFor(function(){
93+
return checkStatus();
94+
}, "the iframe to load and MooTools to be deployed", 6000);
95+
runs(function(){
96+
expect(cb).toHaveBeenCalled();
97+
});
98+
});
99+
100+
});
32101

33-
var Element = this.Element || {};
34-
Element.Events = {};
35-
*/

‎Tests/DOMReady/DOMReady.delayed.html

+27
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8" />
5+
<title>Add MooTools when page is already ready</title>
6+
</head>
7+
<body>
8+
9+
<script>
10+
function callback(){
11+
window.callbackFired = true;
12+
}
13+
14+
function addScript(path){
15+
var script = document.createElement('script');
16+
script.type = 'text/javascript';
17+
script.src = path;
18+
document.getElementsByTagName('head')[0].appendChild(script);
19+
}
20+
21+
window.callbackFired = false;
22+
window.onload = function(){
23+
addScript('mootoolsPath.js');
24+
}
25+
</script>
26+
</body>
27+
</html>

‎Tests/DOMReady/DOMReady.flushA.html

+18
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8" />
5+
<title>Flushed page scenario</title>
6+
<script src="mootoolsPath.js" type="text/javascript"></script>
7+
<script>
8+
window.moments = [];
9+
moments.push(document.readyState);
10+
11+
function callback(){
12+
window.callbackFired = true;
13+
moments.push(document.readyState);
14+
}
15+
window.callbackFired = false;
16+
window.addEvent("domready", callback);
17+
</script>
18+
</head>

‎Tests/DOMReady/DOMReady.flushB.html

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
<body>
2+
<div>body added...</div>
3+
<script>
4+
moments.push(document.readyState);
5+
</script>
6+
</body>

‎Tests/DOMReady/DOMReady.head.html

+21
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8" />
5+
<title>Normal DOMReady scenario</title>
6+
7+
<script src="mootoolsPath.js" type="text/javascript"></script>
8+
</head>
9+
<body>
10+
11+
<script>
12+
function callback(){
13+
window.callbackFired = true;
14+
}
15+
16+
window.callbackFired = false;
17+
window.addEvent('domready', callback);
18+
</script>
19+
</body>
20+
</html>
21+

‎Tests/DOMReady/DOMReady.onAdd.html

+19
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8" />
5+
<title>DOMReady added when page is already loaded</title>
6+
7+
<script src="mootoolsPath.js" type="text/javascript"></script>
8+
</head>
9+
<body>
10+
11+
<script>
12+
function callback(){
13+
window.callbackFired = true;
14+
}
15+
16+
window.callbackFired = false;
17+
</script>
18+
</body>
19+
</html>

‎Tests/gruntfile-options.js

+9-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,15 @@ var karmaOptions = {
6666
captureTimeout: 60000 * 2,
6767
singleRun: true,
6868
frameworks: ['jasmine', 'sinon'],
69-
files: ['Tests/Utilities/*.js', 'mootools-*.js'],
69+
files: [
70+
'Tests/Utilities/*.js',
71+
'mootools-*.js',
72+
{pattern: 'Source/**/*.*', included: false, served: true},
73+
{pattern: 'Tests/DOMReady/*.*', included: false, served: true}
74+
],
75+
proxies: {
76+
'/specsserver/': 'http://localhost:9000/'
77+
},
7078
sauceLabs: {
7179
username: process.env.SAUCE_USERNAME,
7280
accessKey: process.env.SAUCE_ACCESS_KEY,

‎Tests/httpserver.js

+37
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
"use strict";
2+
3+
var fs = require('fs');
4+
var path = require('path');
5+
var http = require('http');
6+
7+
module.exports = function(build){
8+
9+
var getFileContent = function(url, callback){
10+
fs.readFile(url, 'utf-8', function (err, content) {
11+
if (err) return console.log(err);
12+
content = content.replace('mootoolsPath', '/base/mootools-' + build);
13+
callback(content);
14+
});
15+
}
16+
17+
http.createServer(function(req, res){
18+
if (req.url.indexOf('~') != -1){ // multiple files to send as chunks
19+
var partials = req.url.split('~');
20+
partials.push(null); // res.end()
21+
res.writeHead(200, {'Content-Type': 'text/html'});
22+
partials.forEach(function(file, i){
23+
setTimeout(function(){
24+
if (!file) res.end('</html>');
25+
else getFileContent(path.join(__dirname, '/DOMReady/', file), function(data){
26+
res.write(data);
27+
});
28+
}, 2000 * i + 1);
29+
})
30+
} else { // single files
31+
getFileContent(path.join(__dirname, '/DOMReady/', req.url), function(data){
32+
res.end(data);
33+
});
34+
}
35+
}).listen(9000);
36+
}
37+

‎package.json

+1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@
4848
"karma-firefox-launcher": "~0.1.3",
4949
"karma-ie-launcher": "~0.1",
5050
"karma-safari-launcher": "~0.1",
51+
"path": "^0.11.14",
5152
"js-yaml": "^3.0.2"
5253
}
5354
}

0 commit comments

Comments
 (0)