So how did typo pass thru unit tests and BDD?

From: aabes [mailto:notifications@github.com]
Sent: Monday, March 11, 2013 1:38 PM
To: crowbar/barclamp-crowbar
Cc: Kanevsky, Arkady
Subject: Re: [barclamp-crowbar] fix BDD tests around node create [1/1] (#455)


In crowbar_framework/app/controllers/nodes_controller.rb:

> @@ -72,21 +72,25 @@ def show

>

>    # RESTful DELETE of the node resource

>    def destroy

> -    n = Node.find params

> -    unless n.nil?

> -      Jig.all.each {|j| j.delete_node(n)}

> -    end

> +    n = find_ndoe(params)

Specifically - in this case I ran units + bdd. The admin deploy is still churning.

—
Reply to this email directly or view it on GitHub<https://github.com/crowbar/barclamp-crowbar/pull/455/files#r3322718>.