Skip to content

Commit c8c9c84

Browse files
committed
1 parent 75c5864 commit c8c9c84

File tree

5 files changed

+35
-11
lines changed

5 files changed

+35
-11
lines changed

.gitignore

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
node_modules
22
npm-debug.log
33
tmp
4+
coverage/

.travis.yml

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
language: node_js
2+
#language: objective-c
3+
node_js:
4+
- "0.12"
5+
- "0.10"
6+
script:
7+
# - npm install -g grunt-cli
8+
# - npm install
9+
- npm test

lib/appc-coverage.js

+8-6
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,9 @@ var request = require('request'),
1212
const
1313
SUCCESS_UPLOAD = 'Successfully submitted coverage results to coverage.appcelerator.com',
1414
FAILED_UPLOAD = 'Failed to submit coverage results to coverage.appcelerator.com',
15-
SERVER = 'http://localhost:8080/v1';
15+
SERVER = 'https://454bcc35e4c9b0c030c89b311b4322330d84fa3b.cloudapp-enterprise.appcelerator.com';
16+
17+
exports.SERVER = SERVER;
1618

1719
/**
1820
* Get enviromental options such as service and git
@@ -59,18 +61,18 @@ exports.parseLcov = function (lcovString, callback) {
5961
* Upload the data to the coverage server
6062
*/
6163
exports.upload = function (data, callback) {
62-
request.post({
63-
url : SERVER + '/coverage',
64+
var url = SERVER + '/v1/coverage';
65+
request.post(url, {
6466
form : {
6567
type : 'grunt-appc-coverage',
6668
data: data
6769
},
68-
json: true
70+
json: true,
71+
agentOptions: false
6972
}, function (err, response, body) {
70-
if (response.statusCode === 500 || err || body.hasOwnProperty('error')) {
73+
if (err || response.statusCode === 500 || (typeof body === 'object' && body.hasOwnProperty('error'))) {
7174
return callback(FAILED_UPLOAD);
7275
}
73-
7476
return callback(null, SUCCESS_UPLOAD);
7577
});
7678
};

package.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "grunt-appc-coverage",
3-
"version": "0.1.0",
3+
"version": "0.1.4",
44
"description": "Grunt task to load coverage results and submit them to coverage.appcelerator.com",
55
"repository": "https://github.com/appcelerator-modules/grunt-appc-coverage",
66
"author": {
@@ -27,7 +27,7 @@
2727
"grunt-mocha-test": "^0.12.7",
2828
"mocha": "^2.1.0",
2929
"unit.js": "*",
30-
"request": "~2.53.0",
30+
"request": "~2.48.0",
3131
"lcov-parse": "0.0.9"
3232
},
3333
"devDependencies": {
@@ -41,4 +41,4 @@
4141
"scripts": {
4242
"test": "grunt test"
4343
}
44-
}
44+
}

test/appc_coverage_test.js

+14-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
var test = require('unit.js'),
22
coverage = require('../lib/appc-coverage.js'),
3-
fs = require('fs');
3+
fs = require('fs'),
4+
dns = require('dns'),
5+
url = require('url');
46

57
describe('library', function () {
68
it('coverage.getOptions();', function () {
@@ -18,5 +20,15 @@ describe('library', function () {
1820
test.assert(JSON.stringify(data) === fs.readFileSync('./test/expected/coverage.txt').toString(), 'Files did not match');
1921
done();
2022
});
21-
})
23+
});
24+
});
25+
26+
describe('DNS Check', function() {
27+
it('dns.resolve(' + url.parse(coverage.SERVER).host + ')', function (done) {
28+
dns.resolve(url.parse(coverage.SERVER).host, function (err, addresses) {
29+
test.assert(err == null, err);
30+
test.assert(addresses.length > 0, 'No addresses found');
31+
done();
32+
});
33+
});
2234
});

0 commit comments

Comments
 (0)