Merge pull request #768 from particle-iot/feature/sc-129604/skip-prot…

…ected-devices-on-non-protected-products

Skip adding protected devices to non-protection products

src/cmd/product.js

@@ -42,8 +42,8 @@ module.exports = class ProductCommand extends CLICommandBase {
 			.then(result => this.showDeviceAddResult(result));
 	}
 
-	showDeviceAddResult({ product, identifiers, status: { invalidDeviceIds = [], nonmemberDeviceIds = [] } } = {}){
-		identifiers = filterDeviceIdentifiers(identifiers, invalidDeviceIds, nonmemberDeviceIds);
+	showDeviceAddResult({ product, identifiers, status: { invalidDeviceIds = [], nonmemberDeviceIds = [], protectedDeviceIds = [] } } = {}){
+		identifiers = filterDeviceIdentifiers(identifiers, invalidDeviceIds, nonmemberDeviceIds, protectedDeviceIds);
 		const message = [];
 
 		if (identifiers.length){
@@ -75,6 +75,14 @@ module.exports = class ProductCommand extends CLICommandBase {
 			);
 		}
 
+		if (protectedDeviceIds.length){
+			message.push(
+				'Skipped Protected IDs:',
+				dedupeAndStringifyIDList(protectedDeviceIds),
+				''
+			);
+		}
+
 		if (!identifiers.length){
 			throw new Error(message.join(os.EOL));
 		}
@@ -200,9 +208,10 @@ function readDeviceListFile(file){
 		});
 }
 
-function filterDeviceIdentifiers(identifiers, invalid = [], nonmember = []){
+function filterDeviceIdentifiers(identifiers, invalid = [], nonmember = [], protectedDevice = []){
 	return identifiers.reduce((out, x) => {
-		if (!hasDeviceIdentifier(x, invalid) && !hasDeviceIdentifier(x, nonmember)){
+		if (!hasDeviceIdentifier(x, invalid) && !hasDeviceIdentifier(x, nonmember)
+			&& !hasDeviceIdentifier(x, protectedDevice)){
 			out.push(x);
 		}
 		return out;