test for not deleting draft submissions

getodk · Sep 13, 2024 · 9eb00f4 · 9eb00f4
1 parent 76e523b
commit 9eb00f4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git 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
@@ -507,7 +507,4 @@ describe('query module submission purge', () => {
         });
     }));
   });
-
-  // TODO
-  // should not delete a draft submission? or yes?
 });