ruby rspec 例外テスト

Question

下記の内容でエラーが出て困っています。

Failures:

1. argv 引数が数字以外の場合
   Failure/Error: expect (argv(['a'])) .to raise_error
   NoMethodError:
   undefined method `to' for "error":String

   ./sample_1_spec.rb:10:in `block (2 levels) in <top (required)>'

Finished in 0.07311 seconds (files took 2.84 seconds to load)
2 examples, 1 failure

Failed examples:

rspec ./sample_1_spec.rb:9 # argv 引数が数字以外の場合

ファイル内容

ruby↓↓

def argv(argv)
if /\A\d+\z/ =~ argv[0]
argv[0].to_i * 10
else
'error'
end
end

if FILE == $PROGRAM_NAME
puts argv ARGV
end

rspec↓↓

require '../spec_helper.rb'
require './sample_1.rb'

describe 'argv' do
it "引数が数字の場合" do
expect(argv(['1'])).to eq(1 * 10)
end

it "引数が数字以外の場合" do
expect (argv(['a'])) .to raise_error
expect (argv([])) .to raise_error
end

end

どのように対処すればよろしいでしょうか？

Answer 1

次のようにすると良いとおもいます。

ruby:

def argv(argv)
  if /\A\d+\z/ =~ argv[0]
    puts argv[0].to_i * 10
  else
    # puts 'error'
    fail 'error'
  end
end

rspec:

it '引数が数字以外の場合' do
  # expect (argv(['a'])) .to raise_error 
  # expect (argv([])) .to raise_error
  expect { argv(['a']) }.to raise_error('error')
  expect { argv([]) }.to raise_error('error')
end

argv() メソッドでは "error" の文字列を返すのでなく、例外を投げるようにします。
spec コードでは raise_error とすると、例外が投げられたことのチェックになります。
引数に 'error' を指定することで、例外のメッセージ内容もチェックするようにしてみました。

Comments

[sanami]

返事が遅くなってしまい申し訳ありません。
ご丁寧にありがとうございます。

無事コンパイルすることが出来ました。
ちなみにですが、例外を出す方法としてrspecファイルのみを
修正してテスト行うことは可能でしょうか？

[katoy]

rspec 側は、あくまでもテストされる側のメソッドの挙動をテストするだけです。
テストされる側が例外を投げていなければ、repec 側で例外キャッチするテストは 常にNG になるだけです。