Fix windows tests

Author: voltechs

.github/workflows/ci.yml

@@ -35,14 +35,11 @@ jobs:
         if: runner.os == 'Linux'
         run: sudo apt-get update && sudo apt-get install -y build-essential
 
-      - name: Install build dependencies
-        if: runner.os == 'Windows'
+      - name: Install dependencies and compile extension
         run: |
-          choco install mingw
-          refreshenv
-
-      - name: Build native extension for testing
-        run: bundle install
+          bundle install
+          # Compile C extension using rake (standard for C extension gems)
+          bundle exec rake compile
 
       - name: Run tests with coverage
         run: bundle exec rspec

Rakefile

@@ -3,4 +3,40 @@ require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 
+# Compile the C extension
+desc "Compile the native extension"
+task :compile do
+  Dir.chdir('ext/thaw') do
+    sh 'ruby extconf.rb'
+    sh 'make'
+    
+    # Copy compiled extension to lib directory for development
+    extension_files = Dir['thaw_native.{bundle,so,dll}']
+    if extension_files.empty?
+      puts "Warning: No compiled extension found"
+    else
+      extension_file = extension_files.first
+      target_dir = '../../lib/thaw'
+      FileUtils.mkdir_p(target_dir)
+      FileUtils.cp(extension_file, target_dir)
+      puts "Copied #{extension_file} to #{target_dir}/"
+    end
+  end
+end
+
+# Clean compiled files
+desc "Clean compiled extension files"
+task :clean do
+  Dir.chdir('ext/thaw') do
+    sh 'make clean' if File.exist?('Makefile')
+    FileUtils.rm_f(Dir['thaw_native.{bundle,so,dll,o}'])
+    FileUtils.rm_f('Makefile')
+  end
+  # Clean lib directory too
+  FileUtils.rm_f(Dir['lib/thaw/thaw_native.{bundle,so,dll}'])
+end
+
+# Make tests depend on compilation
+task :spec => :compile
+
 task :default => :spec

ext/thaw/extconf.rb

@@ -24,21 +24,25 @@
 when /darwin/
   # macOS specific
   $CFLAGS += " -D__APPLE__"
+  # GCC/Clang flags for macOS
+  $CFLAGS += " -Wno-error -Wno-deprecated-declarations -Wno-strict-prototypes -Wno-compound-token-split-by-macro -w"
 when /linux/
   # Linux specific
   $CFLAGS += " -D__LINUX__"
+  # GCC flags for Linux
+  $CFLAGS += " -Wno-error -Wno-deprecated-declarations -Wno-strict-prototypes -w"
 when /mingw|mswin/
   # Windows specific  
   $CFLAGS += " -D__WINDOWS__"
+  # MinGW/MSYS2 flags for Windows (setup-ruby uses MinGW)
+  if RUBY_PLATFORM =~ /mingw/
+    $CFLAGS += " -Wno-error -Wno-deprecated-declarations -Wno-strict-prototypes -w"
+  else
+    # MSVC flags (if using Visual Studio)
+    $CFLAGS += " /W0"  # Suppress all warnings for MSVC
+  end
 end
 
-# Cross-platform compiler flags - suppress warnings
-$CFLAGS += " -Wno-error"
-$CFLAGS += " -Wno-deprecated-declarations" 
-$CFLAGS += " -Wno-strict-prototypes"
-$CFLAGS += " -Wno-compound-token-split-by-macro"
-$CFLAGS += " -w" # Suppress all warnings
-
 # Force correct 64-bit sizes for platforms where needed
 if RUBY_PLATFORM =~ /darwin/
   $CFLAGS += " -DSIZEOF_LONG=8"

thaw.gemspec

@@ -22,6 +22,7 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'bundler', '>= 2.0'
   spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rake-compiler', '~> 1.0'
   spec.add_development_dependency 'rspec', '~> 3.10'
   spec.add_development_dependency 'simplecov_json_formatter', '~> 0.1'