Author: kclark
Date: Tue Jun 17 18:04:03 2008
New Revision: 668937

URL: http://svn.apache.org/viewvc?rev=668937&view=rev
Log:
Fix name usage in tests. Remove test for TTransport as it really wasn't testing 
anything useful.

Added:
    incubator/thrift/trunk/test/rb/core/transport/test_transport.rb
      - copied, changed from r668936, 
incubator/thrift/trunk/test/rb/core/transport/test_ttransport.rb
Removed:
    incubator/thrift/trunk/test/rb/core/transport/test_ttransport.rb
Modified:
    incubator/thrift/trunk/test/rb/core/test_backwards_compatability.rb

Modified: incubator/thrift/trunk/test/rb/core/test_backwards_compatability.rb
URL: 
http://svn.apache.org/viewvc/incubator/thrift/trunk/test/rb/core/test_backwards_compatability.rb?rev=668937&r1=668936&r2=668937&view=diff
==============================================================================
--- incubator/thrift/trunk/test/rb/core/test_backwards_compatability.rb 
(original)
+++ incubator/thrift/trunk/test/rb/core/test_backwards_compatability.rb Tue Jun 
17 18:04:03 2008
@@ -2,7 +2,7 @@
 
 require 'thrift/thrift'
 
-class TestTException < Test::Unit::TestCase
+class TestThriftException < Test::Unit::TestCase
   def test_has_accessible_message
     msg = "hi there thrift"
     assert_equal msg, Thrift::Exception.new(msg).message

Copied: incubator/thrift/trunk/test/rb/core/transport/test_transport.rb (from 
r668936, incubator/thrift/trunk/test/rb/core/transport/test_ttransport.rb)
URL: 
http://svn.apache.org/viewvc/incubator/thrift/trunk/test/rb/core/transport/test_transport.rb?p2=incubator/thrift/trunk/test/rb/core/transport/test_transport.rb&p1=incubator/thrift/trunk/test/rb/core/transport/test_ttransport.rb&r1=668936&r2=668937&rev=668937&view=diff
==============================================================================
--- incubator/thrift/trunk/test/rb/core/transport/test_ttransport.rb (original)
+++ incubator/thrift/trunk/test/rb/core/transport/test_transport.rb Tue Jun 17 
18:04:03 2008
@@ -1,8 +1,8 @@
 require File.join(File.dirname(__FILE__), '../../test_helper')
 
-require 'thrift/transport/ttransport'
+require 'thrift/transport'
 
-class DummyTransport < TTransport
+class DummyTransport < Thrift::Transport
   def initialize(data)
     @data = data
   end
@@ -13,9 +13,9 @@
 end
 
 # TTransport is basically an abstract class, but isn't raising 
NotImplementedError
-class TestTTransport < Test::Unit::TestCase
+class TestThriftTransport < Test::Unit::TestCase
   def setup
-    @trans = TTransport.new
+    @trans = Thrift::Transport.new
   end
   
   def test_open?
@@ -53,19 +53,3 @@
     assert_nil @trans.flush
   end
 end
-
-class TestTTransportDeprecation < Test::Unit::TestCase
-  def setup
-    @trans = TTransport.new
-  end
-  
-  def test_open?
-    assert_nil @trans.open?
-  end
-    
-  def test_readAll
-    # Implements read
-    t = DummyTransport.new("hello")
-    assert_equal "hello", t.read_all(5)
-  end
-end
\ No newline at end of file


Reply via email to