Skip to content

Commit 813af85

Browse files
committed
Fix tests for fixed AUTOBUILD_WIN_VSTOOLSET setting.
The test was passing AUTOBUILD_VSVER=171, expecting the previous behavior that AUTOBUILD_WIN_VSTOOLSET would *not* be set. But that's exactly what the previous commit is intended to fix: we want AUTOBUILD_VSVER=171 to behave like AUTOBUILD_VSVER=170 and emit the corresponding AUTOBUILD_WIN_VSTOOLSET. Instead, use an old AUTOBUILD_VSVER=120, which predates the table in autobuild_tool_source_environment.py.
1 parent f87301f commit 813af85

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/test_source_environment.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,6 @@ def test_vstoolset_set(self):
326326

327327
@needs_cygwin
328328
def test_vstoolset_not_set_if_vsver_unrecognized(self):
329-
with envvar("AUTOBUILD_VSVER", "171"):
329+
with envvar("AUTOBUILD_VSVER", "120"):
330330
vars = self.read_variables(self.find_data("empty"))
331331
self.assertTrue("AUTOBUILD_WIN_VSTOOLSET" not in vars)

0 commit comments

Comments
 (0)