tiny_tds 1.0.4 → 2.1.5

data/test/bin/install-freetds.sh

@@ -0,0 +1,20 @@
+#!/usr/bin/env bash
+
+set -x
+set -e
+
+if [ -z "$FREETDS_VERSION" ]; then
+  FREETDS_VERSION=$(ruby -r "./ext/tiny_tds/extconsts.rb" -e "puts FREETDS_VERSION")
+fi
+
+wget http://www.freetds.org/files/stable/freetds-$FREETDS_VERSION.tar.gz
+tar -xzf freetds-$FREETDS_VERSION.tar.gz
+cd freetds-$FREETDS_VERSION
+./configure --prefix=/opt/local \
+            --with-openssl=/opt/local \
+            --with-tdsver=7.3
+make
+make install
+cd ..
+rm -rf freetds-$FREETDS_VERSION
+rm freetds-$FREETDS_VERSION.tar.gz

data/test/bin/install-openssl.sh

@@ -0,0 +1,18 @@
+#!/usr/bin/env bash
+
+set -x
+set -e
+
+if [ -z "$OPENSSL_VERSION" ]; then
+  OPENSSL_VERSION=$(ruby -r "./ext/tiny_tds/extconsts.rb" -e "puts OPENSSL_VERSION")
+fi
+
+wget https://www.openssl.org/source/openssl-$OPENSSL_VERSION.tar.gz
+tar -xzf openssl-$OPENSSL_VERSION.tar.gz
+cd openssl-$OPENSSL_VERSION
+./config --prefix=/opt/local --openssldir=/opt/local
+make
+make install_sw install_ssldirs
+cd ..
+rm -rf openssl-$OPENSSL_VERSION
+rm openssl-$OPENSSL_VERSION.tar.gz

data/test/bin/setup.sh

@@ -0,0 +1,19 @@
+#!/usr/bin/env bash
+
+set -x
+set -e
+
+tag=2017-GA
+
+docker pull metaskills/mssql-server-linux-tinytds:$tag
+
+container=$(docker ps -a -q --filter ancestor=metaskills/mssql-server-linux-tinytds:$tag)
+if [[ -z $container ]]; then
+  docker run -p 1433:1433 -d metaskills/mssql-server-linux-tinytds:$tag && sleep 10
+  exit
+fi
+
+container=$(docker ps -q --filter ancestor=metaskills/mssql-server-linux-tinytds:$tag)
+if [[ -z $container ]]; then
+  docker start $container && sleep 10
+fi

data/test/client_test.rb

@@ -2,9 +2,7 @@
 require 'test_helper'
 
 class ClientTest < TinyTds::TestCase
-
-  describe 'With valid credentials' do
-
+  describe 'with valid credentials' do
     before do
       @client = new_connection
     end
@@ -46,25 +44,33 @@ class ClientTest < TinyTds::TestCase
       assert_equal "''hello''", @client.escape("'hello'")
     end
 
-    it 'allows valid iconv character set' do
-      ['CP850', 'CP1252', 'ISO-8859-1'].each do |encoding|
-        client = new_connection(:encoding => encoding)
-        assert_equal encoding, client.charset
-        assert_equal Encoding.find(encoding), client.encoding
-        client.close
+    ['CP850', 'CP1252', 'ISO-8859-1'].each do |encoding|
+      it "allows valid iconv character set - #{encoding}" do
+        begin
+          client = new_connection(:encoding => encoding)
+          assert_equal encoding, client.charset
+          assert_equal Encoding.find(encoding), client.encoding
+        ensure
+          client.close if client
+        end
       end
     end
 
     it 'must be able to use :host/:port connection' do
-      host = ENV['TINYTDS_UNIT_HOST_TEST'] || ENV['TINYTDS_UNIT_HOST']
+      host = ENV['TINYTDS_UNIT_HOST_TEST'] || ENV['TINYTDS_UNIT_HOST'] || 'localhost'
       port = ENV['TINYTDS_UNIT_PORT_TEST'] || ENV['TINYTDS_UNIT_PORT'] || 1433
-      client = new_connection dataserver: nil, host: host, port: port
-      client.close
+      begin
+        client = new_connection dataserver: nil, host: host, port: port
+      ensure
+        client.close if client
+      end
     end unless sqlserver_azure?
-
   end
 
   describe 'With in-valid options' do
+    before(:all) do
+      init_toxiproxy
+    end
 
     it 'raises an argument error when no :host given and :dataserver is blank' do
       assert_raises(ArgumentError) { new_connection :dataserver => nil, :host => nil }
@@ -78,9 +84,16 @@ class ClientTest < TinyTds::TestCase
       options = connection_options :login_timeout => 1, :dataserver => 'DOESNOTEXIST'
       action = lambda { new_connection(options) }
       assert_raise_tinytds_error(action) do |e|
-        assert_equal 20012, e.db_error_number
-        assert_equal 2, e.severity
-        assert_match %r{server name not found in configuration files}i, e.message, 'ignore if non-english test run'
+        # Not sure why tese are different.
+        if ruby_darwin?
+          assert_equal 20009, e.db_error_number
+          assert_equal 9, e.severity
+          assert_match %r{is unavailable or does not exist}i, e.message, 'ignore if non-english test run'
+        else
+          assert_equal 20012, e.db_error_number
+          assert_equal 2, e.severity
+          assert_match %r{server name not found in configuration files}i, e.message, 'ignore if non-english test run'
+        end
       end
       assert_new_connections_work
     end
@@ -119,30 +132,64 @@ class ClientTest < TinyTds::TestCase
       end
     end
 
-    it 'must run this test to prove we account for dropped connections' do
-      skip
+    it 'raises TinyTds exception with tcp socket network failure' do
+      skip if ENV['CI'] && ENV['APPVEYOR_BUILD_FOLDER'] # only CI using docker
+      begin
+        client = new_connection timeout: 2, port: 1234
+        assert_client_works(client)
+        action = lambda { client.execute("waitfor delay '00:00:05'").do }
+
+        # Use toxiproxy to close the TCP socket after 1 second.
+        # We want TinyTds to execute the statement, hit the timeout configured above, and then not be able to use the network to cancel
+        # the network connection needs to close after the sql batch is sent and before the timeout above is hit
+        Toxiproxy[:sqlserver_test].toxic(:slow_close, delay: 1000).apply do
+          assert_raise_tinytds_error(action) do |e|
+            assert_equal 20003, e.db_error_number
+            assert_equal 6, e.severity
+            assert_match %r{timed out}i, e.message, 'ignore if non-english test run'
+          end
+        end
+      ensure
+        assert_new_connections_work
+      end
+    end
+
+    it 'raises TinyTds exception with dead connection network failure' do
+      skip if ENV['CI'] && ENV['APPVEYOR_BUILD_FOLDER'] # only CI using docker
       begin
-        client = new_connection :login_timeout => 2, :timeout => 2
+        client = new_connection timeout: 2, port: 1234
         assert_client_works(client)
-        STDOUT.puts "Disconnect network!"
-        sleep 10
-        STDOUT.puts "This should not get stuck past 6 seconds!"
-        action = lambda { client.execute('SELECT 1 as [one]').each }
+        action = lambda { client.execute("waitfor delay '00:00:05'").do }
+
+        # Use toxiproxy to close the network connection after 1 second.
+        # We want TinyTds to execute the statement, hit the timeout configured above, and then not be able to use the network to cancel
+        # the network connection needs to close after the sql batch is sent and before the timeout above is hit
+        Toxiproxy[:sqlserver_test].toxic(:timeout, timeout: 1000).apply do
+          assert_raise_tinytds_error(action) do |e|
+            assert_equal 20047, e.db_error_number
+            assert_includes [1,9], e.severity
+            assert_match %r{dead or not enabled}i, e.message, 'ignore if non-english test run'
+          end
+        end
+      ensure
+        assert_new_connections_work
+      end
+    end
+
+    it 'raises TinyTds exception with login timeout' do
+      skip if ENV['CI'] && ENV['APPVEYOR_BUILD_FOLDER'] # only CI using docker
+      begin
+        action = lambda do
+          Toxiproxy[:sqlserver_test].toxic(:timeout, timeout: 0).apply do
+            new_connection login_timeout: 1, port: 1234
+          end
+        end
         assert_raise_tinytds_error(action) do |e|
           assert_equal 20003, e.db_error_number
           assert_equal 6, e.severity
           assert_match %r{timed out}i, e.message, 'ignore if non-english test run'
         end
       ensure
-        STDOUT.puts "Reconnect network!"
-        sleep 10
-        action = lambda { client.execute('SELECT 1 as [one]').each }
-        assert_raise_tinytds_error(action) do |e|
-          assert_equal 20047, e.db_error_number
-          assert_equal 1, e.severity
-          assert_match %r{dead or not enabled}i, e.message, 'ignore if non-english test run'
-        end
-        close_client(client)
         assert_new_connections_work
       end
     end
@@ -161,57 +208,68 @@ class ClientTest < TinyTds::TestCase
 
   end
 
-  describe 'Private methods' do
-
+  describe '#parse_username' do
     let(:client) { @client = new_connection }
 
-    it '#parse_username returns username if azure is not true' do
-      username = 'user@abc123.database.windows.net'
-      client.send(:parse_username, username: username).must_equal username
+    it 'returns username if azure is not true' do
+      _(
+        client.send(:parse_username, username: 'user@abc123.database.windows.net')
+      ).must_equal 'user@abc123.database.windows.net'
     end
 
-    it '#parse_username returns short username if azure is true' do
-      client.send(:parse_username,
-        username: 'user@abc123.database.windows.net',
-        host: 'abc123.database.windows.net',
-        azure: true
+    it 'returns short username if azure is true' do
+      _(
+        client.send(
+          :parse_username,
+          username: 'user@abc123.database.windows.net',
+          host:     'abc123.database.windows.net',
+          azure:    true
+        )
       ).must_equal 'user@abc123'
     end
 
-    it '#parse_username returns full username if azure is false' do
-      client.send(:parse_username,
-        username: 'user@abc123.database.windows.net',
-        host: 'abc123.database.windows.net',
-        azure: false
+    it 'returns full username if azure is false' do
+      _(
+        client.send(
+          :parse_username,
+          username: 'user@abc123.database.windows.net',
+          host:     'abc123.database.windows.net',
+          azure:    false
+        )
       ).must_equal 'user@abc123.database.windows.net'
     end
 
-    it '#parse_username returns short username if passed and azure is true' do
-      client.send(:parse_username,
-        username: 'user@abc123',
-        host: 'abc123.database.windows.net',
-        azure: true
+    it 'returns short username if passed and azure is true' do
+      _(
+        client.send(
+          :parse_username,
+          username: 'user@abc123',
+          host:     'abc123.database.windows.net',
+          azure:    true
+        )
       ).must_equal 'user@abc123'
     end
 
-    it '#parse_username returns username with servername if passed and azure is true' do
-      client.send(:parse_username,
-        username: 'user',
-        host: 'abc123.database.windows.net',
-        azure: true
+    it 'returns username with servername if passed and azure is true' do
+      _(
+        client.send(
+          :parse_username,
+          username: 'user',
+          host:     'abc123.database.windows.net',
+          azure:    true
+        )
       ).must_equal 'user@abc123'
     end
 
-    it '#parse_username returns username with servername if passed and azure is false' do
-      client.send(:parse_username,
-        username: 'user',
-        host: 'abc123.database.windows.net',
-        azure: false
+    it 'returns username with servername if passed and azure is false' do
+      _(
+        client.send(
+          :parse_username,
+          username: 'user',
+          host:     'abc123.database.windows.net',
+          azure:    false
+        )
       ).must_equal 'user'
     end
-
   end
-
-
 end
-

data/test/gem_test.rb

@@ -0,0 +1,179 @@
+# encoding: utf-8
+require 'test_helper'
+require 'tiny_tds/gem'
+
+class GemTest < MiniTest::Spec
+  gem_root ||= File.expand_path '../..', __FILE__
+
+  describe TinyTds::Gem do
+
+    # We're going to muck with some system globals so lets make sure
+    # they get set back later
+    original_host = RbConfig::CONFIG['host']
+    original_pwd = Dir.pwd
+
+    after do
+      RbConfig::CONFIG['host'] = original_host
+      Dir.chdir original_pwd
+    end
+
+    describe '#root_path' do
+      let(:root_path) { TinyTds::Gem.root_path }
+
+      it 'should be the root path' do
+        _(root_path).must_equal gem_root
+      end
+
+      it 'should be the root path no matter the cwd' do
+        Dir.chdir '/'
+
+        _(root_path).must_equal gem_root
+      end
+    end
+
+    describe '#ports_root_path' do
+      let(:ports_root_path) { TinyTds::Gem.ports_root_path }
+
+      it 'should be the ports path' do
+        _(ports_root_path).must_equal File.join(gem_root,'ports')
+      end
+
+      it 'should be the ports path no matter the cwd' do
+        Dir.chdir '/'
+
+        _(ports_root_path).must_equal File.join(gem_root,'ports')
+      end
+    end
+
+    describe '#ports_bin_paths' do
+      let(:ports_bin_paths) { TinyTds::Gem.ports_bin_paths }
+
+      describe 'when the ports directories exist' do
+        let(:fake_bin_paths) do
+          ports_host_root = File.join(gem_root, 'ports', 'fake-host-with-dirs')
+          [
+            File.join('a','bin'),
+            File.join('a','inner','bin'),
+            File.join('b','bin')
+          ].map do |p|
+            File.join(ports_host_root, p)
+          end
+        end
+
+        before do
+          RbConfig::CONFIG['host'] = 'fake-host-with-dirs'
+          fake_bin_paths.each do |path|
+            FileUtils.mkdir_p(path)
+          end
+        end
+
+        after do
+          FileUtils.remove_entry_secure(
+            File.join(gem_root, 'ports', 'fake-host-with-dirs'), true
+          )
+        end
+
+        it 'should return all the bin directories' do
+          _(ports_bin_paths.sort).must_equal fake_bin_paths.sort
+        end
+
+        it 'should return all the bin directories regardless of cwd' do
+          Dir.chdir '/'
+          _(ports_bin_paths.sort).must_equal fake_bin_paths.sort
+        end
+      end
+
+      describe 'when the ports directories are missing' do
+        before do
+          RbConfig::CONFIG['host'] = 'fake-host-without-dirs'
+        end
+
+        it 'should return no directories' do
+          _(ports_bin_paths).must_be_empty
+        end
+
+        it 'should return no directories regardless of cwd' do
+          Dir.chdir '/'
+          _(ports_bin_paths).must_be_empty
+        end
+      end
+    end
+
+    describe '#ports_lib_paths' do
+      let(:ports_lib_paths) { TinyTds::Gem.ports_lib_paths }
+
+      describe 'when the ports directories exist' do
+        let(:fake_lib_paths) do
+          ports_host_root = File.join(gem_root, 'ports', 'fake-host-with-dirs')
+          [
+            File.join('a','lib'),
+            File.join('a','inner','lib'),
+            File.join('b','lib')
+          ].map do |p|
+            File.join(ports_host_root, p)
+          end
+        end
+
+        before do
+          RbConfig::CONFIG['host'] = 'fake-host-with-dirs'
+          fake_lib_paths.each do |path|
+            FileUtils.mkdir_p(path)
+          end
+        end
+
+        after do
+          FileUtils.remove_entry_secure(
+            File.join(gem_root, 'ports', 'fake-host-with-dirs'), true
+          )
+        end
+
+        it 'should return all the lib directories' do
+          _(ports_lib_paths.sort).must_equal fake_lib_paths.sort
+        end
+
+        it 'should return all the lib directories regardless of cwd' do
+          Dir.chdir '/'
+          _(ports_lib_paths.sort).must_equal fake_lib_paths.sort
+        end
+      end
+
+      describe 'when the ports directories are missing' do
+        before do
+          RbConfig::CONFIG['host'] = 'fake-host-without-dirs'
+        end
+
+
+        it 'should return no directories' do
+          _(ports_lib_paths).must_be_empty
+        end
+
+        it 'should return no directories regardless of cwd' do
+          Dir.chdir '/'
+          _(ports_lib_paths).must_be_empty
+        end
+      end
+    end
+
+    describe '#ports_host' do
+      {
+        'i686-pc-linux-gnu' => 'i686-pc-linux-gnu',
+        'x86_64-pc-linux-gnu' => 'x86_64-pc-linux-gnu',
+        'i686-w64-mingw32' => 'i686-w64-mingw32',
+        'x86_64-w64-mingw32' => 'x86_64-w64-mingw32',
+        # consolidate this host to our build w64-mingw32 arch
+        'i686-pc-mingw32' => 'i686-w64-mingw32'
+      }.each do |host,expected|
+        describe "on a #{host} architecture" do
+          before do
+            RbConfig::CONFIG['host'] = host
+          end
+
+          it "should return a #{expected} ports host" do
+            _(TinyTds::Gem.ports_host).must_equal expected
+          end
+        end
+      end
+    end
+  end
+end
+