質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.04%

Ruby rspec 引数についてのエラー

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,584

sanami

score 18

コマンドプロンプトからrspecファイルを実行時に以下のエラーが出ていて困っています。

10
F.

Failures:

  1) argv 引数が数字の場合
     Failure/Error: expect(argv(['1'])).to eq(1 * 10)

       expected: 10
            got: nil

       (compared using ==)
     # ./sample_1_spec.rb:6:in `block (2 levels) in <top (required)>'

Finished in 0.00701 seconds (files took 0.53979 seconds to load)
2 examples, 1 failure

Failed examples:

rspec ./sample_1_spec.rb:5 # argv 引数が数字の場合


下記がsample_1_spec.rbファイルの内容です

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

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

  it "引数が数字以外の場合" do
    expect eq('error')
  end
end

下記がsample_1.rbの内容です

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

if FILE == $PROGRAM_NAME
  puts argv ARGV
end

よろしくお願い致します。
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+1

前の回答では、sample_1.rb メソッドの動作を変更してしまっていました。
質問文にあったように puts を内部で行うままにする場合は次のように テストを書くことができます。
sammple_1.rb
def parse_argv(argv)
  if /\A\d+\z/ =~ argv[0]
    puts argv[0].to_i * 10
  else
    puts 'error'
  end
end

if __FILE__ == $PROGRAM_NAME
  parse_argv ARGV
end
spec/sample_1_spec.rb
# equire '../spec_helper.rb'
require './sample_1.rb'

# See  http://stackoverflow.com/questions/16507067
#      > Testing STDOUT output in Rspec

describe 'argv' do
  it "引数が数字の場合" do
    expect { parse_argv(['1']) }.to output("10\n").to_stdout
  end

  it '引数が数字以外の場合' do
    expect { parse_argv(['a']) }.to output("error\n").to_stdout
  end
end

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

sample_1.rb
spec/sample_1_spec.rb の内容を以下に示します。

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

if __FILE__ == $PROGRAM_NAME
  puts parse_argv ARGV
end
spec/sample1_spec.rb
# equire '../spec_helper.rb'
require './sample_1.rb'

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

  it '引数が数字以外の場合' do
    expect(parse_argv(['a'])).to eq('error')
  end
end
実行例は次の様になります。
$ ruby sample_1.rb 1
10
$ ruby sample_1.rb a
error
$ rspec
..

Finished in 0.00137 seconds (files took 0.19976 seconds to load)
2 examples, 0 failures

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/03/22 23:14

    katoyさんいつもご回答ありがとうございます。

    数字以外の場合に文字のaを引き合いにする意味は分かったのですが
    数字以外の場合(文字または引数が入力されていない場合)をあらわすにはどのようにすればよろしいでしょうか?

    可能であればit '引数が数字以外の場合' do 内に収められればと思います。
    よろしくお願い致します。

    キャンセル

  • 2015/03/22 23:33

    自分で下記のように追加してエラーが出なかったのですが問題ないでしょうか?

    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 eq('error')
    expect(argv([])).to eq('error')
    end

    end

    よろしくお願い致します。

    キャンセル

  • 2015/03/22 23:40

    次のようにテストを追加するとよいとおもいます。
    it "引数が 0 の場合" do
    expect(parse_argv(['0'])).to eq(1 * 10)
    end

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

    it '引数の数が0の場合' do
    expect(parse_argv([])).to eq('error')
    end

    it '引数が nil の場合' do
    expect(parse_argv(nil)).to eq('error')
    end

    rspec を実行すると次のエラーが出ます。これは sample_1.rb を修正するとエラーが起こらなくなるようにできるはずです。(あるいは テスト側を変える)
    rspec ./spec/sample_1_spec.rb:13 # argv 引数 0 の場合
    rspec ./spec/sample_1_spec.rb:17 # argv 引数 負の数字の場合
    rspec ./spec/sample_1_spec.rb:25 # argv 引数が nil の場合


    キャンセル

  • 2015/03/22 23:46

    もし余裕があるなら、
    https://teratail.com/questions/7725
    の space を TAB に置き換えるメソッドを ruby で書いて、
    それに対するテストを rspec で書いてみませんか?
    (entab メソッドは、 ruby なら もっと短く書けるはずです。)

    キャンセル

0

argvはARGVを評価して第0要素が数値になる文字列であればto_iして10倍した数値を標準出力に印字し、さもなくば文字列「error」を標準出力に印字する関数ですね。
つまり標準出力に印字することがargvの仕事であり、戻り値はputsの戻り値nilとなります。

expect(argv(['1']))はexpect(nil)になりますね。わかりますか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る