Rspec StringIOを利用したYAML.dumpにてエラーが発生する

Question

事象

あるhashの書き出しをファイルに保存する処理のテストをしていて、StringIOを使ってテストしようとしたところ、
下記エラーメッセージがでました。

  1) Hoge ファイルの作成 use StringIO
     Failure/Error: Hoge.test(def_path)
       #<File (class)> asked to yield |[#<StringIO:0x000055e05af5b758>]| but no block was passed

エラーメッセージから察するにブロック(file)が渡されていない様にみえますが、
RSpecでFileIOをスタブにする
↑ここのお手本を参考にできているつもりでありました。

YAMLの出力テストする場合、どこを改良したら良いかご教示お願いします。

テスト結果

Hoge
  ファイルの作成
    use StringIO (FAILED - 1)

Failures:

  1) Hoge ファイルの作成 use StringIO
     Failure/Error: Hoge.test(def_path)
       #<File (class)> asked to yield |[#<StringIO:0x0000559cfcbfe660>]| but no block was passed
     # ./lib_sc.rb:7:in `test'
     # ./spec/lib_sc_spec.rb:13:in `block (3 levels) in <top (required)>'

Finished in 0.01155 seconds (files took 0.30146 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/lib_sc_spec.rb:10 # Hoge ファイルの作成 use StringIO

ソース

lib_sc.rb

require 'yaml'
require 'json'

class Hoge 
  def self.test(conf_path)
    data = { "fruits" => ["Orange", "Apple", "Grape"] }
    YAML.dump( data, File.open(conf_path, 'w') )
  end
end

spec/lib_sc_spec.rb

require 'stringio'
require 'tempfile'
require 'fileutils'
require_relative '../lib_sc.rb'

RSpec.describe Hoge do
  let(:def_path){ File.join(__dir__, 'Assets/def_file.yml') }

  describe 'ファイルの作成' do
    it 'use StringIO' do
      file = StringIO.new('','w')
      allow(File).to receive(:open).and_yield(file)
      Hoge.test(def_path)
    end
  end
end

環境

$ uname -a
Linux ansible-web 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

$ ruby -v
ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]

$ rspec -v
RSpec 3.9
  - rspec-core 3.9.2
  - rspec-expectations 3.9.2
  - rspec-mocks 3.9.1
  - rspec-support 3.9.3

以上、よろしくお願いします。

Answer 1

File.open(conf_path, 'w'){|f| YAML.dump(data) }

のところが間違っているかと。以下のようにすべきなのでは？

File.open(conf_path, 'w'){|f| f << YAML.dump(data) }

参考: https://docs.ruby-lang.org/ja/latest/class/Psych.html#S_DUMP

Comments

[kurokoSin]

補足いただいてありがとうございました。
こちらのコードでも動作することが確認できました！

Answer 2

allow(File).to receive(:open).and_yield(file)

and_return(file)かと思います。

もしくは

  def self.test(conf_path)
    data = { "fruits" => ["Orange", "Apple", "Grape"] }
    File.open(conf_path, 'w'){|file| YAML.dump( data, file ) }
  end

Comments

[kurokoSin]

素早い回答大変ありがとうございます。
両方ためしてみたところ、前者は想定通りの動きをしてくれました。

ですが、後者のソースを修正する方は例外は発生しなくなったのですが、
テストで中身を書き出すと、""（文字列）になってしまいました。
実際にファイルを書き出しても中身が出力されなくなってしまいました。
後者のソースは何がいけないのでしょうか。

テストコード(spec/lib_sc_spec.rb 抜粋)
-----
describe 'ファイルの作成' do
it 'dont use StringIO' do
Hoge.test(def_path)
end

it 'use StringIO' do
file = StringIO.new('','w')
allow(File).to receive(:open).and_yield(file)
# allow(File).to receive(:open).and_return(file)
Hoge.test(def_path)
p file.string
end
end

テストコード(lib_sc.rb 抜粋)
-----
def self.test(conf_path)
data = { "fruits" => ["Orange", "Apple", "Grape"] }
# YAML.dump( data, File.open(conf_path, 'w') )
File.open(conf_path, 'w'){|f| YAML.dump( data ) }
end

[kurokoSin]

一応望む結果は得られたので、クローズにすべきか迷いましたが、こちらで質問させていただきました。
別途質問を立てるべきであればお申しつけください。

[asm]

> File.open(conf_path, 'w'){|f| YAML.dump( data ) }
File.open(conf_path, 'w'){|f| YAML.dump( data, f ) }
です。

[kurokoSin]

ありがとうございました。
無事　全て望んだ結果になりました！