diff --git a/test/integration/api/submissions.js b/test/integration/api/submissions.js index d47f379d7..2db6b0a96 100644 --- a/test/integration/api/submissions.js +++ b/test/integration/api/submissions.js @@ -1425,6 +1425,18 @@ describe('api: /forms/:id/submissions', () => { .expect(200)) .then(() => asAlice.get('/v1/projects/1/forms/simple/submissions/one') .expect(404))))); + + it('should not let a draft submission be deleted', testService(async (service) => { + const asAlice = await service.login('alice'); + await asAlice.post('/v1/projects/1/forms/simple/draft'); + await asAlice.post('/v1/projects/1/forms/simple/draft/submissions') + .send(testData.instances.simple.one) + .set('Content-Type', 'application/xml') + .expect(200); + // draft submission delete resource does not exist + await asAlice.delete('/v1/projects/1/forms/simple/draft/submissions/one') + .expect(404); + })); }); describe('/:instanceId RESTORE', () => { diff --git a/test/integration/other/submission-purging.js b/test/integration/other/submission-purging.js index dcc8462b4..f702e96d1 100644 --- a/test/integration/other/submission-purging.js +++ b/test/integration/other/submission-purging.js @@ -507,7 +507,4 @@ describe('query module submission purge', () => { }); })); }); - - // TODO - // should not delete a draft submission? or yes? });