Add TypeScript support and basic events tests

Author: heyhippari

babel.config.js

@@ -6,7 +6,8 @@ module.exports = {
                 targets: {
                     node: 'current'
                 }
-            }
-        ]
+            },
+        ],
+        '@babel/preset-typescript'
     ]
 };

package.json

@@ -7,13 +7,17 @@
   "devDependencies": {
     "@babel/core": "^7.2.2",
     "@babel/preset-env": "^7.9.0",
+    "@babel/preset-typescript": "^7.9.0",
     "babel-jest": "^25.1.0",
     "babel-loader": "^8.0.5",
     "eslint": "^6.8.0",
     "eslint-plugin-jest": "^23.8.2",
     "jest": "^25.1.0",
     "jest-junit": "^10.0.0",
     "jsdoc": "^3.6.3",
+    "source-map-loader": "^0.2.4",
+    "ts-loader": "^6.2.2",
+    "typescript": "^3.8.3",
     "webpack": "^4.29.0",
     "webpack-cli": "^3.2.1"
   },

src/events.js

@@ -1,5 +1,4 @@
 function getCallbacks(obj, name) {
-
     if (!obj) {
         throw new Error("obj cannot be null!");
     }
@@ -17,16 +16,13 @@
 }
 
 export default {
-
     on(obj, eventName, fn) {
-
         const list = getCallbacks(obj, eventName);
 
         list.push(fn);
     },
 
     off(obj, eventName, fn) {
-
         const list = getCallbacks(obj, eventName);
 
         const i = list.indexOf(fn);
@@ -36,7 +32,6 @@ export default {
     },
 
     trigger(obj, eventName) {
-
         const eventObject = {
             type: eventName
         };

tests/events.test.js

@@ -0,0 +1,15 @@
+import events from '../src/events';
+
+test('contains an on property', () => {
+    expect(events).toHaveProperty("on");
+});
+
+test('contains an off property', () => {
+    expect(events).toHaveProperty("off");
+});
+
+test('contains a trigger property', () => {
+    expect(events).toHaveProperty("trigger");
+});
+
+

tsconfig.json

@@ -0,0 +1,25 @@
+{
+    "compilerOptions": {
+        "allowJs": true,
+        "allowSyntheticDefaultImports": true,
+        "lib": [
+            "dom",
+            "es2015",
+            "es2016"
+        ],
+        "module": "es2015",
+        "moduleResolution": "node",
+        "noImplicitAny": true,
+        "noUnusedLocals": true,
+        "noUnusedParameters": true,
+        "outDir": "./dist",
+        "preserveConstEnums": true,
+        "removeComments": false,
+        "skipLibCheck": true,
+        "sourceMap": true,
+        "target": "es2016",
+    },
+    "include": [
+        "./src/**/*"
+    ]
+}

webpack.config.js

@@ -1,24 +1,40 @@
 const path = require('path');
 
+var babelLoader = {
+    loader: 'babel-loader',
+    options: {
+        cacheDirectory: true,
+    }
+};
+
 module.exports = {
     entry: {
         'jellyfin-apiclient': 'index.js',
     },
+    devtool: "source-map",
     module: {
         rules: [
             {
-                test: /\.js$/,
-                exclude: /(dist|node_modules)/,
-                use: {
-                    loader: 'babel-loader',
-                    options: {
-                        presets: ['@babel/preset-env']
+                test: /\.ts(x?)$/,
+                exclude: /node_modules/,
+                use: [
+                    babelLoader,
+                    {
+                        loader: 'ts-loader'
                     }
-                }
+                ]
+            },
+            {
+                test: /\.js$/,
+                exclude: /node_modules/,
+                use: [
+                    babelLoader
+                ]
             }
         ],
     },
     resolve: {
+        extensions: ['.ts', '.js'],
         modules: [
             path.resolve(__dirname, 'node_modules'),
             path.resolve(__dirname, 'src')