Skip to content

Commit

Permalink
Test OSX build
Browse files Browse the repository at this point in the history
  • Loading branch information
Fokko committed Feb 13, 2024
1 parent 8ec62c1 commit f72e4f5
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/python-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ ubuntu-22.04, windows-2022, macos-11 ]
os: [ macos-11, macos-12, macos-13, macos-14 ]

steps:
- uses: actions/checkout@v4
Expand All @@ -43,7 +43,7 @@ jobs:

- uses: actions/setup-python@v5
with:
python-version: '3.8'
python-version: '3.11'

- name: Install poetry
run: pip install poetry
Expand Down Expand Up @@ -82,5 +82,5 @@ jobs:

- uses: actions/upload-artifact@v3
with:
name: "release-${{ matrix.os }}"
name: "release-${{ github.event.inputs.version }}"
path: ./wheelhouse/*
7 changes: 7 additions & 0 deletions tests/utils/test_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,13 @@ def test_from_environment_variables_uppercase() -> None:
assert Config().get_catalog_config("PRODUCTION") == {"uri": "https://service.io/api"}


@mock.patch.dict(os.environ, {
"PYICEBERG_CATALOG__PRODUCTION__S3__REGION": "eu-north-1"
})
def test_from_environment_variables_uppercase() -> None:
assert Config().get_catalog_config("PRODUCTION") == {"uri": "https://service.io/api"}


def test_from_configuration_files(tmp_path_factory: pytest.TempPathFactory) -> None:
config_path = str(tmp_path_factory.mktemp("config"))
with open(f"{config_path}/.pyiceberg.yaml", "w", encoding=UTF8) as file:
Expand Down

0 comments on commit f72e4f5

Please sign in to comment.