netscaler 0.3.0 → 0.3.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTM3NDRkODQzNmM4MzYyMDhkZmZhYzFkODYzM2M5ZDdiODI0Mzc4ZQ==
+    YzE4MzhhNWU3MThjODNiN2FmMTQwZWVjMjFkN2JjZWJmMzM0N2VmNQ==
   data.tar.gz: !binary |-
-    ZmMxMWVjMzk2Yjc2ZTAyZTRmOTlhYzEzMWJlY2M2OTUwMTI5NTMxMw==
+    YzM1YjJmMTg1YjY0NDQwMWYwZTY2NGNiZTU4Njc1MDkxODAwZmQwNA==
 SHA512:
   metadata.gz: !binary |-
-    ZGQ5YmJjOTkyNTE0ZjQxNTY0MWQ0MWMxOWE2ZDJmMjVkZTcwMWVkOWVmMWRh
-    Y2UwNDIyNDlkNWJmMjQ1ZWNmZWY3MWY0YjE4NDE0MmVlN2U1NTU3MzhhOWE3
-    YjlhM2ZiOTUxZjJlYTI0MjYzZWM3NmI1NWEwYmQxZWQ0NjhjZWU=
+    N2FiMzk2YmY5YzMyYzhiMjc3MzUxOTRkMDIyOThhYzk4MDI4ZDNlYTkyMTk2
+    MzgwYTk4ODRjZDFmNWRiMjc0MzEyZTJkMzFjYWNhM2U4MGQ5NWE3YjE3YjUy
+    NGVlMzYwMWIzNDkwNjg4MmNjN2Q5M2FjM2MxYWQzOTBhYjc0NGU=
   data.tar.gz: !binary |-
-    ZDM0MmI5YzcxYzU4OTAxZDRmNzJiZWQ5ZGYyN2ZlZGZlOGNjN2NlNGUxYzc0
-    ZDYwY2RmZTAyYmFiMmVlNWQxNTM4OGY1MmY4M2JjZDM3OWIxYTA1NjA0ZGVi
-    NWJhYTNiNWUzM2QwNTcxM2MzZjdjNWMzMzFkMmMzODg2YTk1N2U=
+    YzE4NWY3MzM5Y2Y1MjQyNjMyMjRlZWVjYjZiZTZjNDI3OTdiNDRlODIzMjZh
+    N2QyZmE3NmRmNzMyYjBkNzEyYWFkOTE2MzUzZDNlODUwZWZlZDBkZGIyMTA4
+    ZjU1NjhjNjczNDI1OTdmNDAzODFkYTM3ZjZjZDI0NGEyMTk4OWM=

data/VERSION

@@ -1 +1 @@
-0.3.0
+0.3.1

data/lib/netscaler/service.rb

@@ -30,11 +30,16 @@ module Netscaler
     ##
     # :serverName is optional, if omitted it will return all services
     # configured on the Netscaler.
-    def show(payload) # :args:  :serverName => 'foo'
-      return @netscaler.adapter.get('config/service/', args) if payload.nil?
-      payload = Netscaler.hash_hack(payload)
-      validate_payload(payload, [:serviceName])
-      return @netscaler.adapter.get("config/service/#{payload[:serviceName]}")
+    def show(payload={}) # :args:  :serverName => 'foo'
+      if payload[:serviceName] != nil then
+        payload = Netscaler.hash_hack(payload)
+        validate_payload(payload, [:serviceName])
+        return @netscaler.adapter.get("config/service/#{payload[:serviceName]}")
+      elsif payload == {} then
+        return @netscaler.adapter.get('config/service/')
+      else
+        raise ArgumentError, 'payload supplied must have been missing :serviceName'
+      end
     end
 
     ##

data/netscaler.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: netscaler 0.3.0 ruby lib
+# stub: netscaler 0.3.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "netscaler"
-  s.version = "0.3.0"
+  s.version = "0.3.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Jeremy Custenborder", "David Andrew", "Jarrett Irons"]
-  s.date = "2014-10-26"
+  s.date = "2014-11-08"
   s.description = "Netscaler api working against the Citrix Nitro api.  Currently supports Nitro 9.3.  Hope to add support for 10.X.  Currently has support for som basics such as adding servers/services/servicegroups."
   s.email = ["jeremy@scarcemedia.com", "david.andrew@webtrends.com", "jarrett.irons@gmail.com"]
   s.extra_rdoc_files = [
@@ -50,6 +50,7 @@ Gem::Specification.new do |s|
     "spec/netscaler_spec.rb",
     "spec/policy_stringmap_spec.rb",
     "spec/server_spec.rb",
+    "spec/service_spec.rb",
     "spec/servicegroup_spec.rb",
     "spec/spec_helper.rb"
   ]

data/spec/service_spec.rb

@@ -0,0 +1,41 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'netscaler'
+require 'netscaler/mock_adapter'
+
+describe Netscaler::Server do
+  connection = Netscaler::Connection.new 'hostname'=> 'foo', 'password' => 'bar', 'username'=> 'bar'
+  connection.adapter = Netscaler::MockAdapter.new :body => '{ "errorcode": 0, "message": "Done" }'
+
+  context 'when adding a new service' do
+    it 'a name is required' do
+      expect {
+        connection.service.add(:serverName => 'foo', :serviceType => 'HTTP', :port => '80')
+      }.should raise_error(ArgumentError, /name/)
+    end
+
+    it 'a serverName is required' do
+      expect {
+        connection.service.add(:name => 'foo_80', :serviceType => 'HTTP', :port => '80')
+      }.should raise_error(ArgumentError, /serverName/)
+    end
+
+    it 'a serviceType is required' do
+      expect {
+        connection.service.add(:name => 'foo_80', :serverName => 'foo', :port => '80')
+      }.should raise_error(ArgumentError, /serviceType/)
+    end
+
+    it 'a port is required' do
+      expect {
+        connection.service.add(:name => 'foo_80', :serverName => 'foo', :serviceType => 'HTTP')
+      }.should raise_error(ArgumentError, /port/)
+    end
+
+
+    it 'returns a Hash object if all necessary args are supplied' do
+      result = connection.service.add(:name => 'foo_80', :serverName => 'foo', :serviceType => 'HTTP', :port => '80')
+      result.should be_kind_of(Hash)
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: netscaler
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Jeremy Custenborder
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-26 00:00:00.000000000 Z
+date: 2014-11-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -168,6 +168,7 @@ files:
 - spec/netscaler_spec.rb
 - spec/policy_stringmap_spec.rb
 - spec/server_spec.rb
+- spec/service_spec.rb
 - spec/servicegroup_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/GravityLabs/netscaler