CIMI System testing status - patch submitted for review/debugging only

[rl...@redhat.com]

Putting the system test code out on list for review/discussion. Will use github for the final code submission ...

The system1_test.rb follows the scenario outlines by the last document attached to http://mantis.dmtf.org/mantis/view.php?id=2004. 

Issues to be addressed:

 - There is no add operation for system templates (see DTACLOUD-530) so we have to use the template availble for the mock provider. Problem: a system created from template1 would not meet the requirements of the scenario which stipulates that the system must have two machines (with particular names) and a volume template. template1 has three machine and the volumes are inline.

 - afaict, creating a system using this template *does not* result in the machines being created and attached to the system. The system created contained only one machine. (I sent Marios the full output for debugging). Possibly an incorrect systemCreate xml passed - please comment if this is the case

 - Could not start the system - see DTACLOUD-539 .. therefore the test stops there and cannot continue to check machines and stop the system.

 - Question: would stopping the system be equivalent to deleting it? Should associated machine be deleted when the system is deleted?

[PATCH] CIMI - Test - systems scenario test

[rl...@redhat.com]

From: Ronelle Landy <rl...@redhat.com>

---
 tests/cimi/system1_test.rb | 149 +++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 149 insertions(+)
 create mode 100644 tests/cimi/system1_test.rb

diff --git a/tests/cimi/system1_test.rb b/tests/cimi/system1_test.rb
new file mode 100644
index 0000000..c9bd121
--- /dev/null
+++ b/tests/cimi/system1_test.rb
@@ -0,0 +1,149 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.  The
+# ASF licenses this file to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance with the
+# License.  You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+# License for the specific language governing permissions and limitations
+# under the License.
+
+$:.unshift File.join(File.dirname(__FILE__))
+
+require "test_helper.rb"
+
+class SystemTemplate < CIMI::Test::Spec
+  RESOURCE_URI = "http://schemas.dmtf.org/cimi/1/System"
+
+  ROOTS = [ "machines" , "systemTemplates" , "volumeTemplates"]
+  need_capability('add', 'systems')
+
+  # Cleanup for resources created for the test
+  MiniTest::Unit.after_tests {  teardown(@@created_resources, api.basic_auth) }
+
+  #  Ensure test executes in test plan order
+  i_suck_and_my_tests_are_order_dependent!
+
+  # 1.1: Query the CEP
+  model :subject, :cache => true do |fmt|
+    cep(:accept => fmt)
+  end
+
+  # CEP.machines, CEP.systemTemplates and CEP.volumeTemplates must be set
+  query_the_cep(ROOTS)
+
+  # 1.2 Query the systemTemplate collection
+  cep_json = cep(:accept => :json)
+
+  #model :system_template do |fmt|
+  #  get(cep_json.json[ROOTS[1]]["href"], :accept => :fmt)
+  #end
+
+  it "should return a 200 code" do
+    get(cep_json.json[ROOTS[1]]["href"], :accept => :json).code.must_equal 200
+    get(cep_json.json[ROOTS[1]]["href"], :accept => :xml).code.must_equal 200
+  end
+
+  # Retrieve existing system templates or the address to create a new system template
+  begin
+    # systemplate_add_uri = discover_uri_for("add", "system_templates")
+    it "should have at least one system template" do
+      r = ROOTS[1].underscore.to_sym
+      model = fetch(subject.send(r).href)
+      log.info(model.attribute_values[r][0])
+      assert_equal model.attribute_values[r][0].nil?(), false
+      model.attribute_values[r][0].id.must_be_uri
+
+       # 1.3 Select or Create a System Template
+       puts "System Template: " + get_a(cep_json, "systemTemplate")
+     end
+  rescue RuntimeError =>e
+  end
+
+  system_add_uri = discover_uri_for("add", "systems")
+  # 1.4 Create a new system from the systemTemplate
+  system_created = post(system_add_uri,
+    "<systemCreate xmlns=\"#{CIMI::Test::CIMI_NAMESPACE}\">" +
+    "<name>test_system</name>" +
+    "<systemTemplate href=\"" + get_a(cep_json, "systemTemplate") + "\"/>" +
+    "</systemCreate>", :content_type => :xml)
+
+  model :systems do |fmt|
+    get cep_json.json["systems"]["href"], :accept => fmt
+  end
+
+  it "should add resource machine resource for cleanup", :only => :json do
+    @@created_resources[:machines] << system_created.headers[:location]
+  end
+
+  it "should allow a system to be created" do
+    system_created.headers[:location].must_be_uri
+    system_created.code.must_be_one_of [201, 202]
+  end
+
+  # model :test_system_created do |fmt|
+  #   get(system_created.headers[:location], :accept => fmt)
+  # end
+
+  # 1.5 Query the new System
+  it "should return a representation of a system", :only => :json do
+    puts fetch(system_created.headers[:location]).id
+    #test_system_created
+    test_system_created = get(fetch(system_created.headers[:location]).id, :accept => :json)
+    test_system_created.code.must_equal 200
+    test_system_created.json["resourceURI"].must_equal RESOURCE_URI
+  end
+
+  # 1.6 Query the System SystemMachine collection
+  it "should have a SystemMachine collection with two entries", :only => :json do
+    #test_system_created
+    test_system_created = get(fetch(system_created.headers[:location]).id, :accept => :json)
+    system_machines = get(test_system_created.json["machines"]["href"], :accept => :json)
+    #system_machines.json["count"].must_equal 2
+    #system_machines.json["systemMachines"][0]["id"].  end
+  end
+
+  # 1.7 Query the System SystemVolume collection
+  it "should contain a single entry referencing a Volume named as indicated in the SystemTemplate", :only => :json do
+    #test_system_created
+    test_system_created = get(fetch(system_created.headers[:location]).id, :accept => :json)
+    system_volumes = get(test_system_created.json["volumes"]["href"], :accept => :json)
+    system_volumes.json["count"].must_equal 1
+    #system_volumes.json["systemVolumes"][0]["id"].must_equal "volume from template"
+  end
+
+  # 1.8 Check that the volume is attached to machine(s)
+  # Optional - skipping
+
+  # 1.9 Query the System SystemCredential collection
+  # Optional - skipping
+
+  # 1.10 Starting the new System
+  it "should be able to start the system", :only => :json  do
+    test_system_created = get(fetch(system_created.headers[:location]).id, :accept => :json)
+    unless test_system_created.json["state"].eql?("STARTED")
+      uri = discover_uri_for("start", "", test_system_created.json["operations"])
+      response = post( uri,
+            "<Action xmlns=\"http://schemas.dmtf.org/cimi/1\">" +
+              "<action> http://http://schemas.dmtf.org/cimi/1/action/start</action>" +
+            "</Action>",
+            :accept => :xml, :content_type => :xml)
+      response.code.must_equal 202
+      poll_state(get(fetch(system_created.headers[:location]).id, :accept => :json), "STARTED")
+      get(fetch(system_created.headers[:location]).id, :accept => :json).json["state"].upcase.must_equal "STARTED"
+    end
+  end
+
+  # 1.11 Check that the machines are started
+
+  # 1.12 Stop the new System
+
+  # 1.13 Check that the machines are stopped
+
+end
-- 
1.8.1.4