net-tns 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ae395e30d483af200c9eda9c522288af1cdad4e1
-  data.tar.gz: e4399ebb1818042b98d92b8e18e54de0e0b79d12
+  metadata.gz: c79686c4bd0a8d905987a42092ec1fa49cb140fa
+  data.tar.gz: 5a95a383671604fb14980ea2cfe099055f488db5
 SHA512:
-  metadata.gz: 9bc129315cca6f165f1fdecbb211b436034fdb8606083b7e5d1d1ae80a4bafe87447e1463f1a21042afdb68e8b016b6bb392955f0b7f83165924db4d18840a7c
-  data.tar.gz: f17cdca07bcc38a3199b30ded2eb9fe1384c892d43c8ff2d8865e746e1e8ff116763715cfde565ae339e8422670541a498333defaebd627ab86bdc7c9c57987f
+  metadata.gz: c135287fde902c9019eb0da3c5b38c71fb94d13ee90e4625f622bd7c8d71df0ca690c362c971843621da788f73476437aa281c04bb5698ebc073b3d2da35c8f5
+  data.tar.gz: afdce9a532326bb42af0fc093d6b08f2d4a26bc32369fd2c14c9014524b0b7f911a08e3f5241aa0f85896d021dbb4c80370ca567fb1a8a139a30de21a0b4920a

data/lib/net/tns/client.rb

@@ -24,7 +24,7 @@ module Net
           request = ConnectPacket.new(:data => "(CONNECT_DATA=(COMMAND=VERSION))")
 
           response = conn.send_and_receive( request )
-          raise "Expected AcceptPacket in response" unless response.is_a?(AcceptPacket)
+          raise Exceptions::ProtocolException.new("Expected AcceptPacket in response (got #{response.class})") unless response.is_a?(AcceptPacket)
 
           version_data = response.data
         rescue Exceptions::RefuseMessageReceived => refuse_err

data/lib/net/tns/gem_version.rb

@@ -1,5 +1,5 @@
 module Net
   module TNS
-    VERSION = "1.1.0"
+    VERSION = "1.1.1"
   end
 end

data/lib/net/tns/helpers/string_helpers.rb

@@ -4,33 +4,12 @@ module Net::TNS
   module StringHelpers
     HEXCHARS = [("0".."9").to_a, ("a".."f").to_a].flatten
 
-    #From the Ruby Black Bag (http://github.com/emonti/rbkb/)
-    # Convert a string to ASCII hex string. Supports a few options for format:
-    #
-    # :delim - delimter between each hex byte
-    # :prefix - prefix before each hex byte
-    # :suffix - suffix after each hex byte
-    #
-    def tns_hexify(opts={})
-      delim = opts[:delim]
-      pre = (opts[:prefix] || "")
-      suf = (opts[:suffix] || "")
-
-      if (rx=opts[:rx]) and not rx.kind_of? Regexp
-        raise "rx must be a regular expression for a character class"
-      end
-
-      out=Array.new
-
-      self.each_byte do |c|
-        hc = if (rx and not rx.match c.chr)
-               c.chr
-             else
-               pre + (HEXCHARS[(c >> 4)] + HEXCHARS[(c & 0xf )]) + suf
-             end
-        out << (hc)
-      end
-      out.join(delim)
+    # Adapted from the Ruby Black Bag (http://github.com/emonti/rbkb/)
+    # Convert a string to ASCII hex string
+    def tns_hexify
+      self.each_byte.map do |byte|
+        (HEXCHARS[(byte >> 4)] + HEXCHARS[(byte & 0xf )])
+      end.join()
     end
 
     # Convert ASCII hex string to raw.

data/lib/net/tns/packet.rb

@@ -25,7 +25,7 @@ module Net
         @@tns_packet_types ||= {}
         if @@tns_packet_classes.has_key?(tns_type)
           existing_class = @@tns_packet_classes[tns_type]
-          raise("Duplicate TNS Types Defined: #{existing_class} and #{self} both have a type of #{tns_type}")
+          raise ArgumentError.new("Duplicate TNS Types Defined: #{existing_class} and #{self} both have a type of #{tns_type}")
         end
 
         @@tns_packet_classes[tns_type] = self
@@ -46,13 +46,16 @@ module Net
         end
 
         if header_raw.nil? || header_raw.length != Header::LENGTH
-          raise Exceptions::ProtocolException
+          header_length = header_raw.length unless header_raw.nil?
+          raise Exceptions::ProtocolException.new("Failed to read complete header. Read #{header_length.to_i} bytes.")
         end
 
         header = Header.new()
         header.read( header_raw )
         Net::TNS.logger.debug("Read header. Reported packet length is #{header.packet_length} bytes")
-        raise Exceptions::ProtocolException if header.packet_length > SESSION_DATA_UNIT_SIZE
+        if header.packet_length > SESSION_DATA_UNIT_SIZE
+          raise Exceptions::ProtocolException.new("Packet length in header (#{header.packet_length}) is longer than SDU size.")
+        end
 
         payload_raw = socket.read( header.packet_length - Header::LENGTH )
         packet_raw = header_raw + payload_raw
@@ -62,7 +65,7 @@ module Net
         end
 
         unless packet_raw.length == header.packet_length
-          raise Net::TNS::Exceptions::ProtocolException
+          raise Net::TNS::Exceptions::ProtocolException.new("Failed to read entire packet (read #{packet_raw.length} of #{header.packet_length} bytes).")
         end
 
         new_packet = payload_class.read( packet_raw )

data/lib/net/tti/message.rb

@@ -22,7 +22,7 @@ module Net
         @@ttc_codes ||= {}
         if @@ttc_classes.has_key?(ttc_code)
           existing_class = @@ttc_classes[ttc_code]
-          raise("Duplicate TTC response handlers defined: #{existing_class} and #{self} both have TTC code of #{ttc_code}")
+          raise ArgumentError.new("Duplicate TTC response handlers defined: #{existing_class} and #{self} both have TTC code of #{ttc_code}")
         end
 
         @@ttc_classes[ttc_code] = self

data/lib/net/tti/messages/function_calls/pre_authentication_response.rb

@@ -23,7 +23,7 @@ module Net
 
       def find_param(key, raise_if_not_found=false)
         param = self.parameters.find {|p| p.kvp_key == key}
-        raise "No #{key} parameter found" if param.nil? && raise_if_not_found
+        raise Net::TTI::Exceptions::TTIException.new("No #{key} parameter found") if param.nil? && raise_if_not_found
         return param
       end
       private :find_param

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: net-tns
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Chris Woodbury
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-29 00:00:00.000000000 Z
+date: 2014-12-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bindata
@@ -53,6 +53,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A pure Ruby (partial) implementation of the Oracle TNS protocol
 email: woodbusy@gmail.com
 executables: []