Decrease MAX_TELEGRAM_MESSAGE_LENGTH

Author: mbelsky

src/notifier/__tests__/notifier.spec.js

@@ -249,6 +249,6 @@ describe('_notify', () => {
       updateUser,
     })
 
-    expect(telegram.sendMessage.mock.calls.length).toBe(2)
+    expect(telegram.sendMessage.mock.calls.length).toBe(4)
   })
 })

src/notifier/__tests__/splitLongTelegramMessage.spec.js

@@ -5,7 +5,7 @@ const {
 } = require('../messages')
 const { splitLongTelegramMessage } = require('../splitLongTelegramMessage')
 
-const LONG_MESSAGE = `ABC vs XYZ\n\n`.repeat(833) + 'abcdef\n\n123456'
+const LONG_MESSAGE = `ABC vs XYZ\n\n`.repeat(333) + 'abcdef\n\n123456'
 const UNSPLITABLE_MESSAGE = '0123456789'.repeat(10001)
 
 test('throws `Split function is not defined` error', () => {
@@ -31,7 +31,7 @@ test('splits featured matches message', () => {
 
   const result = splitLongTelegramMessage(message)
 
-  expect(result[0]).toBe(`ABC vs XYZ\n\n`.repeat(833).trim())
+  expect(result[0]).toBe(`ABC vs XYZ\n\n`.repeat(333).trim())
   expect(result[1]).toBe('abcdef\n\n123456')
 })
 

src/notifier/sendMessages.js

@@ -55,7 +55,13 @@ exports.sendMessages = async ({
     const texts = splitLongTelegramMessage(message)
 
     for (const text of texts) {
-      await telegram.sendMessage(userId, text, extra)
+      try {
+        await telegram.sendMessage(userId, text, extra)
+      } catch (e) {
+        log(`Faield to send ${text.length} length message`)
+
+        throw e
+      }
     }
   }
 

src/notifier/splitLongTelegramMessage.js

@@ -1,5 +1,5 @@
 const { MESSAGE_TYPES } = require('./consts')
-const MAX_TELEGRAM_MESSAGE_LENGTH = 10000
+const MAX_TELEGRAM_MESSAGE_LENGTH = 4000
 
 const splitFunctions = {
   [MESSAGE_TYPES.favoriteTeamsMatches]: splitMatchesMessage,