rspec 3.0 stubbing синтаксис - не уверен, что правильно

Question

Я пытаюсь справиться с новым синтаксисом allow (для stubbing) в rspec 3.0 и действительно оценил бы, кто-то смотрит на мой код и говорит мне, правильно это (или неправильно).

download.rb

class Download 
    def download_file 
    # code to download a file 
    end 

    def valid_json 
    # code to verify downloaded file is file type json 
    end 
end 

download_spec.rb

require 'spec_helper' 
require 'json_spec' 

describe Download do 
    let(:download) {Download.new} 

    describe "#download_file" do 
    it 'downloads a file from remote file location ' do 
     allow(download).to receive(:exist).and_return(true) 
     download.download_file 
     expect(download).to have_received(:exist).and_return(true)  
    end 
    end 

    describe "#valid_json" do 
    it 'confirms downloaded file is JSON' do 
     # download.to_json.should have_json_path("id") 
     # download.to_json.should have_json_type(Integer).at_path("id") 
     expect(download.to_json).to have_json_path("id") 
     expect(download.to_json).to have_json_type(Integer).at_path("id") 
    end 
    end 
end 

Большое спасибо заранее!

Answer 1

Я считаю, что вы используете правильный синтаксис для stubbing в RSpec 3.0, но я сомневаюсь в вашем мотиве для его использования в этом случае. Я настоятельно рекомендую вам ознакомиться с презентациями Sandi Metz по тестированию Ruby, которые, мы надеемся, помогут прояснить соответствующий случай, чтобы фактически использовать заглушку. Вот слайд-колода Sandi's Magic Tricks of Testing.

Основываясь на названии вашего теста, кажется, что то, что вы действительно должны тестировать, заключается в том, создан ли файл-файл в результате этого метода. Это единственное, что вам нужно заботиться - прямой публичный эффект отправки сообщения #download_file объекту download. Что-либо кроме этого означает, что вы тестируете , а не интерфейс .

describe Download do 
    let(:download) { Download.new } 

    describe "#file" do 
    context "before file has been downloaded" do 
     it "returns nil" do 
     expect(download.file).to be_nil 
     end 
    end 
    end 

    describe "#download_file" do 
    it 'downloads a file from remote file location ' do 
     # incoming command message to object under test 
     download.download_file 
     # assert direct public side effect 
     expect(download.file).to be_a File 
    end 
    end 
end 

EDIT: Одна вещь, которую я забыл упомянуть о том, что вам может понадобиться уточнить или ваш вопрос здесь или ваше представление о ваших испытаний, являются ли они предназначены в качестве модульных тестов или тестов интеграции. Это может изменить ответ.