Browse Source

[tests] check for range too for #7282

Rudy Ges 1 year ago
parent
commit
b8b2dfccbf

+ 10 - 2
tests/server/src/cases/issues/Issue7282.hx

@@ -1,13 +1,21 @@
 package cases.issues;
 
+import haxe.display.Diagnostic;
+
 class Issue7282 extends TestCase {
 	function test(_) {
-		vfs.putContent("Main.hx", getTemplate("issues/Issue7282/Main.hx"));
+		var content = getTemplate("issues/Issue7282/Main.hx");
+		var transform = Markers.parse(content);
+
+		vfs.putContent("Main.hx", transform.source);
 		var args = ["-main", "Main"];
 		runHaxe(args);
 		assertSuccess();
 		runHaxeJsonCb(args, DisplayMethods.Diagnostics, {file: new FsPath("Main.hx")}, res -> {
-			Assert.equals("Unused variable", (cast res[0].diagnostics[0].args).description);
+			var arg:ReplacableCode = res[0].diagnostics[0].args;
+			Assert.equals("Unused variable", arg.description);
+			Assert.same(transform.range(1,2), res[0].diagnostics[0].range);
+			Assert.same(transform.range(1,2), arg.range);
 		});
 	}
 }

+ 1 - 1
tests/server/test/templates/issues/Issue7282/Main.hx

@@ -3,7 +3,7 @@ import haxe.ds.Option;
 class Main {
 	public static function main() {
 		switch ((null:Option<Int>)) {
-			case Some(value):
+			case Some({-1-}value{-2-}):
 			case None:
 		}
 	}