-
Notifications
You must be signed in to change notification settings - Fork 329
175 lines (149 loc) · 4.45 KB
/
test_cli.yaml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
name: cli test
on:
push:
branches: [main]
pull_request:
env:
TURBO_TOKEN: ${{ secrets.TURBO_TOKEN }}
TURBO_TEAM: marimo
MARIMO_SKIP_UPDATE_CHECK: 1
jobs:
changes:
runs-on: ubuntu-latest
outputs:
cli: ${{ steps.filter.outputs.cli }}
steps:
- uses: actions/checkout@v4
- uses: dorny/paths-filter@v3
id: filter
with:
filters: |
cli:
- '**'
- '!docs/**'
- '!examples/**'
- '!README.md'
build_wheel:
needs: changes
if: ${{ needs.changes.outputs.cli == 'true' }}
name: Build marimo wheel
runs-on: ubuntu-latest
timeout-minutes: 10
defaults:
run:
shell: bash
steps:
- name: 🛑 Cancel Previous Runs
uses: styfle/[email protected]
- name: ⬇️ Checkout repo
uses: actions/checkout@v4
with:
fetch-depth: 0 # so we can run --since on the main branch and turbo can do faster cache hashing
- uses: pnpm/action-setup@v2
with:
version: 9
- name: ⎔ Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- name: 📦 Build frontend
run: make fe
env:
NODE_OPTIONS: '--max_old_space_size=8192'
- name: 🥚 Install Hatch
uses: pypa/hatch@install
- name: 📦 Build marimo wheel
run: hatch build -t wheel
- name: Check for _static/_lsp directory in wheel
run: |
wheel_file=$(ls dist/*.whl)
files_or_dirs_to_check=(
"marimo/_static/"
"marimo/_lsp/"
"marimo/_tutorials/"
"third_party.txt"
"third_party_licenses.txt"
)
for file_or_dir in "${files_or_dirs_to_check[@]}"; do
if ! unzip -l "$wheel_file" | grep -q "$file_or_dir"; then
echo "Error: $file_or_dir not found in the wheel"
exit 1
else
echo "Found $file_or_dir in the wheel"
fi
done
- name: Upload wheel
uses: actions/upload-artifact@v4
with:
name: testwheel
path: dist/*.whl
retention-days: 1
test_cli:
needs: [changes, build_wheel]
if: ${{ needs.changes.outputs.cli == 'true' }}
name: Tests CLI on ${{ matrix.os }} ${{ matrix.python-version }}
runs-on: ${{ matrix.os }}
timeout-minutes: 15
strategy:
matrix:
# TODO(akshayka): consider adding 3.8, 3.9
python-version: ['3.12']
os: [ubuntu-latest, macos-latest, windows-latest]
steps:
- name: 🛑 Cancel Previous Runs
uses: styfle/[email protected]
- uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: 'pip'
- name: Install Python deps
run: |
python -m pip install --upgrade pip
pip install pytest pytest-asyncio nbformat
- name: Download wheel
uses: actions/download-artifact@v4
with:
name: testwheel
- name: Install marimo
shell: bash
run: pip install marimo*whl
- name: Test CLI
shell: bash
run: |
pytest -v tests/_cli/test_cli* --maxfail=2
test_examples:
needs: [changes, build_wheel]
if: ${{ needs.changes.outputs.cli == 'true' }}
name: Tests examples and smoke_tests
runs-on: ubuntu-latest
timeout-minutes: 15
steps:
- name: 🛑 Cancel Previous Runs
uses: styfle/[email protected]
- uses: actions/checkout@v4
- name: Set up Python 3.12
uses: actions/setup-python@v5
with:
python-version: '3.12'
cache: 'pip'
- name: 🐍 Setup uv
uses: yezz123/setup-uv@v4
with:
uv-venv: marimo-venv
- name: Download wheel
uses: actions/download-artifact@v4
with:
name: testwheel
- name: Install Python deps
run: |
uv pip install pytest pytest-asyncio
uv pip install ./marimo*whl
- name: Test examples
shell: bash
run: pytest -v -s tests/_smoke_tests/run_all.py
# TODO(msconick) remove continue-on-error when all tests pass
continue-on-error: true