fix(tests): normalize Windows path handling in Vim 9 CI

Treat win64 and win32unix as Windows in path and job helpers so compile_commands lookups and async command execution behave consistently on GitHub Actions. Normalize NVCC parsed filenames after absolute expansion to keep expected Windows paths stable in handler tests.

Author: hsanson

ale_linters/cuda/nvcc.vim

@@ -23,7 +23,7 @@ function! ale_linters#cuda#nvcc#HandleNVCCFormat(buffer, lines) abort
         \   'lnum': str2nr(l:match[2]),
         \   'type': l:match[4] =~# 'error' ? 'E' : 'W',
         \   'text': l:match[5],
-        \   'filename': fnamemodify(ale#path#Simplify(l:match[1]), ':p'),
+        \   'filename': ale#path#Simplify(fnamemodify(l:match[1], ':p')),
         \}
 
         if !empty(l:match[3])

autoload/ale/c.vim

@@ -5,7 +5,8 @@ call ale#Set('c_parse_makefile', 0)
 call ale#Set('c_always_make', has('unix') && !has('macunix'))
 call ale#Set('c_parse_compile_commands', 1)
 
-let s:sep = has('win32') ? '\' : '/'
+let s:is_windows = has('win32') || has('win64') || has('win32unix')
+let s:sep = s:is_windows ? '\' : '/'
 
 " Set just so tests can override it.
 let g:__ale_c_project_filenames = ['.git/HEAD', 'configure', 'Makefile', 'CMakeLists.txt']
@@ -386,7 +387,7 @@ function! ale#c#ParseCompileCommandsFlags(buffer, file_lookup, dir_lookup) abort
     let l:file_list = get(a:file_lookup, l:buffer_filename, [])
 
     " We may have to look for /foo/bar instead of C:\foo\bar
-    if empty(l:file_list) && has('win32')
+    if empty(l:file_list) && s:is_windows
         " Try without the drive letter.
         let l:no_drive = ale#path#RemoveDriveLetter(l:buffer_filename)
 
@@ -407,7 +408,7 @@ function! ale#c#ParseCompileCommandsFlags(buffer, file_lookup, dir_lookup) abort
     " Try the absolute path to the directory second.
     let l:dir_list = get(a:dir_lookup, l:dir, [])
 
-    if empty(l:dir_list) && has('win32')
+    if empty(l:dir_list) && s:is_windows
         let l:no_drive_dir = ale#path#RemoveDriveLetter(l:dir)
 
         let l:dir_list = get(a:dir_lookup, l:no_drive_dir, [])
@@ -441,7 +442,7 @@ function! ale#c#ParseCompileCommandsFlags(buffer, file_lookup, dir_lookup) abort
             let l:key = fnamemodify(l:buffer_filename, ':r') . l:suffix
             let l:file_list = get(a:file_lookup, l:key, [])
 
-            if empty(l:file_list) && has('win32')
+            if empty(l:file_list) && s:is_windows
                 let l:file_list = get(
                 \   a:file_lookup,
                 \   ale#path#RemoveDriveLetter(l:key),

autoload/ale/job.vim

@@ -10,6 +10,7 @@
 
 " A setting for wrapping commands.
 let g:ale_command_wrapper = get(g:, 'ale_command_wrapper', '')
+let s:is_windows = has('win32') || has('win64') || has('win32unix')
 
 if !has_key(s:, 'job_map')
     let s:job_map = {}
@@ -199,7 +200,7 @@ function! ale#job#PrepareCommand(buffer, command) abort
         return split(l:ale_shell) + split(l:shell_arguments) + [l:command]
     endif
 
-    if has('win32')
+    if s:is_windows
         return 'cmd /s/c "' . l:command . '"'
     endif
 

autoload/ale/path.vim

@@ -1,6 +1,8 @@
 " Author: w0rp <devw0rp@gmail.com>
 " Description: Functions for working with paths in the filesystem.
 
+let s:is_windows = has('win32') || has('win64') || has('win32unix')
+
 " simplify a path, and fix annoying issues with paths on Windows.
 "
 " Forward slashes are changed to back slashes so path equality works better
@@ -27,7 +29,7 @@ endfunction
 " Simplify a path without a Windows drive letter.
 " This function can be used for checking if paths are equal.
 function! ale#path#RemoveDriveLetter(path) abort
-    return has('win32') && a:path[1:2] is# ':\'
+    return s:is_windows && a:path[1:2] =~# '^:[/\\]$'
     \   ? ale#path#Simplify(a:path[2:])
     \   : ale#path#Simplify(a:path)
 endfunction
@@ -145,7 +147,7 @@ endfunction
 
 " Return 1 if a path is an absolute path.
 function! ale#path#IsAbsolute(filename) abort
-    if has('win32')
+    if s:is_windows
         return a:filename[:0] =~# '[\\/]' || a:filename[0:2] =~? '[A-Z]:[/\\]'
     else
         return a:filename[:0] is# '/'
@@ -179,7 +181,7 @@ function! ale#path#GetAbsPath(base_directory, filename) abort
         return ale#path#Simplify(a:filename)
     endif
 
-    let l:sep = has('win32') ? '\' : '/'
+    let l:sep = s:is_windows ? '\' : '/'
 
     return ale#path#Simplify(a:base_directory . l:sep . a:filename)
 endfunction
@@ -192,7 +194,7 @@ function! ale#path#Dirname(path) abort
     endif
 
     " For /foo/bar/ we need :h:h to get /foo
-    if a:path[-1:] is# '/' || (has('win32') && a:path[-1:] is# '\')
+    if a:path[-1:] is# '/' || (s:is_windows && a:path[-1:] is# '\')
         return fnamemodify(a:path, ':h:h')
     endif
 
@@ -233,8 +235,8 @@ endfunction
 
 " Given a path, return every component of the path, moving upwards.
 function! ale#path#Upwards(path) abort
-    let l:pattern = has('win32') ? '\v/+|\\+' : '\v/+'
-    let l:sep = has('win32') ? '\' : '/'
+    let l:pattern = s:is_windows ? '\v/+|\\+' : '\v/+'
+    let l:sep = s:is_windows ? '\' : '/'
     let l:parts = split(ale#path#Simplify(a:path), l:pattern)
     let l:path_list = []
 
@@ -243,7 +245,7 @@ function! ale#path#Upwards(path) abort
         let l:parts = l:parts[:-2]
     endwhile
 
-    if has('win32') && a:path =~# '^[a-zA-z]:\'
+    if s:is_windows && a:path =~# '^[a-zA-z]:\'
         " Add \ to C: for C:\, etc.
         let l:path_list[-1] .= '\'
     elseif a:path[0] is# '/'
@@ -283,7 +285,7 @@ function! ale#path#FromFileURI(uri) abort
     let l:path = ale#uri#Decode(l:encoded_path)
 
     " If the path is like /C:/foo/bar, it should be C:\foo\bar instead.
-    if has('win32') && l:path =~# '^/[a-zA-Z][:|]'
+    if s:is_windows && l:path =~# '^/[a-zA-Z][:|]'
         let l:path = substitute(l:path[1:], '/', '\\', 'g')
         let l:path = l:path[0] . ':' . l:path[2:]
     endif