- minor bug and implementation fixes in 'for' loop visit method

- lexer errors and interpreter errors unit tests
- removed redundant parser testcases

src/Interpreter.py

@@ -230,12 +230,13 @@ def validate_range_node(range_node):
             @return int: The validated range value.
             @raise InterpreterError: If the range value is not an integer.
             """
-            try:
-                runtime_object = range_node.accept(self)
-                return int(runtime_object.value)
-            except ValueError:
-                raise InterpreterError(ErrorType.RUNTIME, "Range value must be integers",
+            runtime_object = range_node.accept(self)
+            if type(runtime_object.value) is not int:
+                raise InterpreterError(ErrorType.RUNTIME,
+                                       f"Range value '{type(runtime_object.value).__name__}' "
+                                       f"cannot be used as an integer",
                                        range_node.start_pos, range_node.end_pos)
+            return runtime_object.value
 
         range_start = validate_range_node(node.range_start)
         range_end = validate_range_node(node.range_end)
@@ -487,12 +488,10 @@ def visit_factor(self, node: 'FactorNode'):
                 return RunTimeObject("boolean", not right_factor.value)
 
             elif left_factor.value == '-':
-                if right_factor.label == "number":
+                try:
                     return RunTimeObject("number", -right_factor.value)
-
-                raise InterpreterError(ErrorType.TYPE,
-                                       f"Value '{right_factor.value}' of variable is not a numeric type",
-                                       self.node_start_pos, self.node_end_pos)
+                except TypeError as e:
+                    raise InterpreterError(ErrorType.TYPE, e.args[0], self.node_start_pos, self.node_end_pos)
         return left_factor
 
     @staticmethod

src/Parser.py

@@ -230,9 +230,6 @@ def parse_statement(self):
                 statement_node = self.parse_print()
             elif self.match(TokenType.INPUT):
                 statement_node = self.parse_input()
-            else:
-                raise ParserError("Unexpected token: " + self.curr_tkn.value,
-                                  self.curr_tkn.line_num, self.curr_tkn.column_num)
 
         statement_node.start_pos = start_pos
         statement_node.end_pos = self.curr_tkn.pos

test.py

@@ -19,7 +19,7 @@ def setUp(self):
         self.lexer = Lexer()
         self.parser = Parser()
         self.interpreter = Interpreter()
-        self.num_random_tests = 50000
+        self.num_random_tests = 25000
 
     def tearDown(self):
         self.lexer = None
@@ -48,12 +48,47 @@ def test_lexer(self):
                 print(e, file=sys.stderr)
                 self.fail()
 
+    def test_lexer_errors(self):
+        header = "Testing Lexer Errors"
+        print(header)
+        print(f"-" * len(header))
+
+        test_dir = os.path.join(self.test_dir, "errors/lexer/")
+        for file in os.listdir(test_dir):
+            if not file.endswith(".lex"):
+                continue
+
+            output_file = file.replace(".lex", ".out")
+            with open(os.path.join(test_dir, output_file)) as f:
+                expected = f.read()
+
+            print(f"[Lexer Error] Running test on: {file}")
+            try:
+                self.lexer.analyze_src_file(test_dir + file)
+
+                token = Token()
+                while token.type != TokenType.ENDMARKER:
+                    token = self.lexer.get_token()
+                    print(" ", token)
+
+                self.fail()
+            except Exception as e:
+                if expected.lower() not in str(e).lower():
+                    print(expected in str(e))
+                    print(e)
+                    self.fail(f"EXPECTED:\n"
+                              f"    {expected}\n"
+                              f"ACTUAL:\n"
+                              f"    {e}")
+            finally:
+                self.lexer.__init__()
+
     def test_parser(self):
         header = "Testing Parser"
         print(header)
         print(f"-" * len(header))
 
-        test_dir = os.path.join(self.test_dir, "parser/")
+        test_dir = os.path.join(self.test_dir, "interpreter/in/")
         for file in os.listdir(test_dir):
             print(f"[Parser] Running test on: {file}")
 
@@ -112,6 +147,33 @@ def test_interpreter(self):
         if failed:
             self.fail()
 
+    def test_interpreter_errors(self):
+        header = "Testing Interpreter Errors"
+        print(header)
+        print(f"-" * len(header))
+
+        test_dir = os.path.join(self.test_dir, "errors/interpreter/")
+        for file in os.listdir(test_dir):
+            if not file.endswith(".ny"):
+                continue
+
+            output_file = file.replace(".ny", ".out")
+            with open(os.path.join(test_dir, output_file)) as f:
+                expected = f.read()
+
+            proc = subprocess.run(
+                ["python3", "nyaa.py", test_dir + file],
+                capture_output=True,
+                text=True
+            )
+
+            print(f"[Interpreter Error] Running test on: {file}")
+            if expected.lower().strip() not in str(proc.stderr).lower().strip():
+                self.fail(f"EXPECTED:\n"
+                          f"    {expected}\n"
+                          f"ACTUAL:\n"
+                          f"    {proc.stderr}")
+
     def test_operator_precedence_expressions(self):
         header = "Testing Operator Precedence Expressions"
         print(header)
@@ -167,7 +229,7 @@ def test_prioritized_expressions(self):
 
         start = -sys.maxsize - 1
         end = sys.maxsize
-        for _ in range(self.num_random_tests // 10):
+        for _ in range(self.num_random_tests // 2):
             a = random.randint(start, end)
             b = random.randint(start, end)
             c = random.randint(start, end)

tests/errors/lexer/identifier_too_long.lex

@@ -0,0 +1 @@
+eiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouv

tests/errors/lexer/identifier_too_long.out

@@ -0,0 +1 @@
+Identifier exceeds the maximum length

tests/errors/lexer/invalid_escape_char.lex

@@ -0,0 +1 @@
+"\0"

tests/errors/lexer/invalid_escape_char.out

@@ -0,0 +1 @@
+Invalid escape character

tests/errors/lexer/string_too_long.lex

@@ -0,0 +1 @@
+"eiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouveiohfoiefhioehfiofheiohfwegvbvjbveuirbevoiubfvouv"

tests/errors/lexer/string_too_long.out

@@ -0,0 +1 @@
+String too long

tests/errors/lexer/unrecognized_char.lex

@@ -0,0 +1 @@
+/?\@^\|

tests/errors/lexer/unrecognized_char.out

@@ -0,0 +1 @@
+Unrecognized character

tests/errors/lexer/unterminated_string.lex

@@ -0,0 +1 @@
+"dfkfgoiggghmgh

tests/errors/lexer/unterminated_string.out

@@ -0,0 +1 @@
+Unterminated string

tests/interpreter/in/for.ny

@@ -25,17 +25,18 @@ uWu_nyaa() => {
     }
     yomu()
 
-    for i => (1 supuritto 1, 10) {
+    a wa 10
+    for i => (1, a) {
         yomu(i)
     }
     yomu()
 
-    for i => (IIE, HAI) {
+    for i => (0, 1) {
         yomu(i)
     }
     yomu()
 
-    for _ => (HAI, IIE) {
+    for _ => (1, 0) {
         yomu(_)
     }
     yomu(_)