Merge ~newell-jensen/maas:lp1760715 into maas:master

[Newell Jensen]

LP: #1760715 -- Fixes how partitions are being updated by calling updateFilesystem instead of updateDisk.

[MAAS Lander (maas-lander)]

UNIT TESTS
-b lp1760715 lp:~newell-jensen/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 077fd818420e134f572025cdbc399f16183a6e50

review: Approve

[Blake Rouse (blake-rouse)]

Looks good.

review: Approve

diff --git a/src/maasserver/static/js/angular/controllers/node_details_storage.js b/src/maasserver/static/js/angular/controllers/node_details_storage.js
index 0f4d120..4816e7a 100644
--- a/src/maasserver/static/js/angular/controllers/node_details_storage.js
+++ b/src/maasserver/static/js/angular/controllers/node_details_storage.js
@@ -1,4 +1,4 @@
-/* Copyright 2015-2017 Canonical Ltd.  This software is licensed under the
+/* Copyright 2015-2018 Canonical Ltd.  This software is licensed under the
  * GNU Affero General Public License version 3 (see the file LICENSE).
  *
  * MAAS Node Storage Controller
@@ -826,7 +826,7 @@ angular.module('MAAS').controller('NodeStorageController', [
             return available;
         };
 
-        // Update the currect mode for the available section and the all
+        // Update the current mode for the available section and the all
         // selected value.
         $scope.updateAvailableSelection = function(force) {
             if(angular.isUndefined(force)) {
@@ -1066,11 +1066,6 @@ angular.module('MAAS').controller('NodeStorageController', [
             var params = {
                 name: disk.name
             };
-            // Only take the disk's name if it is a parition as
-            // these params are passed to update_disk.
-            if(disk.type === "partition") {
-                params.name = params.name.split('-')[0];
-            }
 
             // Do nothing if not valid.
             if($scope.isNameInvalid(disk) ||
@@ -1103,8 +1098,15 @@ angular.module('MAAS').controller('NodeStorageController', [
             }
 
             // Save the options.
-            MachinesManager.updateDisk(
-                $scope.node, disk.block_id, params);
+            if(disk.type === "partition") {
+                MachinesManager.updateFilesystem(
+                    $scope.node, disk.block_id, disk.partition_id,
+                    params.fstype, params.mount_point,
+                    params.mount_options, params.tags);
+            } else {
+                MachinesManager.updateDisk(
+                    $scope.node, disk.block_id, params);
+            }
 
             // Set the options on the object so no flicker occurs while waiting
             // for the new object to be received.
@@ -1114,7 +1116,7 @@ angular.module('MAAS').controller('NodeStorageController', [
             disk.tags = disk.$options.tags;
             disk.$options = {};
 
-            // If the mount_point is set the we need to transition this to
+            // If the mount_point is set then we need to transition this to
             // the filesystem section.
             if(angular.isString(disk.mount_point) && disk.mount_point !== "") {
                 $scope.filesystems.push({
diff --git a/src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js b/src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js
index dd13a80..39c8eb0 100644
--- a/src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js
+++ b/src/maasserver/static/js/angular/controllers/tests/test_node_details_storage.js
@@ -3075,7 +3075,7 @@ describe("NodeStorageController", function() {
 
         it("calls updateDisk with new name for logical volume", function() {
             var controller = makeController();
-            var name = "vg0-lvnew"
+            var name = "vg0-lvnew";
             var disk = {
                 name: name,
                 type: "virtual",
@@ -3098,26 +3098,26 @@ describe("NodeStorageController", function() {
             expect(MachinesManager.updateDisk).toHaveBeenCalled();
         });
 
-        it("calls updateDisk with new name for parition", function() {
+        it("calls updateFilesystem for partition", function() {
             var controller = makeController();
-            var name = "vda-part1";
+            var name = makeName("name");
             var disk = {
-                name: name,
+                name: "",
                 type: "partition",
-                block_id: makeInteger(0, 100),
-                partition_id: makeInteger(0, 100),
                 $options: {
-                    fstype: "",
-                    mountPoint: "",
-                    mountOptions: ""
+                    mountPoint: ""
                 },
+                original: {
+                    name: name
+                }
             };
-            spyOn(MachinesManager, "updateDisk");
+            spyOn(MachinesManager, "updateFilesystem");
 
             $scope.availableConfirmEdit(disk);
             expect(disk.name).toBe(name);
-            expect(MachinesManager.updateDisk).toHaveBeenCalled();
-         });
+            expect(MachinesManager.updateFilesystem).toHaveBeenCalled();
+        });
+
     });
 
     describe("canCreateBcache", function() {