🔀 Merge pull request #515 from ruby/organize-tests

✅ Organize test files

Author: nevans

test/lib/helper.rb

@@ -34,7 +34,21 @@
 
 Test::Unit::TestCase.include Test::Unit::CoreAssertions
 
-class Test::Unit::TestCase
+require "net/imap"
+class Net::IMAP::TestCase < Test::Unit::TestCase
+  def setup
+    Net::IMAP.config.reset
+    @do_not_reverse_lookup = Socket.do_not_reverse_lookup
+    Socket.do_not_reverse_lookup = true
+    @threads = []
+  end
+
+  def teardown
+    assert_join_threads(@threads) unless @threads.empty?
+  ensure
+    Socket.do_not_reverse_lookup = @do_not_reverse_lookup
+  end
+
   def wait_for_response_count(imap, type:, count:,
                               timeout: 0.5, interval: 0.001)
     deadline = Time.now + timeout

test/net/imap/test_imap_authenticators.rb test/net/imap/test_authenticators.rbtest/net/imap/test_imap_authenticators.rb renamed to test/net/imap/test_authenticators.rb

@@ -3,7 +3,7 @@
 require "net/imap"
 require "test/unit"
 
-class IMAPAuthenticatorsTest < Test::Unit::TestCase
+class AuthenticatorsTest < Net::IMAP::TestCase
 
   test "SASL::Authenticators.normalize_name" do
     authenticators = Net::IMAP::SASL::Authenticators

test/net/imap/test_config.rb

@@ -3,7 +3,7 @@
 require "net/imap"
 require "test/unit"
 
-class ConfigTest < Test::Unit::TestCase
+class ConfigTest < Net::IMAP::TestCase
   Config = Net::IMAP::Config
   THIS_VERSION   = Net::IMAP::VERSION.to_f
   NEXT_VERSION   = THIS_VERSION + 0.1

test/net/imap/test_connection_state.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require "net/imap"
+require "test/unit"
+
+class ConnectionStateTest < Net::IMAP::TestCase
+  NotAuthenticated = Net::IMAP::ConnectionState::NotAuthenticated
+  Authenticated    = Net::IMAP::ConnectionState::Authenticated
+  Selected         = Net::IMAP::ConnectionState::Selected
+  Logout           = Net::IMAP::ConnectionState::Logout
+
+  test "#name" do
+    assert_equal "not_authenticated", NotAuthenticated[].name
+    assert_equal "authenticated",     Authenticated[]   .name
+    assert_equal "selected",          Selected[]        .name
+    assert_equal "logout",            Logout[]          .name
+  end
+
+
+  test "#to_sym" do
+    assert_equal :not_authenticated, NotAuthenticated[].to_sym
+    assert_equal :authenticated,     Authenticated[]   .to_sym
+    assert_equal :selected,          Selected[]        .to_sym
+    assert_equal :logout,            Logout[]          .to_sym
+  end
+
+  test "#deconstruct" do
+    assert_equal [:not_authenticated], NotAuthenticated[].deconstruct
+    assert_equal [:authenticated],     Authenticated[]   .deconstruct
+    assert_equal [:selected],          Selected[]        .deconstruct
+    assert_equal [:logout],            Logout[]          .deconstruct
+  end
+
+  test "#deconstruct_keys" do
+    assert_equal({symbol: :not_authenticated}, NotAuthenticated[].deconstruct_keys([:symbol]))
+    assert_equal({symbol: :authenticated},     Authenticated[]   .deconstruct_keys([:symbol]))
+    assert_equal({symbol: :selected},          Selected[]        .deconstruct_keys([:symbol]))
+    assert_equal({symbol: :logout},            Logout[]          .deconstruct_keys([:symbol]))
+    assert_equal({name: "not_authenticated"},  NotAuthenticated[].deconstruct_keys([:name]))
+    assert_equal({name: "authenticated"},      Authenticated[]   .deconstruct_keys([:name]))
+    assert_equal({name: "selected"},           Selected[]        .deconstruct_keys([:name]))
+    assert_equal({name: "logout"},             Logout[]          .deconstruct_keys([:name]))
+  end
+
+  test "#not_authenticated?" do
+    assert_equal true,  NotAuthenticated[].not_authenticated?
+    assert_equal false, Authenticated[]   .not_authenticated?
+    assert_equal false, Selected[]        .not_authenticated?
+    assert_equal false, Logout[]          .not_authenticated?
+  end
+
+  test "#authenticated?" do
+    assert_equal false, NotAuthenticated[].authenticated?
+    assert_equal true,  Authenticated[]   .authenticated?
+    assert_equal false, Selected[]        .authenticated?
+    assert_equal false, Logout[]          .authenticated?
+  end
+
+  test "#selected?" do
+    assert_equal false, NotAuthenticated[].selected?
+    assert_equal false, Authenticated[]   .selected?
+    assert_equal true,  Selected[]        .selected?
+    assert_equal false, Logout[]          .selected?
+  end
+
+  test "#logout?" do
+    assert_equal false, NotAuthenticated[].logout?
+    assert_equal false, Authenticated[]   .logout?
+    assert_equal false, Selected[]        .logout?
+    assert_equal true,  Logout[]          .logout?
+  end
+
+end
+
+

test/net/imap/test_data_lite.rb

@@ -29,7 +29,7 @@
 
 module Net
   class IMAP
-    class TestData < Test::Unit::TestCase
+    class TestData < Net::IMAP::TestCase
       def test_define
         klass = Data.define(:foo, :bar)
         assert_kind_of(Class, klass)

test/net/imap/test_errors.rb

@@ -3,7 +3,7 @@
 require "net/imap"
 require "test/unit"
 
-class IMAPErrorsTest < Test::Unit::TestCase
+class IMAPErrorsTest < Net::IMAP::TestCase
 
   test "ResponseTooLargeError" do
     err = Net::IMAP::ResponseTooLargeError.new

test/net/imap/test_esearch_result.rb

@@ -3,7 +3,7 @@
 require "net/imap"
 require "test/unit"
 
-class ESearchResultTest < Test::Unit::TestCase
+class ESearchResultTest < Net::IMAP::TestCase
   ESearchResult = Net::IMAP::ESearchResult
   SequenceSet   = Net::IMAP::SequenceSet
   ExtensionData = Net::IMAP::ExtensionData

test/net/imap/test_fetch_data.rb

@@ -3,7 +3,7 @@
 require "net/imap"
 require "test/unit"
 
-class FetchDataTest < Test::Unit::TestCase
+class FetchDataTest < Net::IMAP::TestCase
   def fetch_data_class
     Net::IMAP::FetchData
   end
@@ -19,7 +19,7 @@ def fetch_data_class
   end
 end
 
-class UIDFetchDataTest < Test::Unit::TestCase
+class UIDFetchDataTest < Net::IMAP::TestCase
   def fetch_data_class
     Net::IMAP::UIDFetchData
   end