Skip to content

Commit 5d0c277

Browse files
committed
Stop using a RAM disk for the Windows tests
1 parent 2144008 commit 5d0c277

File tree

1 file changed

+8
-19
lines changed

1 file changed

+8
-19
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -167,24 +167,13 @@ jobs:
167167
with:
168168
python-version: ${{ matrix.python }}
169169

170-
# We use a RAMDisk on Windows, since filesystem IO is a big slowdown
171-
# for our tests.
172-
- name: Create a RAMDisk
173-
run: ./tools/ci/New-RAMDisk.ps1 -Drive R -Size 1GB
174-
175-
- name: Setup RAMDisk permissions
176-
run: |
177-
mkdir R:\Temp
178-
$acl = Get-Acl "R:\Temp"
179-
$rule = New-Object System.Security.AccessControl.FileSystemAccessRule(
180-
"Everyone", "FullControl", "ContainerInherit,ObjectInherit", "None", "Allow"
181-
)
182-
$acl.AddAccessRule($rule)
183-
Set-Acl "R:\Temp" $acl
184-
170+
# We use C:\Temp (which is already available on the worker)
171+
# as a temporary directory for all of the tests because the
172+
# default value (under the user dir) is more deeply nested
173+
# and causes tests to fail with "path too long" errors.
185174
- run: pip install nox
186175
env:
187-
TEMP: "R:\\Temp"
176+
TEMP: "C:\\Temp"
188177

189178
# Main check
190179
- name: Run unit tests
@@ -194,7 +183,7 @@ jobs:
194183
-m unit
195184
--verbose --numprocesses auto --showlocals
196185
env:
197-
TEMP: "R:\\Temp"
186+
TEMP: "C:\\Temp"
198187

199188
- name: Run integration tests (group 1)
200189
if: matrix.group == 1
@@ -203,7 +192,7 @@ jobs:
203192
-m integration -k "not test_install"
204193
--verbose --numprocesses auto --showlocals
205194
env:
206-
TEMP: "R:\\Temp"
195+
TEMP: "C:\\Temp"
207196

208197
- name: Run integration tests (group 2)
209198
if: matrix.group == 2
@@ -212,7 +201,7 @@ jobs:
212201
-m integration -k "test_install"
213202
--verbose --numprocesses auto --showlocals
214203
env:
215-
TEMP: "R:\\Temp"
204+
TEMP: "C:\\Temp"
216205

217206
tests-zipapp:
218207
name: tests / zipapp

0 commit comments

Comments
 (0)