test configs

.drone.yml

@@ -0,0 +1,16 @@
+kind: pipeline
+type: docker
+name: compliance
+
+steps:
+- name: lint
+  image: node
+  commands:
+  - yarn install
+  - yarn lint
+- name: test
+  image: node
+  commands:
+  - yarn install
+  - yarn build
+  - yarn test

.eslintrc.json

@@ -66,7 +66,7 @@
         "handle-callback-err": "warn",
         "id-blacklist": "warn",
         "id-match": "warn",
-        "implicit-arrow-linebreak": "warn",
+        "implicit-arrow-linebreak": ["warn", "unix"],
         "indent": [
             "warn",
             4,

.gitignore

@@ -1,3 +1,4 @@
 build
 node_modules
-out.js
+out.js
+coverage

babel.config.js

@@ -0,0 +1,3 @@
+module.exports = {
+    presets: [[ '@babel/preset-env', { targets: { node: 'current' } }]]
+};

jest.config.js

@@ -0,0 +1,195 @@
+/*
+ * For a detailed explanation regarding each configuration property, visit:
+ * https://jestjs.io/docs/configuration
+ */
+
+module.exports = {
+    // All imported modules in your tests should be mocked automatically
+    // automock: false,
+
+    // Stop running tests after `n` failures
+    // bail: 0,
+
+    // The directory where Jest should store its cached dependency information
+    // cacheDirectory: "C:\\Users\\<User>\\AppData\\Local\\Temp\\jest",
+
+    // Automatically clear mock calls, instances, contexts and results before every test
+    clearMocks: true,
+
+    // Indicates whether the coverage information should be collected while executing the test
+    collectCoverage: true,
+
+    // An array of glob patterns indicating a set of files for which coverage information should be collected
+    // collectCoverageFrom: undefined,
+
+    // The directory where Jest should output its coverage files
+    coverageDirectory: "coverage",
+
+    // An array of regexp pattern strings used to skip coverage collection
+    // coveragePathIgnorePatterns: [
+    //   "\\\\node_modules\\\\"
+    // ],
+
+    // Indicates which provider should be used to instrument code for coverage
+    // coverageProvider: "babel",
+
+    // A list of reporter names that Jest uses when writing coverage reports
+    // coverageReporters: [
+    //   "json",
+    //   "text",
+    //   "lcov",
+    //   "clover"
+    // ],
+
+    // An object that configures minimum threshold enforcement for coverage results
+    // coverageThreshold: undefined,
+
+    // A path to a custom dependency extractor
+    // dependencyExtractor: undefined,
+
+    // Make calling deprecated APIs throw helpful error messages
+    // errorOnDeprecated: false,
+
+    // The default configuration for fake timers
+    // fakeTimers: {
+    //   "enableGlobally": false
+    // },
+
+    // Force coverage collection from ignored files using an array of glob patterns
+    // forceCoverageMatch: [],
+
+    // A path to a module which exports an async function that is triggered once before all test suites
+    // globalSetup: undefined,
+
+    // A path to a module which exports an async function that is triggered once after all test suites
+    // globalTeardown: undefined,
+
+    // A set of global variables that need to be available in all test environments
+    // globals: {},
+
+    // The maximum amount of workers used to run your tests. Can be specified as % or a number. E.g. maxWorkers: 10% will use 10% of your CPU amount + 1 as the maximum worker number. maxWorkers: 2 will use a maximum of 2 workers.
+    // maxWorkers: "50%",
+
+    // An array of directory names to be searched recursively up from the requiring module's location
+    // moduleDirectories: [
+    //   "node_modules"
+    // ],
+
+    // An array of file extensions your modules use
+    // moduleFileExtensions: [
+    //   "js",
+    //   "mjs",
+    //   "cjs",
+    //   "jsx",
+    //   "ts",
+    //   "tsx",
+    //   "json",
+    //   "node"
+    // ],
+
+    // A map from regular expressions to module names or to arrays of module names that allow to stub out resources with a single module
+    // moduleNameMapper: {},
+
+    // An array of regexp pattern strings, matched against all module paths before considered 'visible' to the module loader
+    // modulePathIgnorePatterns: [],
+
+    // Activates notifications for test results
+    // notify: false,
+
+    // An enum that specifies notification mode. Requires { notify: true }
+    // notifyMode: "failure-change",
+
+    // A preset that is used as a base for Jest's configuration
+    // preset: undefined,
+
+    // Run tests from one or more projects
+    // projects: undefined,
+
+    // Use this configuration option to add custom reporters to Jest
+    // reporters: undefined,
+
+    // Automatically reset mock state before every test
+    // resetMocks: false,
+
+    // Reset the module registry before running each individual test
+    // resetModules: false,
+
+    // A path to a custom resolver
+    // resolver: undefined,
+
+    // Automatically restore mock state and implementation before every test
+    // restoreMocks: false,
+
+    // The root directory that Jest should scan for tests and modules within
+    // rootDir: undefined,
+
+    // A list of paths to directories that Jest should use to search for files in
+    // roots: [
+    //   "<rootDir>"
+    // ],
+
+    // Allows you to use a custom runner instead of Jest's default test runner
+    // runner: "jest-runner",
+
+    // The paths to modules that run some code to configure or set up the testing environment before each test
+    // setupFiles: [],
+
+    // A list of paths to modules that run some code to configure or set up the testing framework before each test
+    // setupFilesAfterEnv: [],
+
+    // The number of seconds after which a test is considered as slow and reported as such in the results.
+    // slowTestThreshold: 5,
+
+    // A list of paths to snapshot serializer modules Jest should use for snapshot testing
+    // snapshotSerializers: [],
+
+    // The test environment that will be used for testing
+    // testEnvironment: "jest-environment-node",
+
+    // Options that will be passed to the testEnvironment
+    // testEnvironmentOptions: {},
+
+    // Adds a location field to test results
+    // testLocationInResults: false,
+
+    // The glob patterns Jest uses to detect test files
+    // testMatch: [
+    //   "**/__tests__/**/*.[jt]s?(x)",
+    //   "**/?(*.)+(spec|test).[tj]s?(x)"
+    // ],
+
+    // An array of regexp pattern strings that are matched against all test paths, matched tests are skipped
+    // testPathIgnorePatterns: [
+    //   "\\\\node_modules\\\\"
+    // ],
+
+    // The regexp pattern or array of patterns that Jest uses to detect test files
+    // testRegex: [],
+
+    // This option allows the use of a custom results processor
+    // testResultsProcessor: undefined,
+
+    // This option allows use of a custom test runner
+    // testRunner: "jest-circus/runner",
+
+    // A map from regular expressions to paths to transformers
+    // transform: undefined,
+
+    // An array of regexp pattern strings that are matched against all source file paths, matched files will skip transformation
+    // transformIgnorePatterns: [
+    //   "\\\\node_modules\\\\",
+    //   "\\.pnp\\.[^\\\\]+$"
+    // ],
+
+    // An array of regexp pattern strings that are matched against all modules before the module loader will automatically return a mock for them
+    // unmockedModulePathPatterns: undefined,
+
+    // Indicates whether each individual test should be reported during the run
+    // verbose: undefined,
+
+    // An array of regexp patterns that are matched against all source file paths before re-running tests in watch mode
+    // watchPathIgnorePatterns: [],
+
+    // Whether to use watchman for file crawling
+    // watchman: true,
+};

package.json

@@ -1,16 +1,20 @@
 {
   "name": "commandparser",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Parser meant to parse commands and their options for discord bots",
   "main": "build/index.js",
   "author": "Navy.gif",
   "license": "MIT",
   "private": false,
   "devDependencies": {
+    "@babel/core": "^7.20.12",
+    "@babel/preset-env": "^7.20.2",
     "@types/node": "^18.6.2",
     "@typescript-eslint/eslint-plugin": "^5.31.0",
     "@typescript-eslint/parser": "^5.31.0",
     "eslint": "^8.20.0",
+    "eslint-plugin-jest": "^27.2.1",
+    "jest": "^29.4.2",
     "typescript": "^4.7.4"
   },
   "files": [
@@ -18,6 +22,8 @@
   ],
   "scripts": {
     "build": "tsc",
-    "release": "tsc && yarn publish"
+    "test": "jest",
+    "release": "tsc && yarn publish",
+    "lint": "eslint --fix"
   }
 }

src/Parser.ts

@@ -101,6 +101,7 @@ class Parser extends EventEmitter {
         // Parse out subcommands
         if (subcommandGroups.length || subcommands.length) {
             this.debug(`Expecting subcommand`);
+            if(!params.length) throw new ParseError('Expected a subcommand(group), got nothing');
             const [first, second, ...rest] = params;
             group = command.subcommandGroup(first);
 

tests/CommandOption.test.js

@@ -22,3 +22,7 @@ const parser = new Parser({
     const parsed = await parser.parseMessage('test message.author')
     console.log(parsed)
 })();
+
+test('', () => {
+    
+}) 

tests/Parser.test.js

@@ -1,23 +1,11 @@
 const { inspect } = require('node:util');
 const { Parser, Command, OptionType } = require('../build');
 
-const command = new Command({name: 'create', options:[{name: 'registration-code', aliases: ['code'], type: OptionType.SUB_COMMAND}]})
 
-const parser = new Parser({commands: [command], prefix: ''})
-const debug = (a) => console.log(inspect(a));
+test('Command Parser', async () => {
+    const command = new Command({ name: 'create', options: [{ name: 'registration-code', aliases: ['code'], type: OptionType.SUB_COMMAND }, { name: 'test' }] });
+    const parser = new Parser({ commands: [command], prefix: '' });
 
-const handle = async (data) => {
-    const raw = data.toString('utf-8');
-    let words = raw.split(' ').map(word => word.trim());
+    await expect(parser.parseMessage('create')).rejects.toThrow();
 
-    words = words.filter(word => word.length);
-    if (!words.length) 
-        return;
-    debug((words));
-
-    const result = await parser.parseMessage(words.join(' '));
-    debug(result)
-
-}
-
-process.stdin.on('data', handle)
+});