Skip to content

test: Mark add_tests::add_pypi_git as an online test#3586

Merged
ruben-arts merged 1 commit intoprefix-dev:mainfrom
mgorny:add-pypi-git-online
Apr 15, 2025
Merged

test: Mark add_tests::add_pypi_git as an online test#3586
ruben-arts merged 1 commit intoprefix-dev:mainfrom
mgorny:add-pypi-git-online

Conversation

@mgorny
Copy link
Copy Markdown
Contributor

@mgorny mgorny commented Apr 14, 2025

Mark add_tests::add_pypi_git as requiring Internet, as in isolated environment it fails with:

thread 'add_tests::add_pypi_git' panicked at tests/integration_rust/add_tests.rs:915:30:
called `Result::unwrap()` on an `Err` value:   × Request failed after 3 retries
  ├─▶ error sending request for url (https://prefix.dev/conda-forge/noarch/
  │   repodata_shards.msgpack.zst)
  ├─▶ client error (Connect)
  ├─▶ dns error: failed to lookup address information: Name or service not
  │   known
  ╰─▶ failed to lookup address information: Name or service not known

Mark `add_tests::add_pypi_git` as requiring Internet, as in isolated
environment it fails with:

```
thread 'add_tests::add_pypi_git' panicked at tests/integration_rust/add_tests.rs:915:30:
called `Result::unwrap()` on an `Err` value:   × Request failed after 3 retries
  ├─▶ error sending request for url (https://prefix.dev/conda-forge/noarch/
  │   repodata_shards.msgpack.zst)
  ├─▶ client error (Connect)
  ├─▶ dns error: failed to lookup address information: Name or service not
  │   known
  ╰─▶ failed to lookup address information: Name or service not known
```
@mgorny mgorny changed the title Mark add_tests::add_pypi_git as an online test test: Mark add_tests::add_pypi_git as an online test Apr 14, 2025
Copy link
Copy Markdown
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@ruben-arts ruben-arts merged commit aeff4dc into prefix-dev:main Apr 15, 2025
32 of 33 checks passed
@mgorny mgorny deleted the add-pypi-git-online branch April 15, 2025 12:08
@mgorny
Copy link
Copy Markdown
Contributor Author

mgorny commented Apr 15, 2025

Thanks!

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