yup, parentheses did it.   Thanks…

@ssh = SimpleDeploy::Stack::SSH.new @options.merge({ :instances   => [] })

From: Brett Weaver [mailto:notifications@github.com]
Sent: Tuesday, February 19, 2013 9:58 AM
To: intuit/simple_deploy
Cc: Young, Kevin
Subject: Re: [simple_deploy] To address issue 149,150 and 155 for simple deploy, Capistrano error handling (#154)


In spec/stack/ssh_spec.rb:

>      it "should return false when no running instances running" do

> -      options = { :config      => @config_mock,

> -                  :instances   => [],

> -                  :environment => 'test-env',

> -                  :ssh_user    => 'user',

> -                  :ssh_key     => 'key',

> -                  :stack       => @stack_mock,

> -                  :name        => 'test-stack' }

> -      @ssh = SimpleDeploy::Stack::SSH.new options

> +      options_override = { :instances   => [] }

> +      @ssh = SimpleDeploy::Stack::SSH.new @options.merge options_override

Ahh yes, it probably need parens enclosing it.

—
Reply to this email directly or view it on GitHub<https://github.com/intuit/simple_deploy/pull/154/files#r3066369>.