Skip to content

Commit 8b2cdd3

Browse files
committed
Run integration tests only on internal PRs on all target OSes
1 parent f0ed569 commit 8b2cdd3

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

.github/workflows/validate.yml

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@ on:
55
push:
66
paths-ignore:
77
- '**.md'
8-
pull_request:
9-
paths-ignore:
10-
- '**.md'
118

129
jobs:
1310

1411
validate:
15-
runs-on: ubuntu-latest
12+
strategy:
13+
matrix:
14+
os: [ubuntu-latest, windows-latest, macos-latest]
15+
runs-on: ${{ matrix.os }}
1616
steps:
1717
- uses: actions/checkout@v3
1818

@@ -21,14 +21,26 @@ jobs:
2121
with:
2222
python-version: '3.x'
2323

24-
- name: Test with pytest
24+
- name: Integration Test
25+
if: github.repository_owner == '1Password' # don't run integration tests on forked PRs because those don't have access to pipeline secrets
2526
env:
2627
OP_SERVICE_ACCOUNT_TOKEN: ${{ secrets.TEST_SERVICE_ACCOUNT_TOKEN }}
2728
run: |
2829
pip install pytest &&
2930
pip install pytest-asyncio &&
3031
pip install pydantic &&
31-
python -m pytest src/onepassword/*.py
32+
python -m pytest src/onepassword/test_client.py
33+
34+
lint:
35+
name: Lint
36+
runs-on: ubuntu-latest
37+
steps:
38+
- uses: actions/checkout@v3
39+
40+
- name: Set up Python
41+
uses: actions/setup-python@v4
42+
with:
43+
python-version: '3.x'
3244

3345
- name: Lint with Ruff
3446
run: |

0 commit comments

Comments
 (0)