tailor 1.0.1 → 1.1.0

data/{features/support/file_cases/indentation_cases.rb → spec/support/good_indentation_cases.rb}

@@ -283,6 +283,7 @@ when :input
   "Gimme \#{eval('gets')}!"
 end}
 
+=begin
 INDENT_OK[:case_whens_in] =
   %Q{def my_method
   case true
@@ -292,6 +293,7 @@ INDENT_OK[:case_whens_in] =
       puts "blah blah"
   end
 end}
+=end
 
 INDENT_OK[:while_do_loop] =
   %Q{while true do
@@ -610,10 +612,12 @@ INDENT_OK[:multi_line_method_call_ends_with_many_periods] =
   second_level.
   third_level}
 
+=begin
 INDENT_OK[:method_closing_lonely_paren] =
   %Q{def your_thing(one
   )
 end}
+=end
 
 INDENT_OK[:method_lonely_args] =
   %Q{def your_thing(
@@ -678,11 +682,13 @@ INDENT_OK[:block_in_block_ends_on_same_line] =
 
 puts "post ends"}
 
+=begin
 INDENT_OK[:rparen_and_do_same_line] =
   %Q{opt.on('-c', '--config-file FILE',
   "Use a specific config file.") do |config|
   options.config_file = config
 end}
+=end
 
 #-------------------------------------------------------------------------------
 # Single-line keywords
@@ -755,269 +761,3 @@ INDENT_OK[:line_ends_with_label] =
   actual_trailing_spaces:
     lexed_line.last_non_line_feed_event.last.size
 }}
-
-#-------------------------------------------------------------------------------
-# INDENT_1 (1 problem)
-#-------------------------------------------------------------------------------
-INDENT_1 = {}
-
-INDENT_1[:class_indented_end] =
-  %Q{class MyClass
- end}
-
-INDENT_1[:class_indented_singlestatement] =
-  %Q{class MyClass
-   include Something
-end}
-
-INDENT_1[:class_indented_singlestatement_trailing_comment] =
-  %Q{class MyClass
-   include Something     # comment
-end}
-
-INDENT_1[:class_outdented_singlestatement] =
-  %Q{class MyClass
- include Something
-end}
-
-INDENT_1[:def_indented_end] =
-  %Q{def a
- end}
-
-INDENT_1[:def_content_indented_end] =
-  %Q{def a
-  puts "stuff"
- end}
-
-INDENT_1[:class_def_content_outdented_end] =
-  %Q{class A
-  def a
-    puts "stuff"
- end
-end}
-
-INDENT_1[:class_def_outdented_content] =
-  %Q{class A
-  def a
-   puts "stuff"
-  end
-end}
-
-INDENT_1[:class_method_def_using_self_outdented] =
-  %Q{class A
- self.my_method
-    puts 'stuff'
-  end
-end}
-
-INDENT_1[:case_indented_whens_level] =
-  %Q{def my_method
-   case true
-  when true
-    puts "stuff"
-  when false
-    puts "blah blah"
-  end
-end}
-
-INDENT_1[:case_indented_whens_level_trailing_comment] =
-  %Q{def my_method
-   case true        # comment
-  when true
-    puts "stuff"
-  when false
-    puts "blah blah"
-  end
-end}
-
-INDENT_1[:case_outdented_whens_level] =
-  %Q{def my_method
- case true
-  when true
-    puts "stuff"
-  when false
-    puts "blah blah"
-  end
-end}
-
-INDENT_1[:case_when_indented_whens_level] =
-  %Q{def my_method
-  case true
-   when true
-    puts "stuff"
-  when false
-    puts "blah blah"
-  end
-end}
-
-INDENT_1[:case_when_outdented_whens_level] =
-  %Q{def my_method
-  case true
- when true
-    puts "stuff"
-  when false
-    puts "blah blah"
-  end
-end}
-
-INDENT_1[:case_indented_whens_in] =
-  %Q{def my_method
-   case true
-    when true
-      puts "stuff"
-    when false
-      puts "blah blah"
-  end
-end}
-
-INDENT_1[:while_do_indented] =
-  %Q{ while true do
-  puts "something"
-end}
-
-INDENT_1[:while_do_outdented] =
-  %Q{def my_method
- while true do
-    puts "something"
-  end
-end}
-
-INDENT_1[:while_do_content_outdented] =
-  %Q{def my_method
-  while true do
-   puts "something"
-  end
-end}
-
-INDENT_1[:while_do_content_indented] =
-  %Q{def my_method
-  while true do
-     puts "something"
-  end
-end}
-
-INDENT_1[:while_do_indented2] =
-  %Q{i = 0;
-num = 5;
-
- while i < num do
-  puts("Inside the loop i = \#{i}" );
-  i +=1;
-end}
-
-INDENT_1[:while_do_indented2_trailing_comment] =
-  %Q{i = 0;
-num = 5;
-
- while i < num do        # comment
-  puts("Inside the loop i = \#{i}" );
-  i +=1;
-end}
-
-INDENT_1[:until_do_indented] =
-  %Q{i = 0;
-num = 5;
-
- until i > num  do
-  puts("Inside the loop i = \#{i}" );
-  i +=1;
-end}
-
-INDENT_1[:for_do_indented] =
-  %Q{ for i in 1..100 do
-  puts i
-end}
-
-INDENT_1[:loop_do_indented] =
-  %Q{ loop do
-  puts 'looping'
-end}
-
-INDENT_1[:multi_line_string_first_line_indented] =
-  %Q{def a_method
-   if defined? Term::ANSIColor
-    message << %Q{#  \#{(i + 1).to_s.bold}.
-#    * position:  \#{position}
-#    * type:      \#{problem[:type].to_s.red}
-#    * message:   \#{problem[:message].red}
-}
-  else
-    message << %Q{#  \#{(i + 1)}.
-#    * position:  \#{position}
-#    * type:      \#{problem[:type]}
-#    * message:   \#{problem[:message]}
-}
-  end
-end}
-
-INDENT_1[:multi_line_string_first_line_indented_trailing_comment] =
-  %Q{def a_method
-   if defined? Term::ANSIColor     # comment
-    message << %Q{#  \#{(i + 1).to_s.bold}.
-#    * position:  \#{position}
-#    * type:      \#{problem[:type].to_s.red}
-#    * message:   \#{problem[:message].red}
-}
-  else
-    message << %Q{#  \#{(i + 1)}.
-#    * position:  \#{position}
-#    * type:      \#{problem[:type]}
-#    * message:   \#{problem[:message]}
-}
-  end
-end}
-
-INDENT_1[:multi_line_tstring] =
-  %Q{INDENT_OK[:class] =
-%Q{class MyClass
-end}}
-
-#-------------------------------------------------------------------------------
-# Operators
-#-------------------------------------------------------------------------------
-INDENT_1[:multi_line_andop_first_line_indented] =
-  %Q{def end_of_multiline_string?(lexed_line_output)
-   lexed_line_output.any? { |e| e[1] == :on_tstring_end } &&
-    lexed_line_output.none? { |e| e[1] == :on_tstring_beg }
-end}
-
-INDENT_1[:multi_line_andop_first_line_indented_trailing_comment] =
-  %Q{def end_of_multiline_string?(lexed_line_output)
-   lexed_line_output.any? { |e| e[1] == :on_tstring_end } && # comment
-    lexed_line_output.none? { |e| e[1] == :on_tstring_beg }
-end}
-
-INDENT_1[:multi_line_andop_second_line_indented] =
-  %Q{def end_of_multiline_string?(lexed_line_output)
-  lexed_line_output.any? { |e| e[1] == :on_tstring_end } &&
-     lexed_line_output.none? { |e| e[1] == :on_tstring_beg }
-end}
-
-INDENT_1[:multi_line_string_concat_with_plus_out] =
-  %Q{DVR_SSDP_NOTIFICATION_TEMPLATE = File.dirname(__FILE__) +
- '/profiles/DVR5000/ssdp_notification.erb'}
-
-INDENT_1[:multi_line_method_call_end_in] =
-  %Q{def initialize(raw_response)
-  if raw_response.nil? || raw_response.empty?
-    raise RTSP::Error,
-      "#{self.class} received nil string--this shouldn't happen."
-   end
-end}
-
-INDENT_1[:multi_line_method_call_ends_with_period_2nd_line_in] =
-  %Q{unless streamer == MulticastStreamer.instance
-  streamer.state = :DISCONNECTED
-  UnicastStreamer.pool << streamer unless UnicastStreamer.pool.
-     member? streamer
-end}
-
-INDENT_1[:multi_line_method_call_ends_with_many_periods_last_in] =
-  %Q{my_hashie.first_level.
-  second_level.
-    third_level}
-
-INDENT_1[:multi_line_method_call_ends_with_many_periods_last_in_trailing_comment] =
-  %Q{my_hashie.first_level.
-  second_level.
-    third_level  # comment}

data/spec/support/horizontal_spacing_cases.rb

@@ -0,0 +1,136 @@
+H_SPACING_OK = {}
+
+H_SPACING_OK[:short_line_no_newline] = '#' * 79
+H_SPACING_OK[:short_line_newline_at_81] =
+  %Q{'#{'#' * 78}'
+}
+
+=begin
+H_SPACING_OK[:line_split_by_backslash] =
+  %Q{execute 'myscript' do
+  command \\
+    '/some/really/long/path/that/would/be/over/eight/chars.sh'
+  only_if { something }
+end}
+=end
+
+#-------------------------------------------------------------------------------
+# Comma spacing
+#-------------------------------------------------------------------------------
+H_SPACING_OK[:space_after_comma_in_array] = %Q{[1, 2]}
+
+H_SPACING_OK[:trailing_comma] = %Q{def thing(one, two,
+  three)
+end}
+
+H_SPACING_OK[:trailing_comma_with_trailing_comment] =
+  %Q{def thing(one, two,  # Comment!
+  three)
+end}
+
+H_SPACING_OK[:no_before_comma_in_array] = %Q{[1, 2]}
+H_SPACING_OK[:line_ends_with_backslash] =
+  %Q{{ :thing => a_thing,\\
+  :thing2 => another_thing }}
+
+#-------------------------------------------------------------------------------
+# Braces
+#-------------------------------------------------------------------------------
+H_SPACING_OK[:empty_hash] = %Q{{}}
+H_SPACING_OK[:single_line_hash] = %Q{{ :one => 'one' }}
+H_SPACING_OK[:single_line_hash_lonely_braces] = %Q{{
+  :one => 'one'
+}}
+
+H_SPACING_OK[:hash_as_param_in_parens] =
+  %Q{add_headers({ content_length: new_body.length })}
+
+H_SPACING_OK[:two_line_hash] = %Q{{ :one =>
+  'one' }}
+
+H_SPACING_OK[:two_line_hash_trailing_comment] = %Q{{ :one =>    # comment
+  'one' }}
+
+H_SPACING_OK[:three_line_hash] = %Q{{ :one =>
+  'one', :two =>
+  'two' }}
+
+H_SPACING_OK[:single_line_block] = %Q{1..10.times { |n| puts number }}
+H_SPACING_OK[:multi_line_braces_block] = %Q{1..10.times { |n|
+  puts number }}
+
+H_SPACING_OK[:multi_line_qword_using_braces] = %Q{%w{
+  foo
+  bar
+  baz
+}.each do |whatevs|
+  bla
+end}
+
+H_SPACING_OK[:empty_hash_in_multi_line_statement] =
+  %Q{if true
+  {}
+end}
+
+H_SPACING_OK[:multi_line_hash_in_multi_line_statement] =
+  %Q{if true
+  options = {
+    one: 1
+  }
+end}
+
+H_SPACING_OK[:single_line_string_interp] = %Q{`\#{IFCONFIG} | grep \#{ip}`}
+H_SPACING_OK[:single_line_block_in_string_interp] =
+  %Q{"I did this \#{1..10.times { |n| n }} times."}
+
+H_SPACING_OK[:empty_hash_in_string_in_block] =
+  %Q{[1].map { |n| { :first => "\#{n}-\#{{}}" } }}
+
+H_SPACING_OK[:string_interp_with_colonop] =
+  %Q{"\#{::Rails.root}"}
+
+
+
+#-------------------------------------------------------------------------------
+# Brackets
+#-------------------------------------------------------------------------------
+H_SPACING_OK[:empty_array] = %Q{[]}
+H_SPACING_OK[:simple_array] = %Q{[1, 2, 3]}
+H_SPACING_OK[:two_d_array] = %Q{[[1, 2, 3], ['a', 'b', 'c']]}
+H_SPACING_OK[:hash_key_reference] = %Q{thing[:one]}
+H_SPACING_OK[:array_of_symbols] =
+  %Q{transition [:active, :reactivated] => :opened}
+H_SPACING_OK[:array_of_hashes] =
+  %Q{[ { :one => [[1, 2, 3], ['a', 'b', 'c']] },
+  { :two => [[4, 5, 6], ['d', 'e', 'f']] }]}
+
+H_SPACING_OK[:simple_array_lonely_brackets] =
+  %Q{[
+  1, 2,
+  3
+]}
+
+H_SPACING_OK[:simple_nested_array_lonely_brackets] =
+  %Q{def thing
+  [
+    1, 2,
+    3
+  ]
+end}
+
+
+H_SPACING_OK[:empty_array_in_multi_line_statement] =
+  %Q{if true
+  []
+end}
+
+#-------------------------------------------------------------------------------
+# Parens
+#-------------------------------------------------------------------------------
+H_SPACING_OK[:empty_parens] = %Q{def thing(); end}
+H_SPACING_OK[:simple_method_call] = %Q{thing(one, two)}
+H_SPACING_OK[:multi_line_method_call] = %Q{thing(one,
+  two)}
+H_SPACING_OK[:multi_line_method_call_lonely_parens] = %Q{thing(
+  one, two
+)}

data/spec/support/naming_cases.rb

@@ -0,0 +1,26 @@
+NAMING_OK = {}
+
+NAMING_OK[:single_word_method] =
+  %Q{def thing
+end}
+
+NAMING_OK[:two_word_method] =
+  %Q{def thing_one
+end}
+
+#-------------------------------------------------------------------------------
+NAMING_OK[:single_word_class] =
+  %Q{class Thing
+end}
+
+NAMING_OK[:single_word_module] =
+  %Q{module Thing
+end}
+
+NAMING_OK[:two_word_class] =
+  %Q{class ThingOne
+end}
+
+NAMING_OK[:two_word_module] =
+  %Q{module ThingOne
+end}