[前][次][番号順一覧][スレッド一覧]

ruby-changes:67491

From: Daniel <ko1@a...>
Date: Tue, 31 Aug 2021 19:08:13 +0900 (JST)
Subject: [ruby-changes:67491] ceba866864 (master): [rubygems/rubygems] Remove unnecessary tests

https://git.ruby-lang.org/ruby.git/commit/?id=ceba866864

From ceba8668642ae9567c0952724217073462fee9a8 Mon Sep 17 00:00:00 2001
From: Daniel Niknam <mhmd.niknam@g...>
Date: Tue, 24 Aug 2021 01:01:11 +1000
Subject: [rubygems/rubygems] Remove unnecessary tests

We will be passing string to this class so there is no need to be defensive about it.

https://github.com/rubygems/rubygems/commit/30bd52a977
---
 test/rubygems/test_gem_printable_uri.rb | 36 ---------------------------------
 1 file changed, 36 deletions(-)

diff --git a/test/rubygems/test_gem_printable_uri.rb b/test/rubygems/test_gem_printable_uri.rb
index 9f49d65..6d4be6e 100644
--- a/test/rubygems/test_gem_printable_uri.rb
+++ b/test/rubygems/test_gem_printable_uri.rb
@@ -1,5 +1,4 @@ https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_printable_uri.rb#L1
 require_relative 'helper'
-require 'uri'
 require 'rubygems/printable_uri'
 
 class TestPrintableUri < Gem::TestCase
@@ -7,18 +6,6 @@ class TestPrintableUri < Gem::TestCase https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_printable_uri.rb#L6
     assert_equal true, Gem::PrintableUri.parse_uri("https://www.example.com").valid_uri?
   end
 
-  def test_valid_uri_with_empty_uri_object
-    assert_equal true, Gem::PrintableUri.parse_uri(URI("")).valid_uri?
-  end
-
-  def test_valid_uri_with_valid_uri_object
-    assert_equal true, Gem::PrintableUri.parse_uri(URI("https://www.example.com")).valid_uri?
-  end
-
-  def test_valid_uri_with_other_objects
-    assert_equal false, Gem::PrintableUri.parse_uri(Object.new).valid_uri?
-  end
-
   def test_valid_uri_with_invalid_uri
     assert_equal false, Gem::PrintableUri.parse_uri("https://www.example.com:80index").valid_uri?
   end
@@ -35,14 +22,6 @@ class TestPrintableUri < Gem::TestCase https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_printable_uri.rb#L22
     assert_equal nil, Gem::PrintableUri.parse_uri("https://www.example.com").original_password
   end
 
-  def test_original_password_with_empty_uri_object
-    assert_equal nil, Gem::PrintableUri.parse_uri(URI("")).original_password
-  end
-
-  def test_original_password_with_valid_uri_object
-    assert_equal "pass", Gem::PrintableUri.parse_uri(URI("https://user:pass@e...")).original_password
-  end
-
   def test_to_s_with_user_pass
     assert_equal "https://user:REDACTED@e...", Gem::PrintableUri.parse_uri("https://user:pass@e...").to_s
   end
@@ -62,19 +41,4 @@ class TestPrintableUri < Gem::TestCase https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_printable_uri.rb#L41
   def test_to_s_with_invalid_uri
     assert_equal "https://www.example.com:80index", Gem::PrintableUri.parse_uri("https://www.example.com:80index").to_s
   end
-
-  def test_to_s_with_empty_uri_object
-    assert_equal "", Gem::PrintableUri.parse_uri(URI("")).to_s
-  end
-
-  def test_to_s_with_valid_uri_object
-    assert_equal "https://user:REDACTED@e...", Gem::PrintableUri.parse_uri(URI("https://user:pass@e...")).to_s
-  end
-
-  def test_to_s_with_other_objects
-    obj = Object.new
-    obj.stub(:to_s, "my-to-s") do
-      assert_equal "my-to-s", Gem::PrintableUri.parse_uri(obj).to_s
-    end
-  end
 end
-- 
cgit v1.1


--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]