Skip to content

Conversation

@mathiasrw
Copy link
Member

Thank you for the time you are putting into AlaSQL!

@mathiasrw mathiasrw changed the title Use Bun for test suite #2172 round 2 Use Bun for test suite round 2 Nov 14, 2025
@mathiasrw
Copy link
Member Author

@copilot Im having this issue in the pipeline. Can you help us?


test/test298.test.js:
  (pass) Test 298 PLUG-IN TEST > 1. CREATE DATABASE [0.15ms]
  
  Error: Expected constructor: Error
  
  Received function did not throw
  Received value: undefined
  
        at <anonymous> (/home/runner/_work/alasql/alasql/test/test298.test.js:18:6)
  13 | 
  14 | 	test('2.REQURE ECHO plugin', done => {
  15 | 		expect(() => {
  16 | 			var res = alasql('ECHO 1');
  17 | 			//      console.log(1,res);
  18 | 		}).toThrow(Error);
            ^
  error: expect(received).toThrow(expected)
  
  Expected constructor: Error
  
  Received function did not throw
  Received value: undefined
  
        at <anonymous> (/home/runner/_work/alasql/alasql/test/test298.test.js:18:6)
  (fail) Test 298 PLUG-IN TEST > 2.REQURE ECHO plugin [0.21ms]
  Ran 777 tests across 225 files. [1374.00ms]
  Bailed out after 1 failure
  error: script "test-only" exited with code 1

Copy link
Contributor

Copilot AI commented Nov 19, 2025

@mathiasrw I've opened a new pull request, #2181, to work on those changes. Once the pull request is ready, I'll request review from you.

@mathiasrw
Copy link
Member Author

@copilot

  1690 passing (3s)
  408 pending
  1 failing

  1) Test 925 - UNIQUE JSON property
       E) Multiple indexes (unique and regular) on same table:
     SyntaxError: Parse error on line 1:
..._data on BOM5(data->value)
-----------------------^
Expecting 'LITERAL', 'BRALITERAL', 'LPAR', 'NUMBER', 'IF', 'CURRENT_DATE', 'REPLACE', 'DATEADD', 'DATEDIFF', 'TIMESTAMPDIFF', 'INTERVAL', got 'VALUE'
      at parser.parseError (dist/alasql.fs.js:2257:8)
      at Parser.parse (dist/alasql.fs.js:2131:22)
      at alasql.parse (dist/alasql.fs.js:4518:22)
      at alasql.dexec (dist/alasql.fs.js:4774:16)
      at alasql.exec (dist/alasql.fs.js:4730:17)
      at alasql (dist/alasql.fs.js:137:17)
      at Context.<anonymous> (test/test925.js:108:3)
      at process.processImmediate (node:internal/timers:483:21)



error Command failed with exit code 1.

Copy link
Contributor

Copilot AI commented Nov 19, 2025

@mathiasrw I've opened a new pull request, #2199, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants