Merge lp:~jtv/maas/bug-1376990-js-test-lint into lp:~maas-committers/maas/trunk

[Jeroen T. Vermeulen]

Fix some lint in a JavaScript test. The JS lint checker doesn't always show these things, but when it does, it makes development difficult.

Inconsistent lint reports mean that we have to jump in and fix lint when we see it — the author of the code will have checked for lint but not gotten any reports.

Another part of the lint fix is to follow. The part you see here should be uncontroversial: only a test is affected, and it's a known browser compatibility pitfall — a trailing comma after the last element in an array or object breaks hard in IE (or at least it used to, not all that long ago).

Jeroen

[Blake Rouse (blake-rouse)]

review: Approve

[Jeroen T. Vermeulen (jtv)]

Thanks.

[Christian Reis (kiko)]

Land away

review: Approve

=== modified file 'src/maasserver/static/js/tests/test_image_views.js'
--- src/maasserver/static/js/tests/test_image_views.js	2014-10-02 15:40:55 +0000
+++ src/maasserver/static/js/tests/test_image_views.js	2014-10-03 03:01:56 +0000
@@ -180,7 +180,7 @@
                 status: "Complete",
                 downloading: false,
                 numberOfNodes: 1,
-                lastUpdate: '10/1/14',
+                lastUpdate: '10/1/14'
             },
             {
                 id: 2,
@@ -193,7 +193,7 @@
                 status: "Complete",
                 downloading: false,
                 numberOfNodes: 0,
-                lastUpdate: '10/1/14',
+                lastUpdate: '10/1/14'
             },
             {
                 id: 3,
@@ -206,8 +206,8 @@
                 status: "Downloading 13%",
                 downloading: true,
                 numberOfNodes: 0,
-                lastUpdate: '10/1/14',
-            },
+                lastUpdate: '10/1/14'
+            }
         ];
     },
 
@@ -327,12 +327,12 @@
             {
                 id: 1,
                 rtype: Y.maas.enums.BOOT_RESOURCE_TYPE.SYNCED,
-                name: "centos/centos65",
+                name: "centos/centos65"
             },
             {
                 id: 2,
                 rtype: Y.maas.enums.BOOT_RESOURCE_TYPE.SYNCED,
-                name: "centos/centos70",
+                name: "centos/centos70"
             }
         ];
         var view = this.makeImagesView(false, false, none_ubuntu_images);