validateTree() detects missing nodes in declarations+assignments.

Author: ReFreezed

dumbParser.lua

@@ -6120,6 +6120,9 @@ do
 
 		elseif nodeType == "declaration" then
 			local decl = node
+			if not decl.names[1] then
+				addValidationError(path, errors, "Missing name(s).")
+			end
 			for i, ident in ipairs(decl.names) do
 				if ident.type ~= "identifier" then
 					addValidationError(path, errors, "Name %d is not an identifier. (It is '%s')", i, ident.type)
@@ -6137,13 +6140,19 @@ do
 
 		elseif nodeType == "assignment" then
 			local assignment = node
+			if not assignment.targets[1] then
+				addValidationError(path, errors, "Missing target expression(s).")
+			end
 			for i, expr in ipairs(assignment.targets) do
 				if not (expr.type == "identifier" or expr.type == "lookup") then
 					addValidationError(path, errors, "Target %d is not an identifier or lookup. (It is '%s')", i, expr.type)
 				else
 					validateNode(expr, path, errors, "target"..i)
 				end
 			end
+			if not assignment.values[1] then
+				addValidationError(path, errors, "Missing value(s).")
+			end
 			for i, expr in ipairs(assignment.values) do
 				if not EXPRESSION_NODES[expr.type] then
 					addValidationError(path, errors, "Value %d is not an expression. (It is '%s')", i, expr.type)

testsuite.lua

@@ -976,6 +976,8 @@ test("Validate", function()
 	local decl       = parseStatement[[ local x = 1 ]] ; decl.names[1]  = parser.newNode("vararg") ; testInvalid(decl)
 	local decl       = parseStatement[[ local x = 1 ]] ; decl.values[1] = parser.newNode("block")  ; testInvalid(decl)
 
+	local assignment = parseStatement[[ x      = 1     ]] ; assignment.targets[1] = nil                      ; testInvalid(assignment)
+	local assignment = parseStatement[[ x      = 1     ]] ; assignment.values[1]  = nil                      ; testInvalid(assignment)
 	local assignment = parseStatement[[ x, t.k = 1, "" ]] ; assignment.targets[1] = parser.newNode("vararg") ; testInvalid(assignment)
 	local assignment = parseStatement[[ x, t.k = 1, "" ]] ; assignment.values[1]  = parser.newNode("block")  ; testInvalid(assignment)