Skip to content

Commit 5219269

Browse files
authored
fix: correct case on commands for case-sensitive filesystems (#235)
This corrects the renaming of command modules for case-sensitive filesystems.
1 parent ffd08ca commit 5219269

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

63 files changed

+2822
-2158
lines changed

.npmignore

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,8 @@ test.zip
55
testdir.zip
66
.idea
77
*.iml
8+
notes--DO-NOT-COMMIT.txt
9+
10+
**/*.*~
11+
**/#*.*#
12+
**/node_modules/*

lib/commands/addEntryToKVM.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ var descriptor = defaults.defaultDescriptor({
3838
module.exports.descriptor = descriptor;
3939

4040
module.exports.run = function(opts, cb) {
41+
options.validateSync(opts, descriptor);
4142
if (opts.debug) {
4243
console.log('addEntryToKVM: %j', opts);
4344
}

lib/commands/assignUserRole.js

Lines changed: 24 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,45 @@
11
/* jshint node: true */
2-
'use strict';
2+
"use strict";
33

4-
const util = require('util'),
5-
defaults = require('../defaults'),
6-
options = require('../options'),
7-
command_utils = require('./command-utils');
4+
const util = require("util"),
5+
defaults = require("../defaults"),
6+
options = require("../options"),
7+
command_utils = require("./command-utils");
88

99
var descriptor = defaults.defaultDescriptor({
1010
roleName: {
11-
name: 'Role Name',
11+
name: "Role Name",
1212
required: true,
1313
prompt: true
1414
},
1515
email: {
16-
name: 'Developer email',
16+
name: "Developer email",
1717
required: true,
1818
prompt: true
1919
}
2020
});
2121

2222
module.exports.descriptor = descriptor;
2323

24-
module.exports.run = function(opts, cb) {
24+
module.exports.run = function (opts, cb) {
25+
options.validateSync(opts, descriptor);
2526
if (opts.debug) {
26-
console.log('assignUserRole: %j', opts);
27+
console.log("assignUserRole: %j", opts);
2728
}
2829

29-
let body = util.format('id=%s', encodeURIComponent(opts.email));
30-
let uri = util.format('%s/v1/o/%s/userroles/%s/users', opts.baseuri, opts.organization, opts.roleName);
30+
let body = util.format("id=%s", encodeURIComponent(opts.email));
31+
let uri = util.format(
32+
"%s/v1/o/%s/userroles/%s/users",
33+
opts.baseuri,
34+
opts.organization,
35+
opts.roleName
36+
);
3137
let requestOptions = {
32-
uri,
33-
method:'POST',
34-
headers: { 'Content-Type': 'application/x-www-form-urlencoded' },
35-
body,
36-
json:true
37-
};
38-
command_utils.run('assignUserRole', opts,requestOptions, cb);
38+
uri,
39+
method: "POST",
40+
headers: { "Content-Type": "application/x-www-form-urlencoded" },
41+
body,
42+
json: true
43+
};
44+
command_utils.run("assignUserRole", opts, requestOptions, cb);
3945
};

lib/commands/attachFlowHook.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ var descriptor = defaults.defaultDescriptor({
2828
module.exports.descriptor = descriptor;
2929

3030
module.exports.run = function(opts, cb) {
31+
options.validateSync(opts, descriptor);
3132
if (opts.debug) {
3233
console.log('attachFlowHook: %j', opts);
3334
}

lib/commands/clearCache.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
const util = require('util'),
22
defaults = require('../defaults'),
3+
options = require('../options'),
34
command_utils = require('./command-utils');
45

56
var descriptor = defaults.defaultDescriptor({
@@ -19,6 +20,7 @@ var descriptor = defaults.defaultDescriptor({
1920
module.exports.descriptor = descriptor;
2021

2122
module.exports.run = function(opts, cb) {
23+
options.validateSync(opts, descriptor);
2224
if (opts.debug) {
2325
console.log('clearCache: %j', opts);
2426
}

0 commit comments

Comments
 (0)