tech(script): fix parsing param in delete and anonymise script

1024pix · Nov 20, 2024 · 2387600 · 2387600
1 parent d0df70b
commit 2387600
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/api/scripts/prod/delete-and-anonymise-organization-learners.js b/api/scripts/prod/delete-and-anonymise-organization-learners.js
@@ -12,7 +12,7 @@ export class DeleteAndAnonymiseOrgnizationLearnerScript extends Script {
       permanent: true,
       options: {
         organizationLearnerIds: {
-          type: '<array>number',
+          type: 'string',
           describe: 'a list of comma separated organization learner ids',
           demandOption: true,
           coerce: commaSeparatedNumberParser(),
@@ -28,7 +28,6 @@ export class DeleteAndAnonymiseOrgnizationLearnerScript extends Script {
     organizationLearnerRepository = { removeByIds },
   }) {
     const engineeringUserId = process.env.ENGINEERING_USER_ID;
-
     logger.info(`Anonymise ${options.organizationLearnerIds.length} learners`);
     await DomainTransaction.execute(async () => {
       await campaignParticipationRepository.removeByOrganizationLearnerIds({

diff --git a/api/tests/integration/scripts/prod/delete-and-anonymise-organization-learners_test.js b/api/tests/integration/scripts/prod/delete-and-anonymise-organization-learners_test.js
@@ -11,11 +11,10 @@ describe('DeleteAndAnonymiseOrgnizationLearnerScript', function () {
       const { options } = script.metaInfo;
 
       expect(options.organizationLearnerIds).to.deep.include({
-        type: '<array>number',
+        type: 'string',
         describe: 'a list of comma separated organization learner ids',
         demandOption: true,
       });
-      expect(options.organizationLearnerIds.coerce).to.be.a('function');
     });
 
     it('parses list of organizationLearnerIds', async function () {