Clean up IronicNodeStates.update_from_compute_node

This method was needlessly wordy.

This patch does not change any functionality, merely makes the method
easier to read.

Change-Id: I7d66ea5e2146877f70a2e9060d3d4244f9493b9d
Improved coverage for ironic API

Added missing tests for get_one() in ironic.tests.api.*

Change-Id: I3d65441f4bf3857f40e856f692858e9b3ac19568
Merge "Improved coverage for ironic API"
Merge "Clean up IronicNodeStates.update_from_compute_node"
Handling validation in conductor consistently

Change where we do validation in change_node_power_state to follow
where we do it for do_node_deploy and vendor_passthrough. That
is, we perform validation synchronously before we spawn off a
thread to perform the longer running action.

Change-Id: Ic1f791bb5ef49c033e4baff8d3f6900091dbf332
Closes-bug: #1294676