Feature/scoring (#24)

* first pass

* Fixed

* Lint fixes

.eslintrc.json

@@ -12,6 +12,7 @@
 		"ecmaVersion": 2021
 	},
 	"rules": {
-		"semi": [2, "always"]
+		"semi": [2, "always"],
+		"quotes": [2, "single", { "avoidEscape": true }]
 	}
 }

.gitignore

@@ -1,3 +1,6 @@
+# score database
+score.db3*
+
 # Created by https://www.toptal.com/developers/gitignore/api/node
 # Edit at https://www.toptal.com/developers/gitignore?templates=node
 

__tests__/config.test.js

@@ -21,12 +21,13 @@ describe('Tests the config module', () => {
         const config = require('../config');
 
         expect(config).toEqual( {
-            "AllowConfigDump": false,
-            "MessageFetchCount": 50,
-            "OneBlockedPercent": 1,
-            "RealestOneBlockedPercent": 5,
-            "TheRealests": [
-              "kerouac5",
+            'AllowConfigDump': false,
+            'MessageFetchCount': 50,
+            'OneBlockedPercent': 1,
+            'ScoreDatabase': './score.db3',
+            'RealestOneBlockedPercent': 5,
+            'TheRealests': [
+              'kerouac5',
             ]
         });
     });

__tests__/replacer.test.js

@@ -9,7 +9,7 @@ describe('Tests the replacer module', () => {
         'I guess I could get back to working on the bot but honestly I haven’t written code for a living since 2006, zippy would run circles around me'
     ].map(content => ({ 
         content,
-        author: "author"  
+        author: 'author'  
     }));
 
     const channel = {

__tests__/scoring.test.js

@@ -0,0 +1,50 @@
+describe('Tests for the scoring module', () => {
+    let oldEnv;
+
+    beforeAll(() => {
+        oldEnv = process.env;
+    });
+
+    beforeEach(() => {
+        process.env.SCORE_DATABASE = ':memory:';
+        jest.resetModules();
+    });
+
+    afterAll(() => {
+        process.enve = oldEnv;
+    });
+
+    it('keeps score', async () => {
+        const { processScores, getScore } = require('../scoring');
+        await getScore('urch', score => {
+            expect(score).toBe(0);
+        });
+        [
+            'urch++',
+            'urch++',
+            'poly--',
+            'urch++',
+            'urch--'
+        ].forEach(async (phrase) => {
+            await processScores({ content: phrase });
+        });
+        await getScore('urch', score => {
+            expect(score).toBe(2);
+        });
+    });
+
+    it('handles multiline scores', async () => {
+        const { processScores, getScore } = require('../scoring');
+        await getScore('urch', score => {
+            expect(score).toBe(0);
+        });
+
+        await processScores({ content: 'urch++\rpotato++\nurch++\r\nurch++' });
+        await getScore('urch', score => {
+            expect(score).toBe(3);
+        });
+        await getScore('potato', score => {
+            expect(score).toBe(1);
+        });
+    });
+});

config.js

@@ -19,6 +19,11 @@ module.exports = {
      * Number of messages to fetch when retrieving history.
      */
     MessageFetchCount: (messesageFetchCount >= 0) ? messesageFetchCount : 50,
+
+    /**
+     * Path to the scoring database.
+     */
+    ScoreDatabase: process.env.SCORE_DATABASE ?? './score.db3',
 
     /**
      * Percentage change of getting "who is one blocked message" for plebians who should be so lucky to be in

index.js

@@ -1,6 +1,7 @@
 const { Client, GatewayIntentBits } = require('discord.js');
 const config = require('./config');
 const { replaceFirstMessage } = require('./replacer');
+const { processScores, getScore } = require('./scoring');
 
 
 /**
@@ -63,6 +64,17 @@ client.on('messageCreate', async (initialQuery) => {
             initialQuery.channel.send(initialQuery.author.toString() + ' nobody said that, dumb ass');
         }
     }
+    else if (initialQuery.content.indexOf('!score ') == 0)
+    { 
+        const phrase = initialQuery.content.replace(/^!score/, '').trim();
+        getScore(phrase, score => {
+            initialQuery.channel.send(`Score *${phrase}*: ${score}`);
+        });
+    }
+    else
+    {
+        processScores(initialQuery);
+    }
 });
 
 

jest.config.json

@@ -8,10 +8,10 @@
       "clearMocks": true,
       "coverageThreshold": {
         "global": {
-            "branches": 26,
-            "functions": 40,
-            "lines": 35,
-            "statements": 33
+            "branches": 43,
+            "functions": 53,
+            "lines": 55,
+            "statements": 53
         }
     }
 }