refactor(agent): refactor agentService to es6 (#4091)

app/agent/services/agentService.js

@@ -1,50 +1,35 @@
+import angular from 'angular';
+
 import { AgentViewModel } from '../models/agent';
 
-angular.module('portainer.agent').factory('AgentService', [
+angular.module('portainer.agent').factory('AgentService', AgentServiceFactory);
-  '$q',
-  'Agent',
-  'AgentVersion1',
-  'HttpRequestHelper',
-  'Host',
-  'StateManager',
-  function AgentServiceFactory($q, Agent, AgentVersion1, HttpRequestHelper, Host, StateManager) {
-    'use strict';
-    var service = {};
 
-    service.agents = agents;
+function AgentServiceFactory(Agent, AgentVersion1, HttpRequestHelper, Host, StateManager) {
-    service.hostInfo = hostInfo;
+  return {
+    agents,
+    hostInfo,
+  };
 
-    function getAgentApiVersion() {
+  function getAgentApiVersion() {
-      var state = StateManager.getState();
+    const state = StateManager.getState();
-      return state.endpoint.agentApiVersion;
+    return state.endpoint.agentApiVersion;
+  }
+
+  function hostInfo(nodeName) {
+    HttpRequestHelper.setPortainerAgentTargetHeader(nodeName);
+    return Host.info().$promise;
+  }
+
+  async function agents() {
+    const agentVersion = getAgentApiVersion();
+    const service = agentVersion > 1 ? Agent : AgentVersion1;
+    try {
+      const agents = await service.query({ version: agentVersion }).$promise;
+      return agents.map(function (item) {
+        return new AgentViewModel(item);
+      });
+    } catch (err) {
+      throw { msg: 'Unable to retrieve agents', err };
     }
-
+  }
-    function hostInfo(nodeName) {
+}
-      HttpRequestHelper.setPortainerAgentTargetHeader(nodeName);
-      return Host.info().$promise;
-    }
-
-    function agents() {
-      var deferred = $q.defer();
-
-      var agentVersion = getAgentApiVersion();
-      var service = agentVersion > 1 ? Agent : AgentVersion1;
-
-      service
-        .query({ version: agentVersion })
-        .$promise.then(function success(data) {
-          var agents = data.map(function (item) {
-            return new AgentViewModel(item);
-          });
-          deferred.resolve(agents);
-        })
-        .catch(function error(err) {
-          deferred.reject({ msg: 'Unable to retrieve agents', err: err });
-        });
-
-      return deferred.promise;
-    }
-
-    return service;
-  },
-]);