wasm-pack icon indicating copy to clipboard operation
wasm-pack copied to clipboard

fix: update chromedriver download URL, resolve test issues

Open kade-robertson opened this issue 1 year ago • 1 comments

This fixes a few things to get CI working for the repo again:

  • The chromedriver releases were previously 404ing, causing these tests to fail. The URL for where these are hosted was changed, so it's just a simple update to get this working again.
  • In later Cargo releases, the output format for the "Finished..." log line changed. I've fixed this by making matches_cargo accept both the old and new formats.
  • With MacOS 14 (and by extension, M1) runners becoming the macos-latest default, the current geckodriver setup fails for that platform. ~It's not entirely obvious to me why at this point. Downgrading to the macos-13 runner has these tests working again. This is probably the most.. questionable change as it'd be better to track down why the latest runners don't work.~ Edit: It seems like the only thing preventing macOS from working was that I needed to manually install Firefox. I'm guessing this was available in the previous macOS runners by default.

Make sure these boxes are checked! 📦✅

  • [x] You have the latest version of rustfmt installed
$ rustup component add rustfmt
  • [x] You ran cargo fmt on the code base before submitting
  • [x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Closes #1390

kade-robertson avatar May 14 '24 19:05 kade-robertson

Seems like there's some unfortunate caching issues I'll have to dig into.

kade-robertson avatar May 14 '24 21:05 kade-robertson

Seems like there's some unfortunate caching issues I'll have to dig into.

Thank you for looking into this! Appreciated!

drager avatar May 23 '24 06:05 drager

Thank you for looking into this! Appreciated!

This issue seems to be resolved now for the sake of this PR. However, in general I've found some consistency issues with trying to determine what paths to cache when manually in many of my other projects. Are you open to switching to something like Swatinem/rust-cache or mozilla-actions/sccache-action? I've used both successfully quite a bit, though recently all of my projects have moved to the latter. If so I can open up another PR after this is cleared to merge to switch out the caching.

kade-robertson avatar May 23 '24 17:05 kade-robertson