From: David Lutterkort <lut...@redhat.com> --- tests/cimi/part2_test.rb | 7 +++---- tests/cimi/part3_test.rb | 8 ++++---- tests/cimi/part4_test.rb | 20 ++++++++++---------- tests/cimi/part5_test.rb | 4 ++-- tests/cimi/test_helper.rb | 4 ++-- 5 files changed, 21 insertions(+), 22 deletions(-)
diff --git a/tests/cimi/part2_test.rb b/tests/cimi/part2_test.rb index c273614..cc6041d 100644 --- a/tests/cimi/part2_test.rb +++ b/tests/cimi/part2_test.rb @@ -56,8 +56,8 @@ end cep_json = cep(:accept => :json) #discover the 'addURI' for creating Machine add_uri = discover_uri_for("add", "machines") - RestClient.post(add_uri, - "<Machine>" + + post(add_uri, + "<MachineCreate xmlns=\"#{CIMI::Test::CIMI_NAMESPACE}\">" + "<name>cimi_machine_" + fmt.to_s() + "</name>" + "<machineTemplate>" + "<machineConfig " + @@ -65,8 +65,7 @@ end "<machineImage " + "href=\"" + get_a(cep_json, "machineImage") + "\"/>" + "</machineTemplate>" + - "</Machine>", - {'Authorization' => api.basic_auth, :accept => fmt}) + "</MachineCreate>", :content_type => :xml) end it "should add resource for cleanup" do diff --git a/tests/cimi/part3_test.rb b/tests/cimi/part3_test.rb index d7bb45b..9703618 100644 --- a/tests/cimi/part3_test.rb +++ b/tests/cimi/part3_test.rb @@ -39,7 +39,7 @@ class CreateNewMachineFromMachineTemplate < CIMI::Test::Spec #create a machineTemplate for use in these tests: cep_json = cep(:accept => :json) mach_templ_add_uri = discover_uri_for("add", "machineTemplates") - mach_templ_created = RestClient.post(mach_templ_add_uri, + mach_templ_created = post(mach_templ_add_uri, "<MachineTemplateCreate>" + "<name>cimi_machineTemplate1</name>"+ "<description>A CIMI MachineTemplate, created by part3_test.rb</description>"+ @@ -49,7 +49,7 @@ class CreateNewMachineFromMachineTemplate < CIMI::Test::Spec "<machineImage " + "href=\"" + get_a(cep_json, "machineImage") + "\"/>" + "</MachineTemplateCreate>", - {'Authorization' => api.basic_auth, :accept => :json}) + :accept => :json, :content_type => :xml) # 3.1: Query the CEP model :machineTemplate do |fmt| @@ -91,14 +91,14 @@ class CreateNewMachineFromMachineTemplate < CIMI::Test::Spec cep_json = cep(:accept => :json) #discover the 'addURI' for creating Machine add_uri = discover_uri_for("add", "machines") - RestClient.post(add_uri, + post(add_uri, "<Machine>" + "<name>cimi_machine_from_template" + fmt.to_s() + "</name>" + "<description> Created machine from template" + fmt.to_s() + "</description>" + "<machineTemplate " + "href=\"" + get_a(cep_json, "machineTemplate")+ "\"/>" + "</Machine>", - {'Authorization' => api.basic_auth, :accept => fmt}) + :accept => fmt, :content_type => :xml) end it "should add resource for cleanup" do diff --git a/tests/cimi/part4_test.rb b/tests/cimi/part4_test.rb index f065104..72f22af 100644 --- a/tests/cimi/part4_test.rb +++ b/tests/cimi/part4_test.rb @@ -53,8 +53,8 @@ class AddVolumeToMachine < CIMI::Test::Spec # Create a machine to attach the volume cep_json = cep(:accept => :json) machine_add_uri = discover_uri_for("add", "machines") - machine = RestClient.post(machine_add_uri, - "<Machine>" + + machine = post(machine_add_uri, + "<MachineCreate xmlns=\"#{CIMI::Test::CIMI_NAMESPACE}\">" + "<name>cimi_machine</name>" + "<machineTemplate>" + "<machineConfig " + @@ -62,13 +62,13 @@ class AddVolumeToMachine < CIMI::Test::Spec "<machineImage " + "href=\"" + get_a(cep_json, "machineImage") + "\"/>" + "</machineTemplate>" + - "</Machine>", - {'Authorization' => api.basic_auth, :accept => :json}) + "</MachineCreate>", + {:accept => :json, :content_type => :xml}) # 4.3: Create a new Volume model :volume do |fmt| volume_add_uri = discover_uri_for("add", "volumes") - RestClient.post(volume_add_uri, + post(volume_add_uri, "<Volume>" + "<name>cimi_volume_" + fmt.to_s() +"</name>" + "<description>volume for testing</description>" + @@ -77,7 +77,7 @@ class AddVolumeToMachine < CIMI::Test::Spec "</volumeConfig>" + "</volumeTemplate>" + "</Volume>", - {'Authorization' => api.basic_auth, :accept => fmt}) + :accept => fmt, :content_type => :xml) end it "should add resource machine resource for cleanup", :only => :json do @@ -105,7 +105,7 @@ class AddVolumeToMachine < CIMI::Test::Spec log.info(machine.json["id"].to_s() + " is the machine id") volume_add_uri = discover_uri_for("add", "volumes") - volume = RestClient.post(volume_add_uri, + volume = post(volume_add_uri, "<Volume>" + "<name>cimi_volume_for_attach</name>" + "<description>volume for attach testing</description>" + @@ -114,18 +114,18 @@ class AddVolumeToMachine < CIMI::Test::Spec "</volumeConfig>" + "</volumeTemplate>" + "</Volume>", -{'Authorization' => api.basic_auth, :accept => :json}) + :accept => :json, :content_type => :xml) log.info(volume.json["id"].to_s() + " is the volume id") # 4.4: Attach the new Volume to a Machine model :machineWithVolume, :only => :xml do attach_uri = discover_uri_for_subcollection("add", machine.json['id'], "volumes") - RestClient.post(attach_uri, + post(attach_uri, "<MachineVolume xmlns=\"http://schemas.dmtf.org/cimi/1/MachineVolume\">" + "<initialLocation>/dev/sdf</initialLocation>" + "<volume href=\"" + volume.json["id"] + "\"/>" + "</MachineVolume>", - {'Authorization' => api.basic_auth, :accept => :xml}) + :accept => :xml, :content_type => :xml) end it "should have a response code equal to 201 for attaching a volume", :only => :xml do diff --git a/tests/cimi/part5_test.rb b/tests/cimi/part5_test.rb index f1ce6c9..185668a 100644 --- a/tests/cimi/part5_test.rb +++ b/tests/cimi/part5_test.rb @@ -34,7 +34,7 @@ class ManipulateAMachine < CIMI::Test::Spec cep_json = cep(:accept => :json) #discover machine create URI: machine_add_uri = discover_uri_for("add", "machines") - machine_created = RestClient.post(machine_add_uri, + machine_created = post(machine_add_uri, "<Machine>" + "<name>cimi_machine_part5</name>" + "<machineTemplate>" + @@ -44,7 +44,7 @@ class ManipulateAMachine < CIMI::Test::Spec "href=\"" + get_a(cep_json, "machineImage") + "\"/>" + "</machineTemplate>" + "</Machine>", - {'Authorization' => api.basic_auth, :accept => :json}) + :accept => :json, :content_type => :xml) model :machine do |fmt| get machine_created.json["id"], :accept => fmt diff --git a/tests/cimi/test_helper.rb b/tests/cimi/test_helper.rb index e967e55..0d318bf 100644 --- a/tests/cimi/test_helper.rb +++ b/tests/cimi/test_helper.rb @@ -233,11 +233,11 @@ module CIMI::Test::Methods def machine_stop_start(machine, action, state) uri = discover_uri_for(action, "", machine.operations) - response = RestClient.post( uri, + response = post( uri, "<Action xmlns=\"http://schemas.dmtf.org/cimi/1\">" + "<action> http://http://schemas.dmtf.org/cimi/1/action/" + action + "</action>" + "</Action>", - {'Authorization' => api.basic_auth, :accept => :xml }) + :accept => :xml, :content_type => :xml) response.code.must_equal 202 poll_state(machine(:refetch => true), state) machine(:refetch => true).state.upcase.must_equal state -- 1.7.7.6