From ef4f9882cd514a54dbe129d7f455a769b795baf2 Mon Sep 17 00:00:00 2001 From: Josh Holtrop Date: Fri, 12 May 2017 21:08:14 -0400 Subject: [PATCH] change Builder#run base class method to new signature --- lib/rscons/builder.rb | 2 +- spec/rscons/builder_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/rscons/builder.rb b/lib/rscons/builder.rb index 9bf36ef..ed8e4b6 100644 --- a/lib/rscons/builder.rb +++ b/lib/rscons/builder.rb @@ -128,7 +128,7 @@ module Rscons # # @return [String,false] # Name of the target file on success or false on failure. - def run(target, sources, cache, env, vars) + def run(options) raise "This method must be overridden in a subclass" end diff --git a/spec/rscons/builder_spec.rb b/spec/rscons/builder_spec.rb index 7515956..8e4ebc2 100644 --- a/spec/rscons/builder_spec.rb +++ b/spec/rscons/builder_spec.rb @@ -2,7 +2,7 @@ module Rscons describe Builder do describe "#run" do it "raises an error if called directly and not through a subclass" do - expect{subject.run(:target, :sources, :cache, :env, :vars)}.to raise_error /This method must be overridden in a subclass/ + expect{subject.run({})}.to raise_error /This method must be overridden in a subclass/ end end end