9

オブジェクトから Net::SFTP をスタブ化しようとしています。モデルは次のとおりです。

class BatchTask
  require 'net/sftp'

  def get_file_stream(host, username, password, path_to_dir, filename)
    raise ArgumentError if host.nil? or username.nil? or password.nil? or path_to_dir.nil? or filename.nil?
    file_stream = nil
    Net::SFTP.start(host, username, password) do |sftp|
      sftp.dir.glob(path_to_dir, filename) do |entry|
        # Verify the directory contents
        raise RuntimeError(true), "file: #{path_to_dir}/#{filename} not found on SFTP server" if entry.nil?
        file_stream = sftp.file.open("#{path_to_dir}/#{entry.name}")
      end
    end
    file_stream
  end

end

仕様は次のとおりです。

require 'spec_helper'

describe "SftpToServer" do
  let(:ftp) { BatchTask::SftpToServer.new }

 it "should return a file stream" do
    @sftp_mock = mock('sftp')
    @entry = File.stubs(:reads).with("filename").returns(@file)
    @entry_mock = mock('entry')
    @entry_mock.stub(:name).with("filename").and_return("filename")
    @sftp_mock.stub_chain(:dir, :glob).and_yield(@entry_mock)
    Net::SFTP.stub(:start).and_yield(@sftp_mock)
    @sftp_mock.stub_chain(:file, :open).with("filename").and_yield(@file)

    ftp.get_file_stream("ftp.test.com", "user", "password", "some/pathname", "filename").should be_kind_of(IO)
  end

end

スタックトレースは次のとおりです。

Spec::Mocks::MockExpectationError in 'SftpToServer should return a file stream'
Mock "entry" received :name with unexpected arguments
  expected: ("filename")
       got: (no args)
/Users/app/models/batch_task/sftp_to_server.rb:12:in `get_file_stream'
/Users/app/models/batch_task/sftp_to_server.rb:9:in `get_file_stream'
/Users/app/models/batch_task/sftp_to_server.rb:8:in `get_file_stream'
./spec/models/batch_task/sftp_to_server_spec.rb:15:

まず、私のアプローチはここにありますか？SFTP は十分にテストされていると確信できるので、SFTP の機能を削除したいと考えています。代わりに、get_file_stream() 内の「ブラック ボックス」によってファイル ストリームが確実に返されるようにすることに焦点を当てたいと思います。

第二に、これを達成するために sftp.file.open() を適切にスタブするにはどうすればよいですか?

ここでアイデアをお寄せいただきありがとうございます！

4

1 に答える 1

11

まず、2 つの理由から、sftp をモックアウトすることをお勧めします。

  1. sftp テストを書いているのではなく、テストしようとしているものだけをテストすることに集中してください。
  2. テストでネットワークへの依存関係を削除します。制御できない何かが原因でテストが失敗し始めるのは望ましくありません。

エラーに関する限り、これはあなたの差し迫った問題です：

@entry_mock.stub(:name).with("filename").and_return("filename")

ここでentry.name("filename")は、単にentry.name.

次のように変更します。

@entry_mock.stub(:name).and_return("filename")

そして、あなたの乗り方を教えてください。

于 2012-07-31T03:55:48.407 に答える