Fixed yaml-language-server#72

redhat-developer · Jun 6, 2018 · 6830f57 · 6830f57
1 parent c7a0e5b
commit 6830f57
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/languageservice/parser/yamlParser.ts b/src/languageservice/parser/yamlParser.ts
@@ -158,7 +158,7 @@ function recursivelyBuildAst(parent: ASTNode, node: Yaml.YAMLNode): ASTNode {
 
 			switch (type) {
 				case Yaml.ScalarType.null: {
-					return new NullASTNode(parent, name, instance.startPosition, instance.endPosition);
+					return new StringASTNode(parent, name, false, instance.startPosition, instance.endPosition);
 				}
 				case Yaml.ScalarType.bool: {
 					return new BooleanASTNode(parent, name, Yaml.parseYamlBoolean(value), node.startPosition, node.endPosition)

diff --git a/test/schemaValidation.test.ts b/test/schemaValidation.test.ts
@@ -96,6 +96,14 @@ suite("Validation Tests", () => {
 				}).then(done, done);
 			});
 
+			it('Null scalar value should be treated as string', (done) => {
+				let content = `cwd: Null`;
+				let validator = parseSetup(content);
+				validator.then(function(result){
+					assert.equal(result.length, 0);
+				}).then(done, done);
+			});
+
 			it('Anchor should not not error', (done) => {
 				let content = `default: &DEFAULT\n  name: Anchor\nanchor_test:\n  <<: *DEFAULT`;
 				let validator = parseSetup(content);