biz-a-cli 2.3.26 → 2.3.28

package/bin/app.js

@@ -312,7 +312,7 @@ yargs(process.argv.slice(2))
         })()
     }
 )
-.command('$0', 'the default command', () => {}, () => {
-    console.error('Command not found. Please make sure the commands use all lowercase letters.')
-})
+.recommendCommands()
+.demandCommand(1, 'You need at least one command before moving on')
+.strict()
 .parse();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "biz-a-cli",
   "nameDev": "biz-a-cli-dev",
-  "version": "2.3.26",
+  "version": "2.3.28",
   "versionDev": "0.0.30",
   "description": "",
   "main": "bin/index.js",

package/readme.md

@@ -23,7 +23,7 @@
     BizA add -s [Domain] -i [dbindex] -d [full or relative path to app folder]
 
     Example:
-        BizA add-s https://my.domain.com -i 2 -d “c:\biza templates\imamatek”
+        BizA add -s https://my.domain.com -i 2 -d “c:\biza templates\imamatek”
 
 > [!TIP]
 >    BizA add --help

package/tests/app.test.js

@@ -4,6 +4,7 @@ import { Duplex } from 'node:stream'
 import { finished } from 'node:stream/promises'
 import { createDecipheriv } from 'node:crypto'
 import * as tar from "tar"
+import { env } from "../envs/env.js"
 
 describe('Biz-A Apps CLI', ()=>{
 
@@ -162,7 +163,7 @@ describe('Biz-A Apps CLI', ()=>{
             expect(axios.get.mock.calls).toHaveLength(1)
             const getArgs = axios.get.mock.calls[0]
             expect(getArgs).toHaveLength(2)
-            expect(getArgs[0]).toBe('http://localhost:3000/api/issuerKey')
+            expect(getArgs[0]).toBe(`${env.BIZA_SERVER_LINK}/api/issuerKey`)
             expect(getArgs[1]).toHaveProperty('params')
             const params = getArgs[1].params
             expect(params).toHaveProperty('data')
@@ -333,7 +334,7 @@ describe('Biz-A Apps CLI', ()=>{
             expectedFn()
         })
 
-        it.only('Shall not allow empty app', async ()=>{
+        it('Shall not allow empty app', async ()=>{
             mockValidTemplates({})
             mockIssuerKeyResponse()
             await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
@@ -344,17 +345,29 @@ describe('Biz-A Apps CLI', ()=>{
             expect(errorSpy.mock.calls[0][0]).toBe('Nothing to upload. Please recheck your app folder.')
         })
 
-        it.only('Shall use lowercase letters for commands', async ()=>{
-            mockValidTemplates({})
-            mockIssuerKeyResponse()
+        it('Shall recommend command', async ()=>{
+            const exitSpy = jest.spyOn(process, 'exit').mockImplementation(()=>{})
             await runCommand('Add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
-
             expect(logSpy.mock.calls.length).toBe(0)
+            expect(exitSpy).toHaveBeenCalledWith(1) // exit code = 1
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('Did you mean add?')
+        })
 
-            expect(errorSpy.mock.calls.length).toBe(1)
-            expect(errorSpy.mock.calls[0][0]).toBe('Command not found. Please make sure the commands use all lowercase letters.')
+        it('Shall have minimun one command', async ()=>{
+            const exitSpy = jest.spyOn(process, 'exit').mockImplementation()
+            await runCommand('-s', 'https://a.b.c', '-i', 2)
+            expect(logSpy.mock.calls.length).toBe(0)
+            expect(exitSpy).toHaveBeenCalledWith(1)
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('You need at least one command before moving on')
         })
 
+        it('Shall use strict command and options', async ()=>{
+            const exitSpy = jest.spyOn(process, 'exit').mockImplementation()
+            await runCommand('Unknown Command', '-s', 'https://a.b.c', '-i', 2)
+            expect(logSpy.mock.calls.length).toBe(0)
+            expect(exitSpy).toHaveBeenCalledWith(1)
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('Unknown argument: Unknown Command')
+        })
     })
 
     describe('Remove App', ()=>{