Merge pull request #91 from asmundg/explode-when-out-of-emoji

Have a violent reaction when out of emoji

Author: sokra

lib/interpolateName.js

@@ -13,6 +13,10 @@ function encodeStringToEmoji(content, length) {
 	length = length || 1;
 	const emojis = [];
 	do {
+		if(!emojiList.length) {
+			throw new Error("Ran out of emoji");
+		}
+
 		const index = Math.floor(Math.random() * emojiList.length);
 		emojis.push(emojiList[index]);
 		emojiList.splice(index, 1);

test/interpolateName.test.js

@@ -66,6 +66,12 @@ describe("interpolateName()", () => {
 		assert.equal(result1, result2);
 	});
 
+	it("should throw error when out of emoji", () => {
+		assert.throws(() => {
+			loaderUtils.interpolateName.apply(loaderUtils, [{}, "[emoji:5000]", { content: "foo" }]);
+		}, Error, "Ran out of emoji");
+	});
+
 	context("no loader context", () => {
 		const loaderContext = {};
 		run([