fix smoke_test

Author: JalinWang

.github/workflows/build_wheel_on_windows.yml

@@ -56,6 +56,8 @@ jobs:
   build_wheels_windows_2025:
     name: Build wheels on windows-2025 (x64)
     runs-on: windows-g9i
+    outputs:
+      zvec_version: ${{ steps.log_vars.outputs.zvec_version }}
 
     steps:
       - name: Show env info
@@ -84,54 +86,55 @@ jobs:
           pip install --upgrade pip
           pip install cibuildwheel==3.4.0
 
-      - name: Add Git bin to PATH
-        run: echo "C:\Program Files\Git\usr\bin" >> $GITHUB_PATH
-        shell: powershell
+      #      - name: Add Git bin to PATH
+      #        run: echo "C:\Program Files\Git\usr\bin" >> $GITHUB_PATH
+      #        shell: powershell
 
-      # Cache the directory where cibuildwheel outputs the .whl files
       - name: Cache built wheels
         id: cache-wheels
         uses: actions/cache@v4
         with:
           path: wheelhouse
-          # IMPORTANT: Update these file extensions/paths to match the source files
-          # that actually affect your compiled wheel (e.g., .c, .cpp, .rs, .py)
           key: ${{ runner.os }}-wheels-${{ hashFiles('**/pyproject.toml', '**/setup.py', '**/*.py', '**/*.c', '**/*.cpp') }}
 
       - name: Build wheels using cibuildwheel
         if: steps.cache-wheels.outputs.cache-hit != 'true'
         shell: powershell
         run: |
           python -m cibuildwheel --output-dir wheelhouse
-          
-          #      - name: Install and test wheel
-          #        shell: powershell
-          #        run: |
-          #          # cp11
-          #          $wheels = Get-ChildItem wheelhouse/*.whl | Select-Object -First 2
-          #          $wheelPath = $wheels.FullName
-          #
-          #          python -m venv test_env
-          #          $venvPython = "test_env/Scripts/python.exe"
-          #          & $venvPython -m pip install --upgrade pip
-          #          & $venvPython -m pip install $wheelPath
-          #          & $venvPython -c "import zvec; print('Import OK:', zvec.__version__)"
-          
+
+      - name: Install and test wheel
+        shell: powershell
+        run: |
+          $wheels = (Get-ChildItem wheelhouse/zvec-*.whl)[1] # cp311
+          $wheelPath = $wheels.FullName
+
+          python -m venv test_env
+          $venvPython = "test_env/Scripts/python.exe"
+          & $venvPython -m pip install --upgrade pip
+          & $venvPython -m pip install $wheelPath
+          & $venvPython -c "import zvec; print('Import OK:', zvec.__version__)"
+
       - name: Log wheel variables
+        id: log_vars
         shell: powershell
         run: |
           $wheels = Get-ChildItem wheelhouse/*.whl | Select-Object -ExpandProperty FullName
           
-          if ($env:GITHUB_STEP_SUMMARY) {
-            $wheels | Out-File -FilePath $env:GITHUB_STEP_SUMMARY -Append
-          }
+          # Extract zvec version from wheel filename
+          $wheel = Get-ChildItem wheelhouse/zvec-*.whl | Select-Object -First 1
+          $filename = $wheel.Name
+          $version = [regex]::Match($filename, '^zvec-([^-]+)-').Groups[1].Value
           
-          "wheels=$($wheels -join ' ')" | Out-File -FilePath $env:GITHUB_ENV -Append
+          Write-Host "Detected zvec version: $version"
           
-          $wheels = Get-ChildItem wheelhouse/*.whl | Select-Object -ExpandProperty FullName
+          # Output version for job outputs
+          "zvec_version=$version" | Out-File -FilePath $env:GITHUB_OUTPUT -Append
           
           if ($env:GITHUB_STEP_SUMMARY) {
+            "Wheel files:" | Out-File -FilePath $env:GITHUB_STEP_SUMMARY -Append
             $wheels | Out-File -FilePath $env:GITHUB_STEP_SUMMARY -Append
+            "zvec version: $version" | Out-File -FilePath $env:GITHUB_STEP_SUMMARY -Append
           }
           
           "wheels=$($wheels -join ' ')" | Out-File -FilePath $env:GITHUB_ENV -Append
@@ -163,23 +166,34 @@ jobs:
           twine upload --skip-existing --verbose wheelhouse/*.whl
 
   smoke_test:
-    needs: [ publish_wheels, compute_matrix ]
+    needs: [ publish_wheels, compute_matrix, build_wheels_windows_2025 ]
     strategy:
       fail-fast: false
       max-parallel: 4
       matrix: ${{ fromJson(needs.compute_matrix.outputs.matrix) }}
     runs-on: ${{ matrix.platform }}
 
-
     steps:
+      #      - name: Download built wheels
+      #        uses: actions/download-artifact@v4
+      #        with:
+      #          name: windows-wheels
+      #          path: wheelhouse/
+
       - name: Smoke test from PyPI
         shell: powershell
         env:
           PUBLISH_TARGET: ${{ inputs.publish_target || 'testpypi' }}
+          ZVEC_VERSION: ${{ needs.build_wheels_windows_2025.outputs.zvec_version }}
         run: |
-          $wheel = (Get-ChildItem wheelhouse/zvec-*.whl)[1] # cp311
-          $filename = $wheel.Name
-          $version = [regex]::Match($filename, '^zvec-([^-]+)-').Groups[1].Value
+          # Check if version is empty
+          if ([string]::IsNullOrWhiteSpace($env:ZVEC_VERSION)) {
+            Write-Host "ERROR: ZVEC_VERSION is empty or not set. Cannot proceed with smoke test."
+            exit 1
+          }
+          
+          $version = $env:ZVEC_VERSION
+          Write-Host "Using zvec version from build job: $version"
 
           if ($env:PUBLISH_TARGET -eq 'testpypi') {
             $indexArgs = @("--index-url", "https://test.pypi.org/simple/", "--extra-index-url", "https://pypi.org/simple/")
@@ -212,4 +226,4 @@ jobs:
           $venvPython = "test_env/Scripts/python.exe"
           & $venvPython -m pip install --upgrade pip
           & $venvPython -m pip install @indexArgs "zvec==$version"
-          & $venvPython -c "import zvec; print('Import OK:', zvec.__version__)"
+          & $venvPython -c "import zvec; print('Import OK:', zvec.__version__)"