fix: chat scope is parsed and formatted correctly

* fix: chat audience is being parsed properly now

* test: add test for chat formatter

* improvement: differ 0x10 and 0x20 chat message types

* improvement: revert changes used for testing in example.j

package.json

@@ -20,7 +20,7 @@
     "jest": "^23.5.0"
   },
   "scripts": {
-    "test": "jest test/test.js",
+    "test": "jest",
     "coverage": "jest test/test.js --collectCoverage",
     "lint": "eslint --ext .jsx,.js lib/ test/",
     "lint:fix": "eslint --ext .jsx,.js lib/ --fix"

parsers/formatters.js

@@ -37,6 +37,11 @@ const chatModeFormatter = (flag) => {
     case 0x02:
       return 'OBS'
   }
+
+  if (flag >= 3 && flag <= 27) {
+    return `PRIVATE${flag}`
+  }
+
   return flag
 }
 

parsers/gamedata.js

@@ -7,7 +7,7 @@
 
 const { Parser } = require('binary-parser')
 const { CommandDataBlock } = require('./actions')
-const {chatModeFormatter} = require('./formatters')
+const { chatModeFormatter } = require('./formatters')
 
 // 0x17
 const LeaveGameBlock = new Parser()
@@ -45,7 +45,14 @@ const PlayerChatMessageBlock = new Parser()
   .int8('playerId')
   .int16le('byteCount')
   .int8('flags')
-  .int32('chatMode', {formatter: chatModeFormatter})
+  .choice(
+    {tag: 'flags',
+      choices: {
+        0x10: new Parser(),
+        0x20: new Parser().int8('chatMode', {length: 4, formatter: chatModeFormatter, encoding: 'hex'}).skip(3)
+      }
+    }
+  )
   .string('message', {zeroTerminated: true, encoding: 'utf8'})
 
 // 0x22

test/formatters.test.js

@@ -0,0 +1,19 @@
+const formatters = require('../parsers/formatters')
+
+describe('chatModeFormatter', () => {
+  const {chatModeFormatter} = formatters
+  it('correctly handles 0 as ALL', () => {
+    expect(chatModeFormatter(0)).toBe('ALL')
+  })
+  it('correctly handles 1 as ALLY', () => {
+    expect(chatModeFormatter(1)).toBe('ALLY')
+  })
+  it('correctly handles 2 as OBS', () => {
+    expect(chatModeFormatter(2)).toBe('OBS')
+  })
+  for (let i = 0; i < 24; i++) {
+    it(`correctly handles private message to slot ${i}`, () => {
+      expect(chatModeFormatter(3 + i)).toBe(`PRIVATE${i + 3}`)
+    })
+  }
+})