Fixed negative list indices

wordplaydev · Oct 28, 2023 · f1c6a14 · f1c6a14
1 parent b70dee6
commit f1c6a14
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/nodes/ListAccess.test.ts b/src/nodes/ListAccess.test.ts
@@ -16,10 +16,11 @@ test.each([
 
 test.each([
     ['[1 2 3][2]', '2'],
+    ['[1 2 3][5]', '2'],
     ['[1 2 3][0]', 'ø'],
     ['[1 2 3][-1]', '3'],
     ['[1 2 3][-3]', '1'],
-    ['[1 2 3][-4]', 'ø'],
+    ['[1 2 3][-4]', '3'],
 ])('Expect %s to be %s', (code, value) => {
     expect(evaluateCode(code)?.toString()).toBe(value);
 });
diff --git a/src/values/ListValue.ts b/src/values/ListValue.ts
@@ -32,7 +32,7 @@ export default class ListValue extends SimpleValue {
             num === 0
                 ? undefined
                 : this.values.at(
-                      num > 0 ? (num - 1) % this.values.length : num
+                      (num > 0 ? num - 1 : num) % this.values.length
                   );
         return value === undefined ? new NoneValue(this.creator) : value;
     }