Fix a bug with Drop Notation parsing (#598)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "randsum",
-  "version": "4.3.1",
+  "version": "4.3.2",
   "private": false,
   "author": "Alex Jarvis",
   "icon": "https://raw.githubusercontent.com/RANDSUM/randsum-ts/main/icon.webp",

src/constants.ts

@@ -1,6 +1,6 @@
 export const coreNotationPattern = /(?<coreNotationMatch>^\d+[Dd](\d+|{.*}))/
 export const modifierRollPatterns =
-  /(?<dropHighMatch>[Hh]\d*)|(?<dropLowMatch>[Ll]\d*)|(?<dropConstraintsMatch>[Dd]{?([<>|]?\d+,?)*}?)|(?<explodeMatch>!+{?([<>|]?\d+,?)*}?)|(?<uniqueMatch>[Uu]({(\d+,?)+})?)|(?<replaceMatch>[Vv]{?([<>|]?\d+=?\d+,?)*}?)|(?<rerollMatch>[Rr]{?([<>|]?\d,?)*}\d*)|(?<capMatch>[Cc]([<>|]?\d+)*)|(?<plusMatch>\+\d+)|(?<minusMatch>-\d+)/
+  /(?<dropHighMatch>[Hh]\d*)|(?<dropLowMatch>[Ll]\d*)|(?<dropConstraintsMatch>[Dd]{([<>|]?\d+,?)*})|(?<explodeMatch>!+{?([<>|]?\d+,?)*}?)|(?<uniqueMatch>[Uu]({(\d+,?)+})?)|(?<replaceMatch>[Vv]{?([<>|]?\d+=?\d+,?)*}?)|(?<rerollMatch>[Rr]{?([<>|]?\d,?)*}\d*)|(?<capMatch>[Cc]([<>|]?\d+)*)|(?<plusMatch>\+\d+)|(?<minusMatch>-\d+)/
 
 export const completeRollPattern = new RegExp(
   `${coreNotationPattern.source}|${modifierRollPatterns.source}`,

tests/validate-notation.test.ts

@@ -29,6 +29,19 @@ describe('validateNotation', () => {
     })
   })
 
+  describe('when given a nonsensical drop notation that is bugging me', () => {
+    const notation: DiceNotation<number> = '2d5D'
+
+    it('returns an error result', () => {
+      const result = validateNotation(notation)
+
+      expect(result).toEqual({
+        valid: false,
+        description: []
+      })
+    })
+  })
+
   describe('when given a standard notation', () => {
     it('returns a valid result', () => {
       const notation = '2d6'