Skip to content

Commit

Permalink
chore(pocket-ic): bazel test requires-network (#3182)
Browse files Browse the repository at this point in the history
Adding `requires-network` tag to overcome the issue of sporadic failures
(seen [here](https://github.com/dfinity/ic/actions/runs/12320821672),
attempt 1 & 2):
```
 Failed to get result: reqwest::Error { kind: Request, url: "http://127.0.0.1:49325/instances", source: hyper_util::client::legacy::Error(Connect, ConnectError("tcp connect error", Os { code: 1, kind: PermissionDenied, message: "Operation not permitted" })) }
```
  • Loading branch information
marko-k0 authored Dec 13, 2024
1 parent 5da8abf commit 3e77cd2
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions rs/pocket_ic_server/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -189,8 +189,7 @@ rust_test(
},
tags = [
"cpu:8",
# TODO: enable "test_macos" again when we have fixed the following error on Apple Silicon:
#
# TODO: remove 'requires-network' tag when the root cause for sporadic error below on Apple Silicon is identified and fixed.
# ---- test_http_gateway stdout ----
# thread 'test_http_gateway' panicked at rs/pocket_ic_server/tests/test.rs:383:48:
# called `Result::unwrap()` on an `Err` value: reqwest::Error {
Expand All @@ -200,8 +199,8 @@ rust_test(
# ConnectError("tcp connect error", Os { code: 1, kind: PermissionDenied, message: "Operation not permitted" })
# )
# }
#
#"test_macos",
"requires-network",
"test_macos",
],
deps = TEST_DEPENDENCIES,
)
Expand Down

0 comments on commit 3e77cd2

Please sign in to comment.