Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Commit 7860c08

Browse files
StanleyGoldmanStanleyGoldman
committed
Merge branch 'features/octorun-js' into fixes/remove-octokit-and-octorun
# Conflicts: # script # src/GitHub.Api/Application/ApiClient.cs # src/GitHub.Api/Authentication/LoginManager.cs
2 parents 4d5478f + 7856043 commit 7860c08

File tree

16 files changed

+226
-187
lines changed

16 files changed

+226
-187
lines changed

‎octorun/src/api.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
var readlineSync = require("readline-sync");
21
var config = require("./configuration");
32
var octokitWrapper = require("./octokit");
43

@@ -69,7 +68,7 @@ ApiWrapper.prototype.publish = function (name, desc, private, organization, call
6968
description: desc,
7069
private: private
7170
}, function (error, result) {
72-
callback(error, (!result) ? null : result.data.git_url);
71+
callback(error, (!result) ? null : result.data.clone_url);
7372
});
7473
}
7574
else {
@@ -78,7 +77,7 @@ ApiWrapper.prototype.publish = function (name, desc, private, organization, call
7877
description: desc,
7978
private: private
8079
}, function (error, result) {
81-
callback(error, (!result) ? null : result.data.git_url);
80+
callback(error, (!result) ? null : result.data.clone_url);
8281
});
8382
}
8483
};

‎octorun/src/bin/app-login.js

Lines changed: 8 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
var commander = require("commander");
2-
var package = require('../../package.json')
3-
var authentication = require('../authentication')
2+
var package = require('../../package.json');
3+
var authentication = require('../authentication');
44
var endOfLine = require('os').EOL;
5+
var output = require('../output');
56

67
commander
78
.version(package.version)
@@ -13,22 +14,10 @@ var encoding = 'utf-8';
1314
if (commander.twoFactor) {
1415
var handleTwoFactorAuthentication = function (username, password, token) {
1516
authentication.handleTwoFactorAuthentication(username, password, token, function (token) {
16-
process.stdout.write("success");
17-
process.stdout.write(endOfLine);
18-
process.stdout.write(token);
19-
process.stdout.write(endOfLine);
17+
output.success(token);
2018
process.exit();
2119
}, function (error) {
22-
process.stdout.write("error");
23-
process.stdout.write(endOfLine);
24-
process.stdout.write("");
25-
process.stdout.write(endOfLine);
26-
27-
if (error) {
28-
process.stdout.write(error.toString());
29-
process.stdout.write(endOfLine);
30-
}
31-
20+
output.error(error);
3221
process.exit();
3322
});
3423
}
@@ -68,27 +57,13 @@ else {
6857
var handleBasicAuthentication = function (username, password) {
6958
authentication.handleBasicAuthentication(username, password,
7059
function (token) {
71-
process.stdout.write("success");
72-
process.stdout.write(endOfLine);
73-
process.stdout.write(token);
74-
process.stdout.write(endOfLine);
60+
output.success(token);
7561
process.exit();
7662
}, function () {
77-
process.stdout.write("2fa");
78-
process.stdout.write(endOfLine);
79-
process.stdout.write(password);
80-
process.stdout.write(endOfLine);
63+
output.custom("2fa", password);
8164
process.exit();
8265
}, function (error) {
83-
process.stdout.write("error");
84-
process.stdout.write(endOfLine);
85-
process.stdout.write("");
86-
process.stdout.write(endOfLine);
87-
88-
if (error) {
89-
process.stdout.write(error.toString());
90-
process.stdout.write(endOfLine);
91-
}
66+
output.error(error);
9267
process.exit();
9368
});
9469
}

‎octorun/src/bin/app-organizations.js

Lines changed: 9 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
var commander = require("commander");
2-
var package = require('../../package.json')
3-
var ApiWrapper = require('../api')
2+
var package = require('../../package.json');
3+
var ApiWrapper = require('../api');
44
var endOfLine = require('os').EOL;
5+
var output = require('../output');
56

67
commander
78
.version(package.version)
@@ -12,39 +13,22 @@ try {
1213
var apiWrapper = new ApiWrapper();
1314
apiWrapper.getOrgs(function (error, result) {
1415
if (error) {
15-
process.stdout.write("error");
16-
process.stdout.write(endOfLine);
17-
18-
if (error) {
19-
process.stdout.write(error.toString());
20-
process.stdout.write(endOfLine);
21-
}
22-
16+
output.error(error);
2317
process.exit();
2418
}
2519
else {
26-
process.stdout.write("success");
27-
process.stdout.write(endOfLine);
28-
20+
let results = [];
2921
for (var i = 0; i < result.length; i++) {
30-
process.stdout.write(result[i].name);
31-
process.stdout.write(endOfLine);
32-
process.stdout.write(result[i].login);
33-
process.stdout.write(endOfLine);
22+
results.push(result[i].name);
23+
results.push(result[i].login);
3424
}
3525

26+
output.success(results);
3627
process.exit();
3728
}
3829
});
3930
}
4031
catch (error) {
41-
process.stdout.write("Error");
42-
process.stdout.write(endOfLine);
43-
44-
if (error) {
45-
process.stdout.write(error.toString());
46-
process.stdout.write(endOfLine);
47-
}
48-
32+
output.error(error);
4933
process.exit();
5034
}

‎octorun/src/bin/app-validate.js

Lines changed: 6 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
var commander = require("commander");
2-
var package = require('../../package.json')
2+
var package = require('../../package.json');
33
var endOfLine = require('os').EOL;
4-
var ApiWrapper = require('../api')
4+
var ApiWrapper = require('../api');
5+
var output = require('../output');
56

67
commander
78
.version(package.version)
@@ -12,39 +13,16 @@ try {
1213

1314
apiWrapper.verifyUser(function (error, result) {
1415
if (error) {
15-
process.stdout.write("error");
16-
process.stdout.write(endOfLine);
17-
process.stdout.write("");
18-
process.stdout.write(endOfLine);
19-
process.stdout.write("");
20-
process.stdout.write(endOfLine);
21-
22-
if (error) {
23-
process.stdout.write(error.toString());
24-
process.stdout.write(endOfLine);
25-
}
26-
16+
output.error(error)
2717
process.exit();
2818
}
2919
else {
30-
process.stdout.write("success");
31-
process.stdout.write(endOfLine);
32-
process.stdout.write(result.name);
33-
process.stdout.write(endOfLine);
34-
process.stdout.write(result.login);
35-
process.stdout.write(endOfLine);
20+
output.success([result.name, result.login])
3621
process.exit();
3722
}
3823
});
3924
}
4025
catch (error) {
41-
process.stdout.write("Error");
42-
process.stdout.write(endOfLine);
43-
44-
if (error) {
45-
process.stdout.write(error.toString());
46-
process.stdout.write(endOfLine);
47-
}
48-
26+
output.error(error)
4927
process.exit();
5028
}

‎octorun/src/output.js

Lines changed: 70 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,70 @@
1+
var endOfLine = require('os').EOL;
2+
3+
var outputResult = function (status, results, errors) {
4+
process.stdout.write(status);
5+
process.stdout.write(endOfLine);
6+
7+
if (!results) {
8+
process.stdout.write("");
9+
process.stdout.write(endOfLine);
10+
}
11+
else {
12+
if (typeof results === 'string') {
13+
process.stdout.write(results);
14+
process.stdout.write(endOfLine);
15+
}
16+
else if (Array.isArray(results)) {
17+
for (var resultIndex = 0; resultIndex < results.length; resultIndex++) {
18+
var result = results[resultIndex];
19+
if (typeof result !== 'string') {
20+
throw "Unsupported result output";
21+
}
22+
23+
process.stdout.write(result);
24+
process.stdout.write(endOfLine);
25+
}
26+
}
27+
28+
throw "Unsupported result output";
29+
}
30+
31+
if (errors) {
32+
if (typeof errors === 'string') {
33+
process.stdout.write(errors);
34+
process.stdout.write(endOfLine);
35+
}
36+
else if (Array.isArray(errors)) {
37+
for (var errorIndex = 0; errorIndex < errors.length; errorIndex++) {
38+
var error = errors[errorIndex];
39+
if (typeof error !== 'string') {
40+
throw "Unsupported result output";
41+
}
42+
43+
process.stdout.write(error);
44+
process.stdout.write(endOfLine);
45+
}
46+
}
47+
else {
48+
process.stdout.write(errors);
49+
process.stdout.write(endOfLine);
50+
}
51+
}
52+
}
53+
54+
var outputSuccess = function (results) {
55+
outputResult("success", results);
56+
}
57+
58+
var outputCustom = function (status, results) {
59+
outputResult(status, results);
60+
}
61+
62+
var outputError = function (errors) {
63+
outputResult("error", null, errors);
64+
}
65+
66+
module.exports = {
67+
success: outputSuccess,
68+
custom: outputCustom,
69+
error: outputError
70+
};

‎script

0 commit comments

Comments
 (0)