Merge pull request #9 from QSPFoundation/tests/ExtraTestsToCheckParam…

…eterConversion Add extra tests to check type conversion
QSPFoundation · Nov 27, 2024 · f455184 · f455184
2 parents bb18bf2 + 82f26fb
commit f455184
Show file tree

Hide file tree

Showing 7 changed files with 2,772 additions and 4,258 deletions.
diff --git a/src/qsplib/public/qsp-engine-debug.js b/src/qsplib/public/qsp-engine-debug.js
diff --git a/src/qsplib/public/qsp-engine-debug.wasm b/src/qsplib/public/qsp-engine-debug.wasm
diff --git a/src/qsplib/public/qsp-engine-debug.wasm.map b/src/qsplib/public/qsp-engine-debug.wasm.map
diff --git a/src/qsplib/public/qsp-engine.js b/src/qsplib/public/qsp-engine.js
diff --git a/src/qsplib/public/qsp-engine.wasm b/src/qsplib/public/qsp-engine.wasm
diff --git a/tests/api.test.ts b/tests/api.test.ts
@@ -39,7 +39,7 @@ describe('api', () => {
     expect(api.readVariableByKey('test', 'тест')).toBe(254);
   });
 
-  it('should read numeric variable in cyrilic', async () => {
+  it('should read numeric variable in cyrillic', async () => {
     runTestFile(api, `тест = 254`);
 
     expect(api.readVariable('тест')).toBe(254);
@@ -68,7 +68,7 @@ describe('api', () => {
     expect(api.readVariableByKey('$test', 'тест')).toBe('252');
   });
 
-  it('should read string variable in cyrilic', async () => {
+  it('should read string variable in cyrillic', async () => {
     runTestFile(api, `$тест = '254'`);
 
     expect(api.readVariable('$тест')).toBe('254');
@@ -108,7 +108,7 @@ describe('api', () => {
   });
 
   it('should read version', () => {
-    expect(api.version()).toEqual('5.9.0');
+    expect(api.version()).toEqual('5.9.1');
   });
 
   it('should watch variable by index', async () => {

diff --git a/tests/code.test.ts b/tests/code.test.ts
@@ -34,4 +34,60 @@ end
 
     expect(api.readVariable('$type')).toBe('new');
   });
+
+  test('statement argument gets converted to string', () => {
+    const onMsg: Mock = vi.fn();
+    api.on('msg', onMsg);
+
+    runTestFile(api, ` msg 567 ` );
+    expect(onMsg).toHaveBeenCalledWith('567', expect.any(Function));
+    onMsg.mock.calls[0][1]();
+  });
+
+  test('statement argument gets converted to number', () => {
+    const onWait: Mock = vi.fn();
+    api.on('wait', onWait);
+
+    runTestFile(api, ` wait '567' `);
+    expect(onWait).toHaveBeenCalledWith(567, expect.any(Function));
+    onWait.mock.calls[0][1]();
+  });
+
+  test('statement argument cant be converted to number', () => {
+    runTestFile(api, ` wait 'sdsd' `);
+    expect(error).toHaveBeenCalledWith({
+      errorCode: 11,
+      description: 'Type mismatch!',
+      location: 'test',
+      actionIndex: -1,
+      line: 1,
+      localLine: 1,
+      lineSrc: "WAIT 'sdsd'"
+    });
+    error.mockReset();
+  });
+
+  test('function argument gets converted to string', () => {
+    runTestFile(api, ` $x = replace(123456, 456, 34) `);
+    expect(api.readVariable('$x')).toBe('12334');
+  });
+
+  test('function argument gets converted to number', () => {
+    runTestFile(api, ` x = rgb('12', '234', 32) `);
+    expect(api.readVariable('x')).toBe(-14620148);
+  });
+
+  test('function argument cant be converted to number', () => {
+    runTestFile(api, ` x = rgb('123', '65sd', 56) `);
+    expect(error).toHaveBeenCalledWith({
+      errorCode: 11,
+      description: 'Type mismatch!',
+      location: 'test',
+      actionIndex: -1,
+      line: 1,
+      localLine: 1,
+      lineSrc: "X = RGB('123', '65sd', 56)"
+    });
+    error.mockReset();
+  });
 });