Merge pull request #145 from ManageIQ/detailed_database_version

Add methods for sqlserver's productversion, productlevel, edition and include them in inspect.

Author: metaskills

lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -323,8 +323,20 @@ def version
         self.class::VERSION
       end
 
+      def product_level
+        @product_level ||= info_schema_query { select_value("SELECT CAST(SERVERPROPERTY('productlevel') AS VARCHAR(128))")}
+      end
+      
+      def product_version
+        @product_version ||= info_schema_query { select_value("SELECT CAST(SERVERPROPERTY('productversion') AS VARCHAR(128))")}
+      end
+      
+      def edition
+        @edition ||= info_schema_query { select_value("SELECT CAST(SERVERPROPERTY('edition') AS VARCHAR(128))")}
+      end
+      
       def inspect
-        "#<#{self.class} version: #{version}, year: #{@database_year}, connection_options: #{@connection_options.inspect}>"
+        "#<#{self.class} version: #{version}, year: #{@database_year}, product_level: #{product_level.inspect}, product_version: #{product_version.inspect}, edition: #{edition.inspect}, connection_options: #{@connection_options.inspect}>"
       end
 
       def auto_connect

test/cases/adapter_test_sqlserver.rb

@@ -41,6 +41,30 @@ def setup
       assert_match(/version\: \d.\d/,@connection.inspect)
     end
 
+    should 'include database product level in inspect' do
+      assert_match(/product_level\: "\w+/, @connection.inspect)
+    end
+    
+    should 'include database product version in inspect' do
+      assert_match(/product_version\: "\d+/, @connection.inspect)
+    end
+    
+    should 'include database edition in inspect' do
+      assert_match(/edition\: "\w+/, @connection.inspect)
+    end
+    
+    should 'set database product level' do
+      assert_match(/\w+/, @connection.product_level)
+    end
+    
+    should 'set database product version' do
+      assert_match(/\d+/, @connection.product_version)
+    end
+    
+    should 'set database edition' do
+      assert_match(/\w+/, @connection.edition)
+    end
+    
     should 'support migrations' do
       assert @connection.supports_migrations?
     end