Extract some methods

stomar · stomar · commit 8a813c43f73a · 2018-05-09T21:32:08.000+02:00
diff --git a/Rakefile b/Rakefile
@@ -129,7 +129,7 @@ end
 desc "Run some tests on markdown files"
 task :check do
   require_relative "lib/linter"
-  Linter.new.check
+  Linter.new.run
 end
 
 task :ci => [:test, :build]
diff --git a/lib/linter.rb b/lib/linter.rb
@@ -10,6 +10,25 @@ class Linter
   attr_accessor :docs, :posts, :errors
 
   def initialize
+    @docs = []
+    @posts = []
+    @errors = Hash.new {|h, k| h[k] = [] }
+  end
+
+  # Reads files and runs all checks.
+  def run
+    print "Checking markdown files..."
+
+    load_files
+    check
+    report
+
+    exit(1)  if errors.any?
+  end
+
+  private
+
+  def load_files
     md_files = glob("**/*.md")
     skip_patterns = [/README.md/, %r{[^/]*/examples/}, %r{\A_includes/}]
 
@@ -19,13 +38,9 @@ def initialize
 
     @docs = md_files.map {|fn| Document.new(fn) }
     @posts = @docs.select {|doc| doc.filename =~ %r{/_posts/} }
-
-    @errors = Hash.new {|h, k| h[k] = [] }
   end
 
   def check
-    print "Checking markdown files..."
-
     docs.each do |doc|
       errors[doc] << "  missing lang variable"  if doc.lang_missing?
     end
@@ -34,16 +49,6 @@ def check
       errors[doc] << "  missing author variable"  if doc.author_missing?
       errors[doc] << "  date mismatch between filename and YAML front matter (UTC)"  if doc.date_mismatch?
     end
-
-    report
-
-    exit(1)  if errors.any?
-  end
-
-  private
-
-  def glob(pattern)
-    Pathname.glob(pattern).reject {|path| path.expand_path.to_s =~ %r{\A#{Regexp.escape(Bundler.bundle_path.to_s)}/} }.map(&:to_s)
   end
 
   def report
@@ -57,4 +62,8 @@ def report
       end
     end
   end
+
+  def glob(pattern)
+    Pathname.glob(pattern).reject {|path| path.expand_path.to_s =~ %r{\A#{Regexp.escape(Bundler.bundle_path.to_s)}/} }.map(&:to_s)
+  end
 end
