[CI] Enable matrix CI runs different Ruby versions and OSs (#61)

* Enable matrix CI runs for ruby 2.7, 3.0 and 3.1 on ubuntu, macos and windows

* Remove windows-latest for now

* Add GH token to protoc step to avoid getting rate limited

* Try disabling cache

* Allow all to run

* Bring back Rust cache

* Wait for the MockServer to start in Connection integration specs

* Fix signal spec for Ruby 2.7

* Use correct path for Rust cache

* Use OS and Ruby version in Rust cache key to avoid overlap between jobs

* Add comment describing empty hash args

Author: antstorm

.github/workflows/ci.yml

@@ -10,26 +10,41 @@ on: # rebuild any PRs and main branch changes
 jobs:
   # Lint and test the project
   build-lint-test:
-    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby:
+          - 2.7.6
+          - 3.0.4
+          - 3.1.2
+        os:
+          - ubuntu-latest
+          - macos-latest
+    runs-on: ${{ matrix.os }}
     steps:
+      - name: Print build information
+        run: 'echo head_ref: ${{ github.head_ref }}, ref: ${{ github.ref }}, os: ${{ matrix.os }}, ruby: ${{ matrix.ruby }}'
       - uses: actions/checkout@v2
         with:
           submodules: recursive
       - uses: actions-rs/toolchain@v1
         with:
           toolchain: stable
       - uses: arduino/setup-protoc@v1
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
       - uses: ruby/setup-ruby@v1
         with:
-          ruby-version: 3.0
+          ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
       # Needed to compile go test server and worker
       - uses: actions/setup-go@v2
         with:
           go-version: "1.19"
-      - uses: Swatinem/rust-cache@v1
+      - uses: Swatinem/rust-cache@v2
         with:
-          working-directory: bridge
+          workspaces: bridge
+          key: ${{ matrix.os }}-${{ matrix.ruby }}
       - name: Setup Golang test server caches
         uses: actions/cache@v3
         with:

spec/integration/client_spec.rb

@@ -204,7 +204,8 @@ def wait_for_server(url)
       input = { action_signal: 'test-signal' }
       handle = subject.start_workflow(workflow, input, id: id, task_queue: task_queue)
 
-      handle.signal('test-signal', { result: { value: 'test signal arg' } })
+      # Empty hash as the last arg is required in Ruby 2.7 to distinguish hash argument from kwargs
+      handle.signal('test-signal', { result: { value: 'test signal arg' } }, **{})
 
       expect(handle.result).to eq('test signal arg')
     end

spec/integration/connection_spec.rb

@@ -5,13 +5,27 @@
 describe Temporal::Connection do
   mock_address = '0.0.0.0:4444'.freeze
 
+  def wait_for_mock_server(address, attempts, interval = 1)
+    request = Temporal::Api::WorkflowService::V1::GetSystemInfoRequest.new
+    attempts.times do |i|
+      connection = described_class.new("http://#{address}")
+      connection.get_system_info(request)
+      break
+    rescue StandardError => e
+      puts "Error connecting to a mock server: #{e}. Attempt #{i + 1} / #{attempts}"
+      raise if i + 1 == attempts # re-raise upon exhausting attempts
+
+      sleep interval
+    end
+  end
+
   subject { described_class.new("http://#{mock_address}") }
 
   # TODO: For some reason the Bridge doesn't play well with the server in the same
   #       process throwing SegFaults in cases. Needs further investigation
   before(:all) do
     @pid = fork { MockServer.run(mock_address) }
-    sleep 0.1 # wait for the server to boot up
+    wait_for_mock_server(mock_address, 10)
   end
   after(:all) { Process.kill('QUIT', @pid) }