Commit fefae13d authored by Andrew Fontaine's avatar Andrew Fontaine

Fix environment approvals

By convention, API calls take an id for project ID, not a projectId. E2E
tests would fix this.
parent 95b17b01
...@@ -114,7 +114,7 @@ export default { ...@@ -114,7 +114,7 @@ export default {
this.loading = true; this.loading = true;
this.show = false; this.show = false;
action({ action({
projectId: this.projectId, id: this.projectId,
deploymentId: this.upcomingDeployment.id, deploymentId: this.upcomingDeployment.id,
comment: this.comment, comment: this.comment,
}) })
......
...@@ -188,7 +188,7 @@ describe('ee/environments/components/environment_approval.vue', () => { ...@@ -188,7 +188,7 @@ describe('ee/environments/components/environment_approval.vue', () => {
await button.trigger('click'); await button.trigger('click');
expect(api).toHaveBeenCalledWith({ expect(api).toHaveBeenCalledWith({
projectId: '5', id: '5',
deploymentId: environment.upcomingDeployment.id, deploymentId: environment.upcomingDeployment.id,
comment: 'comment', comment: 'comment',
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment