Fix bugg in alg variable diagnostics

Author: vetlek

packages/septic/src/diagnostics.ts

@@ -278,7 +278,7 @@ export function validateAlg(
         diagnostics.push(diagnostic);
         return diagnostics;
     }
-    const visitor = new AlgVisitor(true);
+    const visitor = new AlgVisitor();
     visitor.visit(expr);
     visitor.calcs.forEach((calc) => {
         diagnostics.push(
@@ -291,7 +291,7 @@ export function validateAlg(
                 variable,
                 doc,
                 contextProvider,
-                offsetStartAlg,
+                algPositionTransformer,
             ),
         );
     });
@@ -302,7 +302,7 @@ export function validateAlgVariable(
     variable: AlgLiteral,
     doc: TextDocument,
     contextProvider: SepticContext,
-    offsetStartAlg: number,
+    algPositionTransformer: AlgPositionTransformer,
 ): SepticDiagnostic[] {
     if (isPureJinja(variable.value)) {
         return [];
@@ -317,10 +317,7 @@ export function validateAlgVariable(
         return [
             createDiagnostic(
                 SepticDiagnosticLevel.warning,
-                {
-                    start: doc.positionAt(offsetStartAlg + variable.start),
-                    end: doc.positionAt(offsetStartAlg + variable.end),
-                },
+                algPositionTransformer(variable.start, variable.end),
                 `Reference to undefined variable: ${variable.value}`,
                 SepticDiagnosticCode.missingReference,
             ),
@@ -333,10 +330,7 @@ export function validateAlgVariable(
         return [
             createDiagnostic(
                 SepticDiagnosticLevel.error,
-                {
-                    start: doc.positionAt(offsetStartAlg + variable.end - 1),
-                    end: doc.positionAt(offsetStartAlg + variable.end),
-                },
+                algPositionTransformer(variable.end - 1, variable.end),
                 `Missing public property for variable`,
                 SepticDiagnosticCode.missingPublicProperty,
             ),
@@ -358,21 +352,15 @@ export function validateAlgVariable(
         return [
             createDiagnostic(
                 SepticDiagnosticLevel.error,
-                {
-                    start: doc.positionAt(
-                        offsetStartAlg +
-                            variable.start +
-                            variableParts[0]!.length +
-                            1,
-                    ),
-                    end: doc.positionAt(offsetStartAlg + variable.end),
-                },
+                algPositionTransformer(
+                    variable.end - variableParts[1]!.length,
+                    variable.end,
+                ),
                 `Unknown public property ${variableParts[1]} for ${referencedObjects[0]!.type}'`,
                 SepticDiagnosticCode.unknownPublicProperty,
             ),
         ];
     }
-
     return [];
 }
 
@@ -593,11 +581,7 @@ function validateParamType(
     algPositionTransformer: AlgPositionTransformer,
 ): SepticDiagnostic[] {
     if (types[0]!.startsWith("value")) {
-        return validateValueParamType(
-            expr,
-            contextProvider,
-            algPositionTransformer,
-        );
+        return [];
     }
     return validateObjectParamType(
         expr,
@@ -642,36 +626,6 @@ function validateObjectParamType(
     ];
 }
 
-function validateValueParamType(
-    expr: AlgExpr,
-    contextProvider: SepticContext,
-    algPositionTransformer: AlgPositionTransformer,
-): SepticDiagnostic[] {
-    if (!isAlgExprObjectReference(expr)) {
-        return [];
-    }
-    const exprLiteral = expr as AlgLiteral;
-    if (isPureJinja(exprLiteral.value)) {
-        return [];
-    }
-    if (
-        contextProvider.validateReferences(
-            exprLiteral.value.split(".")[0]!,
-            defaultRefValidationFunction,
-        )
-    ) {
-        return [];
-    }
-    return [
-        createDiagnostic(
-            SepticDiagnosticLevel.warning,
-            algPositionTransformer(expr.start, expr.end),
-            `Reference to undefined variable: ${exprLiteral.value}`,
-            SepticDiagnosticCode.missingReference,
-        ),
-    ];
-}
-
 function isAlgExprObjectReference(expr: AlgExpr) {
     return expr instanceof AlgLiteral && expr.type === AlgTokenType.identifier;
 }

packages/septic/src/test/diagnostics.test.ts

@@ -197,6 +197,21 @@ describe("Test diagnostics for references in algs", () => {
             SepticDiagnosticCode.unknownPublicProperty,
         );
     });
+    it("Expect diagnostics for unknown property when variable is first param in calc", () => {
+        const text = `
+            Mvr: TestMvr
+			CalcPvr:  TestCalcPvr 
+				Text1= "Test"
+				Alg= "-abs(TestMvr.Tests)"
+		`;
+
+        const cnfg = parseSepticForTest(text);
+        const diag = validateAlgs(cnfg, cnfg);
+        expect(diag.length).to.equal(1);
+        expect(diag[0]!.code).to.equal(
+            SepticDiagnosticCode.unknownPublicProperty,
+        );
+    });
 });
 
 describe("Test datatype diagnostics in algs", () => {