also verify error position for constructors' errors

eclipse-xtext · Jan 8, 2024 · 96d25ab · 96d25ab
1 parent d017375
commit 96d25ab
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/...pse.xtend.core.tests/src/org/eclipse/xtend/core/tests/validation/XtendValidationTest.java b/...pse.xtend.core.tests/src/org/eclipse/xtend/core/tests/validation/XtendValidationTest.java
@@ -408,13 +408,19 @@ public void clearPreferences() {
 	}
 
 	@Test public void testMissingConstructor() throws Exception {
-		XtendClass clazz = clazz("class K extends test.NoDefaultConstructor {}");
-		helper.assertError(clazz, XTEND_CLASS, MISSING_CONSTRUCTOR);
+		var source = "class K extends test.NoDefaultConstructor {}";
+		XtendClass clazz = clazz(source);
+		helper.assertError(clazz, XTEND_CLASS, MISSING_CONSTRUCTOR,
+				source.indexOf("K"), 1,
+				"must define an explicit constructor");
 	}
 
 	@Test public void testMissingSuperConstructorCall() throws Exception {
-		XtendClass clazz = clazz("class K extends test.NoDefaultConstructor { new() {} }");
-		helper.assertError(clazz, XTEND_CONSTRUCTOR, MUST_INVOKE_SUPER_CONSTRUCTOR);
+		var source = "class K extends test.NoDefaultConstructor { new() {} }";
+		XtendClass clazz = clazz(source);
+		helper.assertError(clazz, XTEND_CONSTRUCTOR, MUST_INVOKE_SUPER_CONSTRUCTOR,
+				source.indexOf("new() {}"), "new() {}".length(),
+				"Another constructor must be invoked explicitly");
 	}
 
 	@Test public void testSuperConstructorCall() throws Exception {