Merge pull request #64 from mishina2228/migrate-to-gh-actions

Migrate CI from Travis to GitHub Actions

Author: mfazekas

.github/workflows/ci.yml

@@ -0,0 +1,23 @@
+name: CI
+on:
+  pull_request:
+  push: { branches: master }
+permissions:
+  contents: read
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby-version: ['2.3', '2.4', '2.5', '2.6', '2.7', '3.0', '3.1', '3.2', 'ruby-head']
+    continue-on-error: ${{ matrix.ruby-version == 'ruby-head' }}
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Ruby ${{ matrix.ruby-version }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+      - name: Run Tests
+        run: bundle exec rake test

.travis.yml

@@ -4,8 +4,8 @@ source 'https://rubygems.org'
 gemspec
 
 # TODO: add to gemspec
-gem "bundler", "~> 1.11"
-gem "rake", "~> 12.0"
+gem "bundler", ">= 1.11"
+gem "rake", ">= 12.0"
 
 gem 'byebug', group: %i[development test] if !Gem.win_platform? && RUBY_ENGINE == "ruby"
 

Gemfile

@@ -76,6 +76,6 @@ RDoc::Task.new do |rdoc|
   rdoc.rdoc_files.include("bin/*.rb")
   rdoc.rdoc_files.include("lib/**/*.rb")
   extra_files.each { |file|
-    rdoc.rdoc_files.include(file) if File.exists?(file)
+    rdoc.rdoc_files.include(file) if File.exist?(file)
   }
 end

Rakefile

@@ -1,5 +1,5 @@
 module Net
-  module SCP
+  class SCP
     # A class for describing the current version of a library. The version
     # consists of three parts: the +major+ number, the +minor+ number, and the
     # +tiny+ (or +patch+) number.

lib/net/scp/version.rb

@@ -1,5 +1,5 @@
 require 'test/unit'
-require 'mocha/setup'
+require 'mocha/test_unit'
 
 begin
   gem 'net-ssh', ">= 2.0.0"

test/common.rb

@@ -4,26 +4,26 @@ class TestSCP < Net::SCP::TestCase
   def test_start_without_block_should_return_scp_instance
     ssh = stub('session', :logger => nil)
     Net::SSH.expects(:start).
-      with("remote.host", "username", :password => "foo").
+      with("remote.host", "username", { :password => "foo" }).
       returns(ssh)
 
     ssh.expects(:close).never
-    scp = Net::SCP.start("remote.host", "username", :password => "foo")
+    scp = Net::SCP.start("remote.host", "username", { :password => "foo" })
     assert_instance_of Net::SCP, scp
     assert_equal ssh, scp.session
   end
 
   def test_start_with_block_should_yield_scp_and_close_ssh_session
     ssh = stub('session', :logger => nil)
     Net::SSH.expects(:start).
-      with("remote.host", "username", :password => "foo").
+      with("remote.host", "username", { :password => "foo" }).
       returns(ssh)
 
     ssh.expects(:loop)
     ssh.expects(:close)
 
     yielded = false
-    Net::SCP.start("remote.host", "username", :password => "foo") do |scp|
+    Net::SCP.start("remote.host", "username", { :password => "foo" }) do |scp|
       yielded = true
       assert_instance_of Net::SCP, scp
       assert_equal ssh, scp.session
@@ -34,26 +34,26 @@ def test_start_with_block_should_yield_scp_and_close_ssh_session
 
   def test_self_upload_should_instatiate_scp_and_invoke_synchronous_upload
     scp = stub('scp')
-    scp.expects(:upload!).with("/path/to/local", "/path/to/remote", :recursive => true)
+    scp.expects(:upload!).with("/path/to/local", "/path/to/remote", { :recursive => true })
 
     Net::SCP.expects(:start).
-      with("remote.host", "username", :password => "foo").
+      with("remote.host", "username", { :password => "foo" }).
       yields(scp)
 
     Net::SCP.upload!("remote.host", "username", "/path/to/local", "/path/to/remote",
-      :ssh => { :password => "foo" }, :recursive => true)
+      { :ssh => { :password => "foo" }, :recursive => true })
   end
 
   def test_self_download_should_instatiate_scp_and_invoke_synchronous_download
     scp = stub('scp')
-    scp.expects(:download!).with("/path/to/remote", "/path/to/local", :recursive => true).returns(:result)
+    scp.expects(:download!).with("/path/to/remote", "/path/to/local", { :recursive => true }).returns(:result)
 
     Net::SCP.expects(:start).
-      with("remote.host", "username", :password => "foo").
+      with("remote.host", "username", { :password => "foo" }).
       yields(scp)
 
     result = Net::SCP.download!("remote.host", "username", "/path/to/remote", "/path/to/local",
-      :ssh => { :password => "foo" }, :recursive => true)
+      { :ssh => { :password => "foo" }, :recursive => true })
 
     assert_equal :result, result
   end