Implement self.prefetch for zabbix_hostgroup

This is another change for #570.

Author: baurmatt

lib/puppet/provider/zabbix_hostgroup/ruby.rb

@@ -2,14 +2,32 @@
 Puppet::Type.type(:zabbix_hostgroup).provide(:ruby, parent: Puppet::Provider::Zabbix) do
   confine feature: :zabbixapi
 
+  def self.instances
+    api_hostgroups = zbx.hostgroups.all
+    api_hostgroups.map do |group_name, _id|
+      new(
+        ensure: :present,
+        name: group_name
+      )
+    end
+  end
+
+  def self.prefetch(resources)
+    instances.each do |prov|
+      if (resource = resources[prov.name])
+        resource.provider = prov
+      end
+    end
+  end
+
   def create
     # Connect to zabbix api
     hgid = zbx.hostgroups.create(name: @resource[:name])
     hgid
   end
 
   def exists?
-    zbx.hostgroups.get_id(name: @resource[:name])
+    @property_hash[:ensure] == :present
   end
 
   def destroy

spec/unit/puppet/provider/zabbix_hostgroup/ruby.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Puppet::Type.type(:zabbix_hostgroup).provider(:ruby) do
+  let(:resource) do
+    Puppet::Type.type(:zabbix_hostgroup).new(
+      name: 'Testgroup'
+    )
+  end
+  let(:provider) { resource.provider }
+
+  it 'be an instance of the correct provider' do
+    expect(provider).to be_an_instance_of Puppet::Type::Zabbix_hostgroup::ProviderRuby
+  end
+
+  [:instances, :prefetch].each do |method|
+    it "should respond to the class method #{method}" do
+      expect(described_class).to respond_to(method)
+    end
+  end
+
+  [:create, :exists?, :destroy].each do |method|
+    it "should respond to the instance method #{method}" do
+      expect(described_class.new).to respond_to(method)
+    end
+  end
+end