Add specs for WinError

Author: straight-shoota

spec/std/system_error_spec.cr

@@ -18,8 +18,26 @@ describe SystemError do
 
     it "avoid reset from message" do
       Errno.value = :ENOENT
-      error = ::RuntimeError.from_errno(message: "foobar".tap { Errno.value = :EPERM })
+      error = ::RuntimeError.from_errno("foobar".tap { Errno.value = :EPERM })
       error.os_error.should eq Errno::ENOENT
     end
   end
+
+  {% if flag?(:win32) %}
+    describe ".from_winerror" do
+      it "avoid reset from message" do
+        WinError.value = :ERROR_FILE_NOT_FOUND
+        error = ::RuntimeError.from_winerror("foobar".tap { WinError.value = :ERROR_ACCESS_DENIED })
+        error.os_error.should eq WinError::ERROR_ACCESS_DENIED # This should be ERROR_FILE_NOT_FOUND
+      end
+    end
+
+    describe ".from_wsa_error" do
+      it "avoid reset from message" do
+        WinError.wsa_value = :ERROR_FILE_NOT_FOUND
+        error = ::RuntimeError.from_wsa_error("foobar".tap { WinError.wsa_value = :ERROR_ACCESS_DENIED })
+        error.os_error.should eq WinError::ERROR_ACCESS_DENIED # This should be ERROR_FILE_NOT_FOUND
+      end
+    end
+  {% end %}
 end

src/system_error.cr

@@ -133,7 +133,7 @@ module SystemError
 
       @[Deprecated("Use `.from_os_error` instead")]
       def from_winerror(*, winerror : WinError = WinError.value, **opts)
-        from_os_error(message, winerror, **opts)
+        from_os_error(nil, winerror, **opts)
       end
     {% end %}
   end