Test-Commit-Revert：Rubyでレガシーコードをテストするための便利なワークフロー

この手法の主な目標は、テストケースを追加するたびにコードを少しよく理解することです。これにより、テストカバレッジが自然に増加し、そうでなければ不可能だった多くのリファクタリングのブロックが解除されます。

TCRの利点の1つは、多くのシナリオで役立つことです。テストがまったくないコードでも、部分的にテストされたコードでも使用できます。テストに合格しなかった場合は、変更を元に戻して再試行します。

どのように使用できますか？

Kent Beckは、さまざまな記事やビデオ（最後にリンクされています）で、プロジェクト内の特定のファイルが保存された後に実行されるスクリプトを使用するのが良いアプローチであることを示しています。

これは、テストしようとしているプロジェクトに大きく依存します。エディターでプラグインを使用してファイルを保存するたびに実行される次のスクリプトのようなものは、良いスタートです：

(rspec && git commit -am "WIP") || git reset --hard

Visual Studio Codeを使用している場合、保存のたびに実行するのに適したプラグインは「runonsave」です。上記のコマンドまたは同様のコマンドをプロジェクトに含めることができます。この場合、構成ファイル全体は

{
  "folders": [{ "path": "." }],
  "settings": {
    "emeraldwalk.runonsave": {
      "commands": [
        {
          "match": "*.rb",
          "cmd": "cd ${workspaceRoot} && rspec && git commit -am WIP || git reset --hard"
        }
      ]
    }
  }
}

後で、コマンドラインで直接、またはGithubを使用している場合はPRをマージするときに、Gitを使用してコミットを潰すことができることを覚えておいてください。

。

これは、作業中のブランチで行ったすべてのコミットに対して、メインブランチで1つのコミットのみを取得することを意味します。 Githubのこの図は、それをよく説明しています：

。

TCRを使用した最初のテストの作成

簡単な例を使用して、テクニックを説明します。動作していることがわかっているクラスがありますが、それを変更する必要があります。

変更を加えて展開するだけです。ただし、プロセスで何も壊さないようにする必要があります。これは常に良い考えです。

# worker.rb
class Worker
  def initialize(age, active_years, veteran)
    @age = age
    @active_years = active_years
    @veteran = veteran
  end

  def can_retire?
    return true if @age >= 67
    return true if @active_years >= 30
    return true if @age >= 60 && @active_years >= 25
    return true if @veteran && @active_years > 25

    false
  end
end

最初のステップは、テスト用の新しいファイルを作成することです。これにより、そこにテストを追加できるようになります。 can_retire?の最初の行を見ました

  def can_retire?
    return true if @age >= 67
    ...
    ...
  end

したがって、最初にこのケースをテストできます：

# specs/worker_spec.rb
require_relative './../worker'

describe Worker do
  describe 'can_retire?' do
    it "should return true if age is higher than 67" do

    end
  end
end

簡単なヒントを次に示します。TCRを使用している場合、保存するたびに、テストに合格しないと最新の変更が表示されなくなります。したがって、アサーションを含む1つまたは複数の行を実際に記述して保存する前に、テストを「セットアップ」するためのできるだけ多くのコードが必要です。

上記のファイルをそのように保存すると、テスト用の行を追加できます。

require_relative './../worker'

describe Worker do
  describe 'can_retire?' do
    it "should return true if age is higher than 67" do
      expect(Worker.new(70, 10, false).can_retire?).to be_true ## This line can disappear when we save now
    end
  end
end

保存するときに、新しい行が消えない場合は、良い仕事をしています。テストに合格しました！

# frozen_string_literal: true

require_relative './../worker'

describe Worker do
  describe 'can_retire?' do
    it 'should return true if age is higher than 67' do
      expect(Worker.new(70, 10, false).can_retire?).to be true
    end

    it 'should return true if age is 67' do
      expect(Worker.new(67, 10, false).can_retire?).to be true
    end

    it 'should return true if age is less than 67' do
      expect(Worker.new(50, 10, false).can_retire?).to be false
    end

    it 'should return true if active years is higher than 30' do
      expect(Worker.new(60, 31, false).can_retire?).to be true
    end

    it 'should return true if active years is 30' do
      expect(Worker.new(60, 30, false).can_retire?).to be true
    end
  end
end

# frozen_string_literal: true

require_relative './../worker'

describe Worker do
  describe 'can_retire?' do
    it 'should return true if age is higher than 67' do
      expect(Worker.new(70, 10, false).can_retire?).to be true
    end

    it 'should return true if age is 67' do
      expect(Worker.new(67, 10, false).can_retire?).to be true
    end

    it 'should return true if age is less than 67' do
      expect(Worker.new(50, 10, false).can_retire?).to be false
    end

    it 'should return true if active years is higher than 30' do
      expect(Worker.new(60, 31, false).can_retire?).to be true
    end

    it 'should return true if active years is 30' do
      expect(Worker.new(20, 30, false).can_retire?).to be true
    end

    it 'should return true if age is higher than 60 and active years is higher than 25' do
      expect(Worker.new(60, 30, false).can_retire?).to be true
    end

    it 'should return true if age is higher than 60 and active years is higher than 25' do
      expect(Worker.new(61, 30, false).can_retire?).to be true
    end

    it 'should return true if age is 60 and active years is higher than 25' do
      expect(Worker.new(60, 30, false).can_retire?).to be true
    end

    it 'should return true if age is higher than 60 and active years is 25' do
      expect(Worker.new(61, 25, false).can_retire?).to be true
    end

    it 'should return true if age is 60 and active years is 25' do
      expect(Worker.new(60, 25, false).can_retire?).to be true
    end

    it 'should return true if is veteran and active years is higher than 25' do
      expect(Worker.new(60, 25, false).can_retire?).to be true
    end
  end
end