From 8e9b826acfe47b049b3376ea1b2b9fd51c60e2e0 Mon Sep 17 00:00:00 2001
From: "Abinoam P. Marques Jr" <abinoam@gmail.com>
Date: Fri, 7 Jul 2017 05:24:23 -0300
Subject: Use guard clauses (rubocop warning)

---
 test/test_highline.rb         |  8 +++-----
 test/test_string_extension.rb | 22 +++++++++++-----------
 2 files changed, 14 insertions(+), 16 deletions(-)

(limited to 'test')

diff --git a/test/test_highline.rb b/test/test_highline.rb
index 94ab060..e215986 100755
--- a/test/test_highline.rb
+++ b/test/test_highline.rb
@@ -1168,11 +1168,9 @@ class TestHighLine < Minitest::Test
 
   class NameClass
     def self.parse(string)
-      if string =~ /^\s*(\w+),\s*(\w+)\s+(\w+)\s*$/
-        new(Regexp.last_match(2), Regexp.last_match(3), Regexp.last_match(1))
-      else
-        raise ArgumentError, "Invalid name format."
-      end
+      raise ArgumentError, "Invalid name format." unless
+        string =~ /^\s*(\w+),\s*(\w+)\s+(\w+)\s*$/
+      new(Regexp.last_match(2), Regexp.last_match(3), Regexp.last_match(1))
     end
 
     def initialize(first, middle, last)
diff --git a/test/test_string_extension.rb b/test/test_string_extension.rb
index 8f97dbe..44b514e 100644
--- a/test/test_string_extension.rb
+++ b/test/test_string_extension.rb
@@ -35,18 +35,18 @@ class TestStringExtension < Minitest::Test
 
   def test_Highline_String_is_yaml_serializable
     require "yaml"
-    unless Gem::Version.new(YAML::VERSION) < Gem::Version.new("2.0.2")
-      highline_string =
-        HighLine::String.new("Yaml didn't messed with HighLine::String")
-      yaml_highline_string = highline_string.to_yaml
-      yaml_loaded_string =
-        YAML.safe_load(yaml_highline_string, [HighLine::String])
+    return if Gem::Version.new(YAML::VERSION) < Gem::Version.new("2.0.2")
 
-      assert_equal "Yaml didn't messed with HighLine::String",
-                   yaml_loaded_string
-      assert_equal highline_string, yaml_loaded_string
-      assert_instance_of HighLine::String, yaml_loaded_string
-    end
+    highline_string =
+      HighLine::String.new("Yaml didn't messed with HighLine::String")
+    yaml_highline_string = highline_string.to_yaml
+    yaml_loaded_string =
+      YAML.safe_load(yaml_highline_string, [HighLine::String])
+
+    assert_equal "Yaml didn't messed with HighLine::String",
+                 yaml_loaded_string
+    assert_equal highline_string, yaml_loaded_string
+    assert_instance_of HighLine::String, yaml_loaded_string
   end
 
   def test_highline_string_respond_to_color
-- 
cgit v1.2.1