replace usage of all and spread

This commit is contained in:
Jake McDermott
2018-02-13 16:40:20 -05:00
parent 0ff94c63f2
commit bf0683f7fe
3 changed files with 25 additions and 31 deletions

View File

@@ -77,6 +77,4 @@ module.exports = {
post, post,
patch, patch,
put, put,
all: axios.all,
spread: axios.spread
}; };

View File

@@ -3,10 +3,8 @@ import uuid from 'uuid';
import { AWX_E2E_PASSWORD } from './settings'; import { AWX_E2E_PASSWORD } from './settings';
import { import {
all,
get, get,
post, post,
spread
} from './api'; } from './api';
const session = `e2e-${uuid().substr(0, 8)}`; const session = `e2e-${uuid().substr(0, 8)}`;
@@ -83,14 +81,14 @@ const getInventorySource = (namespace = session) => {
getInventoryScript(namespace) getInventoryScript(namespace)
]; ];
return all(promises) return Promise.all(promises)
.then(spread((inventory, inventoryScript) => getOrCreate('/inventory_sources/', { .then(([inventory, inventoryScript]) => getOrCreate('/inventory_sources/', {
name: `${namespace}-inventory-source-custom`, name: `${namespace}-inventory-source-custom`,
description: namespace, description: namespace,
source: 'custom', source: 'custom',
inventory: inventory.id, inventory: inventory.id,
source_script: inventoryScript.id source_script: inventoryScript.id
}))); }));
}; };
const getAdminAWSCredential = (namespace = session) => { const getAdminAWSCredential = (namespace = session) => {
@@ -101,8 +99,8 @@ const getAdminAWSCredential = (namespace = session) => {
}) })
]; ];
return all(promises) return Promise.all(promises)
.then(spread((me, credentialType) => { .then(([me, credentialType]) => {
const [admin] = me.data.results; const [admin] = me.data.results;
return getOrCreate('/credentials/', { return getOrCreate('/credentials/', {
@@ -116,7 +114,7 @@ const getAdminAWSCredential = (namespace = session) => {
security_token: 'AAAAAAAAAAAAAAAA' security_token: 'AAAAAAAAAAAAAAAA'
} }
}); });
})); });
}; };
const getAdminMachineCredential = (namespace = session) => { const getAdminMachineCredential = (namespace = session) => {
@@ -125,17 +123,16 @@ const getAdminMachineCredential = (namespace = session) => {
getOrCreate('/credential_types/', { name: 'Machine' }) getOrCreate('/credential_types/', { name: 'Machine' })
]; ];
return all(promises) return Promise.all(promises)
.then(spread((me, credentialType) => { .then(([me, credentialType]) => {
const [admin] = me.data.results; const [admin] = me.data.results;
return getOrCreate('/credentials/', { return getOrCreate('/credentials/', {
name: `${namespace}-credential-machine-admin`, name: `${namespace}-credential-machine-admin`,
description: namespace, description: namespace,
credential_type: credentialType.id, credential_type: credentialType.id,
user: admin.id user: admin.id
}); });
})); });
}; };
const getTeam = (namespace = session) => getOrganization(namespace) const getTeam = (namespace = session) => getOrganization(namespace)
@@ -227,15 +224,15 @@ const getJobTemplate = (namespace = session) => {
getUpdatedProject(namespace) getUpdatedProject(namespace)
]; ];
return all(promises) return Promise.all(promises)
.then(spread((inventory, credential, project) => getOrCreate('/job_templates/', { .then(([inventory, credential, project]) => getOrCreate('/job_templates/', {
name: `${namespace}-job-template`, name: `${namespace}-job-template`,
description: namespace, description: namespace,
inventory: inventory.id, inventory: inventory.id,
credential: credential.id, credential: credential.id,
project: project.id, project: project.id,
playbook: 'hello_world.yml', playbook: 'hello_world.yml',
}))); }));
}; };
const getWorkflowTemplate = (namespace = session) => { const getWorkflowTemplate = (namespace = session) => {
@@ -256,8 +253,8 @@ const getWorkflowTemplate = (namespace = session) => {
getJobTemplate(namespace), getJobTemplate(namespace),
]; ];
const workflowNodePromise = all(resources) const workflowNodePromise = Promise.all(resources)
.then(spread((workflowTemplate, source, project, jobTemplate) => { .then(([workflowTemplate, source, project, jobTemplate]) => {
const workflowNodes = workflowTemplate.related.workflow_nodes; const workflowNodes = workflowTemplate.related.workflow_nodes;
const unique = 'unified_job_template'; const unique = 'unified_job_template';
@@ -267,16 +264,17 @@ const getWorkflowTemplate = (namespace = session) => {
getOrCreate(workflowNodes, { [unique]: source.id }, [unique]), getOrCreate(workflowNodes, { [unique]: source.id }, [unique]),
]; ];
const createSuccessNodes = (projectNode, jobNode, sourceNode) => all([ const createSuccessNodes = ([projectNode, jobNode, sourceNode]) => Promise.all([
getOrCreate(projectNode.related.success_nodes, { id: jobNode.id }), getOrCreate(projectNode.related.success_nodes, { id: jobNode.id }),
getOrCreate(jobNode.related.success_nodes, { id: sourceNode.id }), getOrCreate(jobNode.related.success_nodes, { id: sourceNode.id }),
]); ]);
return all(nodes).then(spread(createSuccessNodes)); return Promise.all(nodes)
})); .then(createSuccessNodes);
});
return all([workflowTemplatePromise, workflowNodePromise]) return Promise.all([workflowTemplatePromise, workflowNodePromise])
.then(spread((workflowTemplate, nodes) => workflowTemplate)); .then(([workflowTemplate, nodes]) => workflowTemplate);
}; };
const getAuditor = (namespace = session) => getOrganization(namespace) const getAuditor = (namespace = session) => getOrganization(namespace)
@@ -320,10 +318,10 @@ const getJobTemplateAdmin = (namespace = session) => {
})); }));
const assignRolePromise = Promise.all([userPromise, rolePromise]) const assignRolePromise = Promise.all([userPromise, rolePromise])
.then(spread((user, role) => post(`/api/v2/roles/${role.id}/users/`, { id: user.id }))); .then(([user, role]) => post(`/api/v2/roles/${role.id}/users/`, { id: user.id }));
return Promise.all([userPromise, assignRolePromise]) return Promise.all([userPromise, assignRolePromise])
.then(spread(user => user)); .then(([user, assignment]) => user);
}; };
const getProjectAdmin = (namespace = session) => { const getProjectAdmin = (namespace = session) => {
@@ -343,10 +341,10 @@ const getProjectAdmin = (namespace = session) => {
})); }));
const assignRolePromise = Promise.all([userPromise, rolePromise]) const assignRolePromise = Promise.all([userPromise, rolePromise])
.then(spread((user, role) => post(`/api/v2/roles/${role.id}/users/`, { id: user.id }))); .then(([user, role]) => post(`/api/v2/roles/${role.id}/users/`, { id: user.id }));
return Promise.all([userPromise, assignRolePromise]) return Promise.all([userPromise, assignRolePromise])
.then(spread(user => user)); .then(([user, assignment]) => user);
}; };
const getInventorySourceSchedule = (namespace = session) => getInventorySource(namespace) const getInventorySourceSchedule = (namespace = session) => getInventorySource(namespace)

View File

@@ -1,5 +1,3 @@
import { all } from '../api';
import { import {
getAdminAWSCredential, getAdminAWSCredential,
getAdminMachineCredential, getAdminMachineCredential,
@@ -49,7 +47,7 @@ module.exports = {
getUpdatedProject().then(obj => { data.project = obj; }) getUpdatedProject().then(obj => { data.project = obj; })
]; ];
all(promises) Promise.all(promises)
.then(() => { .then(() => {
client.useCss(); client.useCss();