Merge pull request #11 from GDIBass/v0.0.1

v0.0.1 - test

Author: GDIBass

.eslintrc.json

@@ -12,6 +12,7 @@
       "eslint-comments/no-use": "off",
       "import/no-namespace": "off",
       "no-unused-vars": "off",
+      "filenames/match-regex": "off",
       "@typescript-eslint/no-unused-vars": "error",
       "@typescript-eslint/explicit-member-accessibility": ["error", {"accessibility": "no-public"}],
       "@typescript-eslint/no-require-imports": "error",
@@ -22,6 +23,8 @@
       "@typescript-eslint/consistent-type-assertions": "error",
       "@typescript-eslint/explicit-function-return-type": ["error", {"allowExpressions": true}],
       "@typescript-eslint/func-call-spacing": ["error", "never"],
+      "no-shadow": "off",
+      "@typescript-eslint/no-shadow": ["error"],
       "@typescript-eslint/no-array-constructor": "error",
       "@typescript-eslint/no-empty-interface": "error",
       "@typescript-eslint/no-explicit-any": "error",

.github/workflows/check-dist.yml

@@ -23,11 +23,12 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
-      - name: Set Node.js 16.x
+      - name: Set Node.js 16.19.0
         uses: actions/setup-node@v3.6.0
         with:
-          node-version: 16.x
-
+          node-version: 16.19.0
+      - name: Check npm version
+        run: npm --version
       - name: Install dependencies
         run: npm ci
 
@@ -38,14 +39,14 @@ jobs:
 
       - name: Compare the expected and actual dist/ directories
         run: |
-          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+          if [ "$(git diff --text --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
             echo "Detected uncommitted changes after build.  See status below:"
             git diff
             exit 1
           fi
         id: diff
 
-      # If index.js was different than expected, upload the expected version as an artifact
+      # If index.js was different from expected, upload the expected version as an artifact
       - uses: actions/upload-artifact@v3
         if: ${{ failure() && steps.diff.conclusion == 'failure' }}
         with:

.github/workflows/publish.yml

@@ -0,0 +1,19 @@
+name: Publish
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3.6.0
+        with:
+          node-version: 16
+          registry-url: https://registry.npmjs.org/
+      - run: npm install
+      - run: npm publish --access public
+        env:
+          NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}

.github/workflows/test.yml

@@ -10,7 +10,9 @@ jobs:
   build: # make sure build/ci work properly
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
+      - name: setup node-gyp
+        run: npm install -g node-gyp@latest
       - run: |
           npm install
       - run: |
@@ -20,5 +22,3 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - uses: ./
-        with:
-          milliseconds: 1000

__tests__/alliance.test.ts

@@ -23,7 +23,7 @@ test('duplicate id for member_status fails', async () => {
   } catch (error) {
     expect(error).toBeInstanceOf(InvalidConfig);
     if (error instanceof InvalidConfig) {
-      expect(error.message).toContain('Duplicate ID found in child: member_status');
+      expect(error.message).toContain('Duplicate ID found | child=member_status');
     }
   }
 });
@@ -56,7 +56,7 @@ test('missing member status fails', async () => {
   } catch (error) {
     expect(error).toBeInstanceOf(MissingRequiredKey);
     if (error instanceof MissingRequiredKey) {
-      expect(error.message).toContain('Config is missing key: AllianceConfig:member_status');
+      expect(error.message).toContain('Config is missing key | AllianceConfig:member_status');
     }
   }
 });

__tests__/colors.test.ts

@@ -1 +1,6 @@
+import {expect, test} from '@jest/globals'
+
+test('colors.test.js', async() => {
+  expect(true).toBeTruthy();
+});
 // TODO: Test this

__tests__/config/AllianceConfig.test.ts

@@ -1 +1,6 @@
+import {expect, test} from '@jest/globals'
+
+test('AllianceConfig.test.ts', async() => {
+  expect(true).toBeTruthy();
+});
 // TODO: This and all negative tests

__tests__/config/AscensionsConfig.test.ts

@@ -1 +1,6 @@
+import {expect, test} from '@jest/globals'
+
+test('AscensionsConfig.test.ts', async() => {
+  expect(true).toBeTruthy();
+});
 // TODO: This and all negative tests

__tests__/config/ClassesConfig.test.ts

@@ -1 +1,6 @@
+import {expect, test} from '@jest/globals'
+
+test('ClassesConfig.test.ts', async() => {
+  expect(true).toBeTruthy();
+});
 // TODO: This and all negative tests

__tests__/config/ColorsConfig.test.ts

@@ -1 +1,6 @@
+import {expect, test} from '@jest/globals'
+
+test('ColorsConfig.test.ts', async() => {
+  expect(true).toBeTruthy();
+});
 // TODO: Test this