Suggest global variables

Author: styd

lib/did_you_mean/spell_checkers/name_error_checkers/variable_name_checker.rb

@@ -2,14 +2,15 @@
 
 module DidYouMean
   class VariableNameChecker
-    attr_reader :name, :method_names, :lvar_names, :ivar_names, :cvar_names
+    attr_reader :name, :method_names, :gvar_names, :lvar_names, :ivar_names, :cvar_names
 
     NAMES_TO_EXCLUDE = { 'foo' => [:fork] }
     NAMES_TO_EXCLUDE.default = []
     RB_PREDEFINED_OBJECTS = [:false, :true, :nil]
 
     def initialize(exception)
       @name       = exception.name.to_s.tr("@", "")
+      @gvar_names = global_variables.select {|g| g[/\$[A-Za-z]/] }
       @lvar_names = exception.respond_to?(:local_variables) ? exception.local_variables : []
       receiver    = exception.receiver
 
@@ -21,7 +22,7 @@ def initialize(exception)
 
     def corrections
       @corrections ||= SpellChecker
-                     .new(dictionary: (RB_PREDEFINED_OBJECTS + lvar_names + method_names + ivar_names + cvar_names))
+                     .new(dictionary: (RB_PREDEFINED_OBJECTS + gvar_names + lvar_names + method_names + ivar_names + cvar_names))
                      .correct(name) - NAMES_TO_EXCLUDE[@name]
     end
   end

test/spell_checking/variable_name_check_test.rb

@@ -49,6 +49,15 @@ def test_corrections_include_method_from_module
     assert_match "Did you mean?  from_module", error.to_s
   end
 
+  def test_corrections_include_global_variable_name
+    error = assert_raises(NameError) do
+      stin
+    end
+
+    assert_correction :$stdin, error.corrections
+    assert_match "Did you mean?  $stdin", error.to_s
+  end
+
   def test_corrections_include_local_variable_name
     person = person = nil
     error = (eprson rescue $!) # Do not use @assert_raises here as it changes a scope.