Merge pull request #87 from ruby/enable-windows-test

Enabled windows-latest on GHA
ruby · Nov 29, 2024 · fe2635b · fe2635b
2 parents 60df08b + 38f0e36
commit fe2635b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -15,7 +15,12 @@ jobs:
     strategy:
       matrix:
         ruby: ${{ fromJson(needs.ruby-versions.outputs.versions) }}
-        os: [ ubuntu-latest, macos-latest ]
+        os: [ ubuntu-latest, macos-latest, windows-latest ]
+        exclude:
+          - ruby: 2.6
+            os: windows-latest
+          - ruby: 2.7
+            os: windows-latest
     runs-on: ${{ matrix.os }}
     steps:
     - uses: actions/checkout@v4

diff --git a/test/net/smtp/test_smtp.rb b/test/net/smtp/test_smtp.rb
@@ -789,7 +789,7 @@ def loop
           @sock.puts "502 5.5.2 Error: command not recognized\r\n"
         end
       end
-    rescue Errno::ECONNRESET
+    rescue Errno::ECONNRESET, Errno::ECONNABORTED
       nil
     end
   end
-Original file line number
+Diff line change
@@ Expand Up / @@ -789,7 +789,7 @@ def loop @@
               @sock.puts "502 5.5.2 Error: command not recognized\r\n"
             end
           end
-        rescue Errno::ECONNRESET
+        rescue Errno::ECONNRESET, Errno::ECONNABORTED
           nil
         end
       end
@@ Expand Down @@