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/