Add support for setting external app user ID (#153)

Author: Matt Willer

src/commands/users/create.js

@@ -55,6 +55,9 @@ class UsersCreateCommand extends BoxCommand {
 		if (flags['avatar-url']) {
 			options.avatar_url = flags['avatar-url'];
 		}
+		if (flags['external-id']) {
+			options.external_app_user_id = flags['external-id'];
+		}
 
 		if (flags['app-user']) {
 			user = await this.client.enterprise.addAppUser(args.name, options);
@@ -74,12 +77,16 @@ UsersCreateCommand.flags = {
 	'app-user': flags.boolean({
 		description: 'Set this user as an app user'
 	}),
+	'external-id': flags.string({
+		description: 'External ID for app users',
+		dependsOn: ['app-user'],
+	}),
 	'id-only': flags.boolean({
 		description: 'Return only an ID to output from this command'
 	}),
 	'sync-enable': flags.boolean({
 		description: 'Enable Box Sync for this user',
-		exclusion: ['sync-disable'],
+		exclusive: ['sync-disable'],
 		allowNo: true
 	}),
 	'exempt-from-device-limits': flags.boolean({

src/commands/users/update.js

@@ -62,6 +62,9 @@ class UsersUpdateCommand extends BoxCommand {
 		if (flags.name) {
 			updates.name = flags.name;
 		}
+		if (flags['external-id']) {
+			updates.external_app_user_id = flags['external-id'];
+		}
 
 		let user = await this.client.users.update(args.id, updates);
 		await this.output(user);
@@ -143,6 +146,9 @@ UsersUpdateCommand.flags = {
 	timezone: flags.string({ description: 'The user\'s timezone. Input format follows tz database timezones' }),
 	'avatar-url': flags.string({ description: 'URL of the user\'s avatar image' }),
 	login: flags.string({ description: 'Change the user\'s primary email address used for logging into Box '}),
+	'external-id': flags.string({
+		description: 'External ID for app users',
+	}),
 };
 
 UsersUpdateCommand.args = [

test/commands/users.test.js

@@ -706,18 +706,23 @@ describe('Users', () => {
 
 		test
 			.nock(TEST_API_ROOT, api => api
-				.post('/2.0/users', { name, is_platform_access_only: true })
+				.post('/2.0/users', {
+					name,
+					is_platform_access_only: true,
+					external_app_user_id: 'foo',
+				})
 				.reply(201, fixture)
 			)
 			.stdout()
 			.command([
 				'users:create',
 				name,
 				'--app-user',
+				'--external-id=foo',
 				'--id-only',
 				'--token=test'
 			])
-			.it('should create a new app user when --app-user flag is passed', ctx => {
+			.it('should create a new app user with external ID when App User flags are passed', ctx => {
 				assert.equal(ctx.stdout, `${JSON.parse(fixture).id}${os.EOL}`);
 			});
 
@@ -863,6 +868,10 @@ describe('Users', () => {
 			'name flag': [
 				'--name=Bob Smith',
 				{name: 'Bob Smith'}
+			],
+			'external ID flag': [
+				'--external-id=foo',
+				{external_app_user_id: 'foo'}
 			]
 		}, function(flag, body) {