Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI builds fail on a no-op #338

Closed
reedy opened this issue Nov 3, 2023 · 3 comments
Closed

CI builds fail on a no-op #338

reedy opened this issue Nov 3, 2023 · 3 comments

Comments

@reedy
Copy link
Contributor

reedy commented Nov 3, 2023

On #337 which is a no-op, the CI builds fail for various reasons - https://github.com/tympanix/Electorrent/actions/runs/6743622454

@tympanix
Copy link
Owner

tympanix commented Nov 7, 2023

Hi @reedy

It seems the CI tests degraded over time without any source code changes. I need to figure out what in the CI environment changed

@tympanix
Copy link
Owner

tympanix commented Nov 7, 2023

The acceptance tests regressed because; som docker images used "latest" and broke when new images where published (nginx), electron-builder did not support python3 and python2 was removed for "macos-latest" on Github Actions

@tympanix tympanix closed this as completed Nov 7, 2023
@reedy
Copy link
Contributor Author

reedy commented Nov 20, 2023

Still a test failure outstanding...

Run xvfb-run -a -- npm test -- -g transmission

> [email protected] test /home/runner/work/Electorrent/Electorrent
> mocha 'test/**/*.spec.ts' "-g" "transmission"



  given transmission service is running (docker-compose)
    given tls/ssl reverse proxy is running (docker-compose)
      given application is running
        1) "before all" hook for "user is logging in with https"


  0 passing (50s)
  1 failing

  1) given transmission service is running (docker-compose)
       given tls/ssl reverse proxy is running (docker-compose)
         given application is running
           "before all" hook for "user is logging in with https":
     Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/Electorrent/Electorrent/test/fixtures/transmission/transmission.spec.ts)
      at listOnTimeout (internal/timers.js:557:17)
      at processTimers (internal/timers.js:500:7)

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

No branches or pull requests

2 participants