Implemented routes in markdown, closes #178

Author: ryanfox1985

Diff for: .rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2016-12-15 16:54:50 +0100 using RuboCop version 0.46.0.
+# on 2016-12-17 01:21:04 +0100 using RuboCop version 0.46.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -94,14 +94,14 @@ Lint/UselessAccessModifier:
   Exclude:
     - 'lib/annotate/annotate_routes.rb'
 
-# Offense count: 16
+# Offense count: 17
 Metrics/AbcSize:
   Max: 144
 
 # Offense count: 3
 # Configuration parameters: CountComments.
 Metrics/BlockLength:
-  Max: 135
+  Max: 134
 
 # Offense count: 2
 Metrics/BlockNesting:
@@ -111,17 +111,22 @@ Metrics/BlockNesting:
 Metrics/CyclomaticComplexity:
   Max: 36
 
-# Offense count: 334
+# Offense count: 353
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Metrics/LineLength:
   Max: 543
 
-# Offense count: 24
+# Offense count: 25
 # Configuration parameters: CountComments.
 Metrics/MethodLength:
   Max: 70
 
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ModuleLength:
+  Max: 116
+
 # Offense count: 7
 Metrics/PerceivedComplexity:
   Max: 41
@@ -131,7 +136,7 @@ Style/AccessorMethodName:
   Exclude:
     - 'lib/annotate.rb'
 
-# Offense count: 1
+# Offense count: 3
 # Cop supports --auto-correct.
 Style/AlignArray:
   Exclude:
@@ -284,12 +289,13 @@ Style/ExtraSpacing:
     - 'spec/integration/rails_4.2.0/Gemfile'
     - 'spec/integration/rails_4.2.0/config.ru'
 
-# Offense count: 8
+# Offense count: 9
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 # SupportedStyles: format, sprintf, percent
 Style/FormatString:
   Exclude:
     - 'lib/annotate/annotate_models.rb'
+    - 'lib/annotate/annotate_routes.rb'
 
 # Offense count: 181
 # Cop supports --auto-correct.
@@ -307,7 +313,7 @@ Style/GuardClause:
     - 'lib/tasks/migrate.rake'
     - 'spec/integration/rails_2.3_with_bundler/config/boot.rb'
 
-# Offense count: 58
+# Offense count: 57
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles, UseHashRocketsWithSymbolValues, PreferHashRocketsForNonAlnumEndingSymbols.
 # SupportedStyles: ruby19, hash_rockets, no_mixed_keys, ruby19_no_mixed_keys
@@ -321,14 +327,13 @@ Style/IfUnlessModifier:
   Exclude:
     - 'lib/annotate/annotate_models.rb'
 
-# Offense count: 6
+# Offense count: 5
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
 # SupportedStyles: special_inside_parentheses, consistent, align_brackets
 Style/IndentArray:
   Exclude:
     - 'lib/annotate/annotate_models.rb'
-    - 'lib/annotate/annotate_routes.rb'
 
 # Offense count: 6
 # Cop supports --auto-correct.
@@ -514,7 +519,7 @@ Style/Semicolon:
     - 'bin/annotate'
     - 'spec/integration/rails_2.3_with_bundler/config/initializers/unified_initializer.rb'
 
-# Offense count: 13
+# Offense count: 12
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 # SupportedStyles: space, no_space
@@ -597,7 +602,7 @@ Style/SpaceInsideStringInterpolation:
   Exclude:
     - 'lib/annotate/annotate_models.rb'
 
-# Offense count: 252
+# Offense count: 253
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles, ConsistentQuotesInMultiline.
 # SupportedStyles: single_quotes, double_quotes
@@ -648,18 +653,18 @@ Style/TrailingCommaInLiteral:
     - 'spec/integration/rails_4.1.1/lib/tasks/auto_annotate_models.rake'
     - 'spec/integration/rails_4.2.0/lib/tasks/auto_annotate_models.rake'
 
-# Offense count: 1
+# Offense count: 2
 # Cop supports --auto-correct.
 Style/TrailingWhitespace:
   Exclude:
+    - 'spec/annotate/annotate_routes_spec.rb'
     - 'spec/integration/rails_2.3_with_bundler/db/schema.rb'
 
-# Offense count: 3
+# Offense count: 2
 # Cop supports --auto-correct.
 Style/UnneededInterpolation:
   Exclude:
     - 'bin/annotate'
-    - 'lib/annotate/annotate_routes.rb'
 
 # Offense count: 8
 # Cop supports --auto-correct.

Diff for: lib/annotate/annotate_routes.rb

@@ -18,33 +18,61 @@
 # Released under the same license as Ruby. No Support. No Warranty.
 #
 module AnnotateRoutes
-  PREFIX = '# == Route Map'
+  PREFIX = '== Route Map'.freeze
+  PREFIX_MD = '## Route Map'.freeze
+  HEADER_ROW = ['Prefix', 'Verb', 'URI Pattern', 'Controller#Action']
+
+  class << self
+    def content(line, maxs, options = {})
+      return line.rstrip unless options[:format_markdown]
+      # info<< "# #{ '-' * ( max_size + md_names_overhead ) } | #{'-' * md_type_allowance} | #{ '-' * 27 }\n"
+      # info<< sprintf( "# %-#{max_size + md_names_overhead}.#{max_size + md_names_overhead}s | %-#{md_type_allowance}.#{md_type_allowance}s | %s\n", 'Name', 'Type', 'Attributes' )
+      line.each_with_index.map do |elem, index|
+        min_length = maxs.map { |arr| arr[index] }.max || 0
+
+        sprintf("%-#{min_length}.#{min_length}s", elem)
+      end.join(' | ')
+    end
 
-  def self.do_annotations(options={})
-    return unless routes_exists?
+    def header(options = {})
+      routes_map = app_routes_map(options)
 
-    routes_map = AnnotateRoutes.app_routes_map(options)
+      out = ["# #{options[:format_markdown] ? PREFIX_MD : PREFIX}" + (options[:timestamp] ? " (Updated #{Time.now.strftime('%Y-%m-%d %H:%M')})" : '')]
+      out += ['#']
+      return out if routes_map.size.zero?
 
-    header = [
-        "#{PREFIX}" + (options[:timestamp] ? " (Updated #{Time.now.strftime('%Y-%m-%d %H:%M')})" : ''), '#'
-    ] + routes_map.map { |line| "# #{line}".rstrip }
+      maxs = [HEADER_ROW.map(&:size)] + routes_map[1..-1].map { |line| line.split.map(&:size) }
+      max = maxs.map(&:max).max
 
-    existing_text = File.read(routes_file)
+      if options[:format_markdown]
+        out += ["# #{content(HEADER_ROW, maxs, options)}"]
+        out += ["# #{content(['-' * max, '-' * max, '-' * max, '-' * max], maxs, options)}"]
+      else
+        out += ["# #{content(routes_map[0], maxs, options)}"]
+      end
 
-    if write_contents(existing_text, header, options)
-      puts "#{routes_file} annotated."
+      out + routes_map[1..-1].map { |line| "# #{content(options[:format_markdown] ? line.split(' ') : line, maxs, options)}" }
     end
-  end
 
-  def self.remove_annotations(options={})
-    return unless routes_exists?
-    existing_text = File.read(routes_file)
-    content, where_header_found = strip_annotations(existing_text)
+    def do_annotations(options = {})
+      return unless routes_exists?
+      existing_text = File.read(routes_file)
 
-    content = strip_on_removal(content, where_header_found)
+      if write_contents(existing_text, header(options), options)
+        puts "#{routes_file} annotated."
+      end
+    end
+
+    def remove_annotations(options={})
+      return unless routes_exists?
+      existing_text = File.read(routes_file)
+      content, where_header_found = strip_annotations(existing_text)
 
-    if write_contents(existing_text, content, options)
-      puts "Removed annotations from #{routes_file}."
+      content = strip_on_removal(content, where_header_found)
+
+      if write_contents(existing_text, content, options)
+        puts "Removed annotations from #{routes_file}."
+      end
     end
   end
 

Diff for: spec/annotate/annotate_routes_spec.rb

@@ -17,6 +17,47 @@ def mock_file(stubs = {})
     AnnotateRoutes.do_annotations
   end
 
+  describe 'Annotate#example' do
+    before(:each) do
+      expect(File).to receive(:exists?).with(ROUTE_FILE).and_return(true)
+
+      expect(File).to receive(:read).with(ROUTE_FILE).and_return("")
+      expect(AnnotateRoutes).to receive(:`).with('rake routes').and_return('                                      Prefix Verb       URI Pattern                                               Controller#Action
+                                   myaction1 GET        /url1(.:format)                                           mycontroller1#action
+                                   myaction2 POST       /url2(.:format)                                           mycontroller2#action
+                                   myaction3 DELETE|GET /url3(.:format)                                           mycontroller3#action')
+
+      expect(AnnotateRoutes).to receive(:puts).with(ANNOTATION_ADDED)
+    end
+
+    it 'annotate normal' do
+      expect(File).to receive(:open).with(ROUTE_FILE, 'wb').and_yield(mock_file)
+      expect(@mock_file).to receive(:puts).with("
+# == Route Map
+#
+#                                       Prefix Verb       URI Pattern                                               Controller#Action
+#                                    myaction1 GET        /url1(.:format)                                           mycontroller1#action
+#                                    myaction2 POST       /url2(.:format)                                           mycontroller2#action
+#                                    myaction3 DELETE|GET /url3(.:format)                                           mycontroller3#action\n")
+
+      AnnotateRoutes.do_annotations
+    end
+
+    it 'annotate markdown' do
+      expect(File).to receive(:open).with(ROUTE_FILE, 'wb').and_yield(mock_file)
+      expect(@mock_file).to receive(:puts).with("
+# ## Route Map
+#
+# Prefix    | Verb       | URI Pattern     | Controller#Action   
+# --------- | ---------- | --------------- | --------------------
+# myaction1 | GET        | /url1(.:format) | mycontroller1#action
+# myaction2 | POST       | /url2(.:format) | mycontroller2#action
+# myaction3 | DELETE|GET | /url3(.:format) | mycontroller3#action\n")
+
+      AnnotateRoutes.do_annotations(format_markdown: true)
+    end
+  end
+
   describe 'When adding' do
     before(:each) do
       expect(File).to receive(:exists?).with(ROUTE_FILE).and_return(true)