Fix #266 Add a test and update the CHANGELOG

This was fixed in the v0.14.0 release but we forgot to add a test and
update the CHANGELOG.md file.

Signed-off-by: Remy Suen <[email protected]>

Author: rcjsuen

CHANGELOG.md

@@ -18,6 +18,7 @@ All notable changes to this project will be documented in this file.
   - prevent invalid heredocs from crashing the formatter ([rcjsuen/dockerfile-utils#127](https://github.com/rcjsuen/dockerfile-utils/issues/127))
 - textDocument/definition
   - stop resolving definitions to build stages after the selected line ([rcjsuen/dockerfile-language-service#130](https://github.com/rcjsuen/dockerfile-language-service/issues/130))
+  - prevent invalid heredocs from crashing the server ([#266](https://github.com/rcjsuen/dockerfile-language-server/issues/266))
 - textDocument/documentHighlight
   - build stage references in FROM instructions will now be highlighted ([rcjsuen/dockerfile-language-service#128](https://github.com/rcjsuen/dockerfile-language-service/issues/128))
 - textDocument/rename

test/server.test.ts

@@ -2314,6 +2314,39 @@ describe("Dockerfile LSP Tests", function() {
 		});
 	});
 
+	it("issue #266", function (finished) {
+		this.timeout(5000);
+		sendNotification("textDocument/didOpen", {
+			textDocument: {
+				languageId: "dockerfile",
+				version: 1,
+				uri: "uri://dockerfile/266.txt",
+				text: "RUN <<'END_OF_PATCH\nEND_OF_PATCH\nRUN <<'SCRIPT'\nSCRIPT"
+			}
+		});
+
+
+		const requestId = sendRequest("textDocument/definition", {
+			textDocument: {
+				uri: "uri://dockerfile/266.txt",
+			},
+			position: {
+				line: 0,
+				character: 10
+			}
+		});
+
+		const listener = (json) => {
+			if (json.id === requestId) {
+				lspProcess.removeListener("message", listener);
+				assert.strictEqual(json.error, undefined);
+				assert.strictEqual(json.result, null);
+				finished();
+			}
+		};
+		lspProcess.on("message", listener);
+	});
+
 	after(() => {
 		// terminate the forked LSP process after all the tests have been run
 		lspProcess.kill();