Update dependency np to v8

This was caused by the randomiser using the wrong brackets.

We needed the maximum index of the length - 1, but the randomiser would
take the 1 AFTER randomising, adding brackets made it do this BEFORE.

.gitignore

@@ -1,3 +1,4 @@
-node_modules
+node_modules/
-dist
+dist/
-yarn-error.log
+coverage/
+yarn-error.log

package.json

@@ -1,6 +1,6 @@
 {
   "name": "random-bunny",
-  "version": "2.1.1",
+  "version": "2.1.3",
   "description": "Get a random subreddit image url",
   "license": "MIT",
   "author": "Vylpes",
@@ -42,14 +42,15 @@
     "eslint": "^8.35.0",
     "jest": "^29.4.3",
     "jest-mock-extended": "^3.0.3",
-    "np": "^7.7.0",
+    "np": "^8.0.0",
     "ts-jest": "^29.0.5",
     "ts-mockito": "^2.6.1",
     "ts-node": "^10.9.1",
     "typescript": "^5.0.0"
   },
   "resolutions": {
-    "np/**/got": "^11.8.5"
+    "np/**/got": "^11.8.5",
+    "**/semver": "^7.5.2"
   },
   "files": [
     "dist"

src/index.ts

@@ -41,8 +41,8 @@ export default async function randomBunny(subreddit: string, sortBy?: string): P
         return {
             IsSuccess: false,
         };
-    } else if (dataWithImages.length > 1) {
+    } else {
-        random = Math.floor((Math.random() * dataWithImages.length - 1) + 0); // Between 0 and (size - 1)
+        random = Math.floor((Math.random() * (dataWithImages.length - 1)) + 0); // Between 0 and (size - 1)
     }
 
     const randomSelect = dataWithImages[random];