fix: update chromedriver download URL, resolve test issues
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_cargoaccept both the old and new formats. - With MacOS 14 (and by extension, M1) runners becoming the
macos-latestdefault, the current geckodriver setup fails for that platform. ~It's not entirely obvious to me why at this point. Downgrading to themacos-13runner 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
rustfmtinstalled
$ rustup component add rustfmt
- [x] You ran
cargo fmton 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
Seems like there's some unfortunate caching issues I'll have to dig into.
Seems like there's some unfortunate caching issues I'll have to dig into.
Thank you for looking into this! Appreciated!
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.