Actions mingw - add tool/test for WEBrick in spec tests

[MSP-Greg]

Some spec tests require WEBrick, and the spec tests are run from install with the MinGW CI.

[k0kubun]

I agree this fixes the issue, but the real problem is that .github/workflows/mingw.yml is not using make test-spec, which leads this kind of individual fix. Could you check if we can replace ruby ../mspec/bin/mspec -j (since #2791) with make test-spec?

[MSP-Greg]

Yes, it might be a few hours.

Since there is little testing of the install, I thought doing so for spec tests might be helpful. Then again, when it breaks (which has been rare), the cause may not be clear...

[MSP-Greg]

I hate December. This passed in my fork, so something has changed. I'll review. Thanks.

EDIT: wrong folder. Must be losing my mind...

[MSP-Greg]

@k0kubun

Will test make test-spec shortly in my fork. Ruby-loco tests install folder (both mingw & mswin), so no need to do so here...

[k0kubun]

Ruby-loco tests install folder (both mingw & mswin), so no need to do so here...

👍

[MSP-Greg]

@k0kubun

I ran CI in my fork using make test-spec and had one failure:

An exception occurred during: loading D:/a/ruby/ruby/src/spec/ruby/optional/capi/rbasic_spec.rb ERROR
Errno::ENOENT: No such file or directory @ rb_file_s_mtime - /install/bin/x64-msvcrt-ruby300.dll
D:/a/ruby/ruby/src/spec/ruby/optional/capi/spec_helper.rb:49:in `mtime'
D:/a/ruby/ruby/src/spec/ruby/optional/capi/spec_helper.rb:49:in `compile_extension'
D:/a/ruby/ruby/src/spec/ruby/optional/capi/spec_helper.rb:129:in `load_extension'
D:/a/ruby/ruby/src/spec/ruby/optional/capi/rbasic_spec.rb:4:in `<top (required)>'

Maybe due to the folder layout used for src, build, & install. Took a break, now using the equivalent of:

ruby ../mspec/bin/mspec -j -I../../tool/lib

Both ruby-loco builds passed with that. I think the -I placement in 'Fix the path properly' is incorrect.

Again, I apologize for the folder error...

[k0kubun]

Maybe we can fix the broken spec separately. Could you file the current make test-spec (+ a guard for skipping it) change first?