allow from_definition to be invoked from base class

This helps testing.

libraries/pacemaker/cib_object.rb

@@ -36,24 +36,28 @@ module Pacemaker
       def from_name(name)
         definition = get_definition(name)
         return nil unless definition and ! definition.empty?
-        obj_type = type(definition)
+        from_definition(definition)
-        subclass = @@subclasses[obj_type]
-        unless subclass
-          raise "No subclass of #{self.name} was registered with type '#{obj_type}'"
-        end
-        obj = subclass.from_definition(definition)
-        return nil unless obj
-        unless name == obj.name
-          raise "Name '#{obj.name}' in definition didn't match name '#{name}' used for retrieval"
-        end
-        obj
       end
 
       def from_definition(definition)
-        obj = new(name)
+        if method(__method__).owner == self.singleton_class
-        obj.definition = definition
+          # Invoked via (this) base class
-        obj.parse_definition
+          obj_type = type(definition)
-        obj
+          subclass = @@subclasses[obj_type]
+          unless subclass
+            raise "No subclass of #{self.name} was registered with type '#{obj_type}'"
+          end
+          return subclass.from_definition(definition)
+        else
+          # Invoked via subclass
+          obj = new(name)
+          unless name == obj.name
+            raise "Name '#{obj.name}' in definition didn't match name '#{name}' used for retrieval"
+          end
+          obj.definition = definition
+          obj.parse_definition
+          obj
+        end
       end
     end
 

spec/libraries/pacemaker/resource/primitive_spec.rb

@@ -18,6 +18,11 @@ describe Pacemaker::Resource::Primitive do
     expect(obj.is_a? Pacemaker::Resource::Primitive).to be_true
   end
 
+  it "should be instantiated via Pacemaker::CIBObject.from_definition" do
+    obj = Pacemaker::CIBObject.from_definition(@primitive.definition_string)
+    expect(obj.is_a? Pacemaker::Resource::Primitive).to be_true
+  end
+
   it "should barf if the loaded definition's type is not primitive" do
     Mixlib::ShellOut.any_instance.stub(:error!)
     expect_any_instance_of(Mixlib::ShellOut) \