electron tests

[khassel]

• fix setup to run with xserver and labwc
• remove electronParams from calendar_spec.js (forgotten in electron tests: fixes for running under new github image ubuntu-24.04 #3680)

fixes running tests locally without labwc installed

follow up to #3680

[sdetweil]

I pulled this pr, and still see problems launching electron

npm run test:electron

  Development console tests
    ✕ should open browserwindow and dev console (772 ms)

  ● Electron app environment › should open browserwindow

    electron.launch: Process failed to launch!

      21 | 	}
      22 |
    > 23 | 	global.electronApp = await electron.launch({ args: electronParams });
         | 	                     ^
      24 |
      25 | 	await global.electronApp.firstWindow();
      26 |

      at Object.<anonymous>.exports.startApplication (tests/electron/helpers/global-setup.js:23:23)
      at Object.<anonymous> (tests/electron/env_spec.js:6:3)

  ● Development console tests › should open browserwindow and dev console

cal tests fail too
I did a new npm install , on node 20.18.1

[khassel]

and npm start works?

[sdetweil]

yes without problem

[khassel]

is DISPLAY variable set?

Will convert to draft and test on an xserver system ...

[sdetweil]

yes, thru npm start. which starts x11 same as before Sam

…

On Sun, Jan 5, 2025 at 4:36 PM Karsten Hassel ***@***.***> wrote: is DISPLAY variable set? Will convert to draft and test on an xserver system ... — Reply to this email directly, view it on GitHub <#3684 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALNYHXTMM3TV2JVCJUVMI32JGXXBAVCNFSM6AAAAABUUPSTJ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZRG43TGNZRHE> . You are receiving this because you commented.Message ID: ***@***.***>

[khassel]

@sdetweil can you please check again with last changes, thx!

[sdetweil]

browser starting, electron test successful!!