Merge pull request #21 from electricimp/develop

v1.4.0

Author: Pavel Petroshenko

lib/DeviceGroups.js

@@ -43,7 +43,8 @@ class DeviceGroups extends Entities {
         };
         this._validCreateAttributes = {
             name : true,
-            description : false
+            description : false,
+            region : false
         };
         this._validUpdateAttributes = {
             name : true,
@@ -141,6 +142,9 @@ class DeviceGroups extends Entities {
     //                                     this must be unique for all DeviceGroups in this Product
     //                                 'description' (String, optional) - an optional free-form 
     //                                     description of the Device Group
+    //                                 'region' (String, optional) - a Device Group's region,
+    //                                     May be specified if the new Device Group is of the production 
+    //                                     or pre_production type only
     //     productionTarget :      Optional production_target relationship of the Device Group to be
     //         Object              created. Must be specified for pre_factoryfixture and factoryfixture device groups.
     //                             The valid keys are:
@@ -152,9 +156,17 @@ class DeviceGroups extends Entities {
     // Returns:                    Promise that resolves when the Device Group is successfully created, 
     //                             or rejects with an error
     create(productId, type, attributes, productionTarget = null) {
-        const error = ParamsChecker.validateNonEmpty(productId) ||
+        let error = ParamsChecker.validateNonEmpty(productId) ||
             DeviceGroups.validateType(type, 'type') ||
             this._validateProductionTarget(type, productionTarget, true);
+        if (!error && 'region' in attributes &&
+            !(type == DeviceGroups.TYPE_PRE_PRODUCTION || type == DeviceGroups.TYPE_PRODUCTION)) {
+            error = new Errors.InvalidDataError(
+                Util.format(
+                    'region may be specified for "%s" or "%s" device groups only',
+                    DeviceGroups.TYPE_PRE_PRODUCTION,
+                    DeviceGroups.TYPE_PRODUCTION));
+        }
         if (error) {
             return Promise.reject(error);
         }

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "imp-central-api",
-  "version": "1.3.1",
+  "version": "1.4.0",
   "description": "Electric Imp impCentral API v5 Client",
   "main": "index.js",
   "scripts": {