Added test suite for insert text and fixed several small bugs

Author: Zaydek Michels-Gualtieri

src/RichTextEditor/utils/keyDownTypeFor/insertText.js

@@ -1,4 +1,4 @@
-import keyCodeFor from "../keyCodeFor"
+import keyCodeFor from "lib/Client/keyCodeFor"
 
 // Returns the UTF-8 decoded rune count.
 function runeCount(str) {
@@ -16,7 +16,8 @@ const insertText = {
 	},
 	insertTab(e) {
 		const ok = (
-			!e.ctrlKey && // Negates browser shortcuts shift-ctrl-tab and ctrl-tab
+			!e.ctrlKey && // Negates browser shortcuts
+			!e.metaKey && // Negates operating system shortcuts
 			e.keyCode === keyCodeFor("Tab")
 		)
 		return ok

src/RichTextEditor/utils/keyDownTypeFor/insertText.test.js

@@ -0,0 +1,110 @@
+import insertText from "./insertText"
+import keyCodeFor from "lib/Client/keyCodeFor"
+
+test("insertText(...)", () => {
+	expect(insertText.insertText({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: false,
+		key: "a",
+	})).toBeTruthy()
+	expect(insertText.insertText({
+		shiftKey: true,
+		ctrlKey: false,
+		metaKey: false,
+		key: "A",
+	})).toBeTruthy()
+	expect(insertText.insertText({
+		shiftKey: false,
+		ctrlKey: true,
+		metaKey: false,
+		key: "a",
+	})).not.toBeTruthy()
+	expect(insertText.insertText({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: true,
+		key: "a",
+	})).not.toBeTruthy()
+})
+
+test("insertTab(...)", () => {
+	expect(insertText.insertTab({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Tab"),
+	})).toBeTruthy()
+	expect(insertText.insertTab({
+		shiftKey: true,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Tab"),
+	})).toBeTruthy()
+	expect(insertText.insertTab({
+		shiftKey: false,
+		ctrlKey: true,
+		metaKey: false,
+		keyCode: keyCodeFor("Tab"),
+	})).not.toBeTruthy()
+	expect(insertText.insertTab({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: true,
+		keyCode: keyCodeFor("Tab"),
+	})).not.toBeTruthy()
+})
+
+test("insertSoftParagraph(...)", () => {
+	expect(insertText.insertSoftParagraph({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).not.toBeTruthy()
+	expect(insertText.insertSoftParagraph({
+		shiftKey: true,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).toBeTruthy()
+	expect(insertText.insertSoftParagraph({
+		shiftKey: false,
+		ctrlKey: true,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).not.toBeTruthy()
+	expect(insertText.insertSoftParagraph({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: true,
+		keyCode: keyCodeFor("Enter"),
+	})).not.toBeTruthy()
+})
+
+test("insertHardParagraph(...)", () => {
+	expect(insertText.insertHardParagraph({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).toBeTruthy()
+	expect(insertText.insertHardParagraph({
+		shiftKey: true,
+		ctrlKey: false,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).not.toBeTruthy()
+	expect(insertText.insertHardParagraph({
+		shiftKey: false,
+		ctrlKey: true,
+		metaKey: false,
+		keyCode: keyCodeFor("Enter"),
+	})).toBeTruthy()
+	expect(insertText.insertHardParagraph({
+		shiftKey: false,
+		ctrlKey: false,
+		metaKey: true,
+		keyCode: keyCodeFor("Enter"),
+	})).toBeTruthy()
+})