Skip to content

Commit

Permalink
tests: add publishing w/ pypi token via env vars
Browse files Browse the repository at this point in the history
  • Loading branch information
abn committed Mar 5, 2024
1 parent 374babb commit 57af6ed
Showing 1 changed file with 20 additions and 5 deletions.
25 changes: 20 additions & 5 deletions tests/publishing/test_publisher.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,16 +91,17 @@ def test_publish_raises_error_for_undefined_repository(
publisher.publish("my-repo", None, None)


def test_publish_uses_token_if_it_exists(
fixture_dir: FixtureDirGetter, mocker: MockerFixture, config: Config
def assert_publish_uses_token_if_it_exists(
fixture_dir: FixtureDirGetter, mocker: MockerFixture, config: Config | None = None
) -> None:
uploader_auth = mocker.patch("poetry.publishing.uploader.Uploader.auth")
uploader_upload = mocker.patch("poetry.publishing.uploader.Uploader.upload")
poetry = Factory().create_poetry(fixture_dir("sample_project"))
poetry._config = config
poetry.config.merge({"pypi-token": {"pypi": "my-token"}})
publisher = Publisher(poetry, NullIO())

if config:
poetry._config = config

publisher = Publisher(poetry, NullIO())
publisher.publish(None, None, None)

assert [("__token__", "my-token")] == uploader_auth.call_args
Expand All @@ -110,6 +111,20 @@ def test_publish_uses_token_if_it_exists(
] == uploader_upload.call_args


def test_publish_uses_token_if_it_exists(
fixture_dir: FixtureDirGetter, mocker: MockerFixture, config: Config
) -> None:
config.merge({"pypi-token": {"pypi": "my-token"}})
assert_publish_uses_token_if_it_exists(fixture_dir, mocker, config)


def test_publish_uses_env_token_if_it_exists(
fixture_dir: FixtureDirGetter, mocker: MockerFixture, environ: None
) -> None:
os.environ["POETRY_PYPI_TOKEN_PYPI"] = "my-token"
assert_publish_uses_token_if_it_exists(fixture_dir, mocker)


def test_publish_uses_cert(
fixture_dir: FixtureDirGetter, mocker: MockerFixture, config: Config
) -> None:
Expand Down

0 comments on commit 57af6ed

Please sign in to comment.