Skip to content

Add workaround for cargo-nextest bug#376

Merged
ipetkov merged 3 commits into
ipetkov:masterfrom
9999years:nextest-macos
Aug 30, 2023
Merged

Add workaround for cargo-nextest bug#376
ipetkov merged 3 commits into
ipetkov:masterfrom
9999years:nextest-macos

Conversation

@9999years

@9999years 9999years commented Aug 30, 2023

Copy link
Copy Markdown
Contributor

Prevents this error when running tests with craneLib.cargoNextest:

(...)
dyld: Library not loaded: @rpath/libtest-77ee8c29c330e4a3.dylib
  Referenced from: /private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a
  Reason: image not found
error: creating test list failed
Caused by:
  for `hdk_derive::proc-macro/hdk_derive`, running command `/private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a --list --format terse` failed
Caused by:
  command ["/private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a", "--list", "--format", "terse"] exited with code <signal 6>
Error: Process completed with exit code 104.

See: nextest-rs/nextest#267

Checklist

  • added tests to verify new behavior
  • added an example template or updated an existing one
  • updated docs/API.md (or general documentation) with changes
  • updated CHANGELOG.md

Prevents this error:

```
(...)
dyld: Library not loaded: @rpath/libtest-77ee8c29c330e4a3.dylib
  Referenced from: /private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a
  Reason: image not found
error: creating test list failed
Caused by:
  for `hdk_derive::proc-macro/hdk_derive`, running command `/private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a --list --format terse` failed
Caused by:
  command ["/private/tmp/holochain_repo/target/fast-test/deps/hdk_derive-73ec051829ad694a", "--list", "--format", "terse"] exited with code <signal 6>
Error: Process completed with exit code 104.
```

nextest-rs/nextest#267
@9999years 9999years marked this pull request as ready for review August 30, 2023 23:22
@ipetkov

ipetkov commented Aug 30, 2023

Copy link
Copy Markdown
Owner

Thanks for the contribution @9999years and thanks for adding the test! Happy to merge this as soon as CI is passing

(worth noting that we don't run the full test suite on darwin, but we do exercise the examples which include a nextest run so this should smoke test just fine)

@ipetkov ipetkov enabled auto-merge (squash) August 30, 2023 23:32
@ipetkov ipetkov merged commit 1746047 into ipetkov:master Aug 30, 2023
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