From 89667273da25ca94ef612310738e28764e020b74 Mon Sep 17 00:00:00 2001
From: Arian Wardak <arianwardak@gmail.com>
Date: Fri, 23 May 2025 19:26:36 +0200
Subject: [PATCH 1/2] Fix #180. Changed the symbols [] to instead use $&

---
 src/lexer.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lexer.c b/src/lexer.c
index 6db4781..8b557f3 100644
--- a/src/lexer.c
+++ b/src/lexer.c
@@ -229,11 +229,11 @@ static bool IsCharacter() {
       token.type = kTokenDot;
 
       break;
-    case '[':
+    case '$':
       token.type = kTokenLeftBracket;
 
       break;
-    case ']':
+    case '&':
       token.type = kTokenRightBracket;
 
       break;

From 542e6b6e9bc3761adf22386fd70565a26f6ef31f Mon Sep 17 00:00:00 2001
From: Arian Wardak <arianwardak@gmail.com>
Date: Sat, 24 May 2025 03:33:26 +0200
Subject: [PATCH 2/2] Fix #182. Changed conditionals testing to use
 testAssertEqual

---
 tests/main.c    |   1 +
 tests/vm_test.c | 113 +++++++++++++++++++++++++-----------------------
 tests/vm_test.h |   1 +
 3 files changed, 62 insertions(+), 53 deletions(-)

diff --git a/tests/main.c b/tests/main.c
index c9728a0..8438eba 100644
--- a/tests/main.c
+++ b/tests/main.c
@@ -92,6 +92,7 @@ int main() {
   RUN_TEST(TestIfGreaterThanComparison);
   RUN_TEST(TestIfLessThanComparison);
   RUN_TEST(TestElse);
+  RUN_TEST(TestNestedConditionals);
 
   return UNITY_END();
 }
diff --git a/tests/vm_test.c b/tests/vm_test.c
index 37e785d..47ee02e 100644
--- a/tests/vm_test.c
+++ b/tests/vm_test.c
@@ -22,6 +22,15 @@ void TestVMArithmetic(const int expected, const char* code) {
   ResetInterpreterState();
 }
 
+void TestConditionalResult(const int expected, const char* const code) {
+  FillProgramBufferAndParse(code);
+  RunVm();
+
+  TEST_ASSERT_EQUAL(expected, global_variables[0].as.number);
+
+  ResetInterpreterState();
+}
+
 void TestPlus() { TestVMArithmetic(4, "x:int=2+2"); }
 
 void TestMinusVM() { TestVMArithmetic(1, "x:int=3-2"); }
@@ -34,70 +43,68 @@ void TestDivide() { TestVMArithmetic(2, "x:int=4/2"); }
 void TestModulo() { TestVMArithmetic(1, "x:int=5%2"); }
 
 void TestIfTrueExecutesBlock() {
-  FillProgramBufferAndParse("if (1) print(\"yes\") endif");
-
-  bool saw_jump_if_false = false;
-  bool saw_print = false;
-  bool saw_halt = false;
-
-  for (size_t i = 0; i < kInstructionsSize; ++i) {
-    if (instructions[i] == kOpJumpIfFalse) {
-      saw_jump_if_false = true;
-    }
-
-    if (instructions[i] == kOpPrint) {
-      saw_print = true;
-    }
-
-    if (instructions[i] == kOpHalt) {
-      saw_halt = true;
-    }
-  }
-
-  TEST_ASSERT_TRUE_MESSAGE(saw_jump_if_false,
-                           "Missing kOpJumpIfFalse for 'if' check");
-  TEST_ASSERT_TRUE_MESSAGE(saw_print, "Missing kOpPrint inside 'if' block");
-  TEST_ASSERT_TRUE_MESSAGE(saw_halt, "Missing kOpHalt at end");
-
-  RunVm();
+  // NOLINTNEXTLINE(cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors)
+  TestConditionalResult(42,
+                        "x: int = 0"
+                        "if(true)"
+                        "  x = 42"
+                        "else"
+                        "  x = 99"
+                        "endif"
+                        "print(x)");
 }
 
 void TestIfFalseSkipsBlock() {
-  FillProgramBufferAndParse("if (0) print(\"no\") endif");
-
-  bool saw_print = false;
-  bool saw_jump = true;
-
-  for (size_t i = 0; i < kInstructionsSize; ++i) {
-    if (instructions[i] == kOpPrint) {
-      saw_print = true;
-    }
-
-    if (instructions[i] == kOpJump) {
-      saw_jump = false;
-    }
-  }
-
-  TEST_ASSERT_TRUE_MESSAGE(saw_print, "Missing kOpPrint inside 'if' block");
-  TEST_ASSERT_TRUE_MESSAGE(saw_jump, "kOpJump should not exist in bytecode");
-
-  RunVm();
+  TestConditionalResult(0,
+                        "x: int = 0\n"
+                        "if(false)\n"
+                        "  x = 123\n"
+                        "endif\n"
+                        "print(x)");
 }
 
 void TestIfGreaterThanComparison() {
-  FillProgramBufferAndParse("if (5 > 2) print(\"greater\") endif");
-
-  RunVm();
+  TestConditionalResult(1,
+                        "x: int = 0\n"
+                        "if(5 > 2)\n"
+                        "  x = 1\n"
+                        "endif\n"
+                        "print(x)");
 }
 
 void TestIfLessThanComparison() {
-  FillProgramBufferAndParse("if (1 < 0) print(\"should not appear\") endif");
-
-  RunVm();
+  TestConditionalResult(0,
+                        "x: int = 0\n"
+                        "if(1 < 0)\n"
+                        "  x = 1\n"
+                        "endif\n"
+                        "print(x)");
 }
 
 void TestElse() {
-  FillProgramBufferAndParse("if(3>2) print(\"foo\") else print(\"bar\") endif");
+  // NOLINTNEXTLINE(cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors)
+  TestConditionalResult(99,
+                        "x: int = 0\n"
+                        "if(false)\n"
+                        "  x = 42\n"
+                        "else\n"
+                        "  x = 99\n"
+                        "endif\n"
+                        "print(x)");
+}
 
-  RunVm();
+void TestNestedConditionals() {
+  // NOLINTNEXTLINE(cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors)
+  TestConditionalResult(222,
+                        "x: int = 0\n"
+                        "if(true)\n"
+                        "  if(false)\n"
+                        "    x = 111\n"
+                        "  else\n"
+                        "    x = 222\n"
+                        "  endif\n"
+                        "else\n"
+                        "  x = 999\n"
+                        "endif\n"
+                        "print(x)");
 }
diff --git a/tests/vm_test.h b/tests/vm_test.h
index 2830237..07a0c67 100644
--- a/tests/vm_test.h
+++ b/tests/vm_test.h
@@ -11,5 +11,6 @@ void TestIfFalseSkipsBlock();
 void TestIfGreaterThanComparison();
 void TestIfLessThanComparison();
 void TestElse();
+void TestNestedConditionals();
 
 #endif  // CONDITIONALS_TEST_H