Rewrite Linter test for a valid page

stomar · stomar · commit 883cbcc26194 · 2021-01-04T20:54:12.000+01:00
This starts moving away from fixture based tests towards
tests that are completely defined in the test files.

Also add a test helper for the Linter tests.
diff --git a/Rakefile b/Rakefile
@@ -133,6 +133,6 @@ require "rake/testtask"
 Rake::TestTask.new(:"test-linter") do |t|
   t.description = "Run tests for the Linter library"
   t.libs = ["test", "lib"]
-  t.test_files = ["test/test_linter.rb"]
+  t.test_files = ["test/test_linter.rb"] + FileList['test/test_linter_*.rb']
   t.verbose = true
 end
diff --git a/test/helper.rb b/test/helper.rb
@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+require "minitest/autorun"
+require "fileutils"
+require "pathname"
+
+
+TEST_DIR = File.expand_path(__dir__)
+TEMP_DIR = File.join(TEST_DIR, "tmp")
+
+
+def setup_tempdir
+  FileUtils.mkdir_p(TEMP_DIR)
+
+  File.exist?(TEMP_DIR) ? TEMP_DIR : nil
+end
+
+def chdir_tempdir
+  setup_tempdir  unless File.exist?(TEMP_DIR)
+  Dir.chdir(TEMP_DIR)
+end
+
+def teardown_tempdir
+  FileUtils.rm_rf(TEMP_DIR)  if File.exist?(TEMP_DIR)
+end
+
+def create_releases_file
+  FileUtils.mkdir_p("_data")
+  FileUtils.touch("_data/releases.yml")
+end
+
+def create_file(path, content)
+  raise "path must be relative"  unless Pathname.new(path).relative?
+
+  dir = File.dirname(path)
+  FileUtils.mkdir_p(dir)
+  File.open(path, "w") {|f| f.write content }
+end
+
+def linter_output
+  stdout, _stderr = capture_io { Linter.new(exit_on_errors: false).run }
+
+  stdout
+end
diff --git a/test/test_linter_valid_page.rb b/test/test_linter_valid_page.rb
@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require "helper"
+require "linter"
+
+
+describe Linter do
+
+  before do
+    chdir_tempdir
+    create_releases_file
+
+    @ok = "Checking markdown files... ok\n"
+  end
+
+  after do
+    teardown_tempdir
+  end
+
+  it "checks ok a valid page" do
+    content = <<~PAGE
+      ---
+      layout: page
+      title: "Page"
+      lang: en
+      ---
+
+      Content
+    PAGE
+
+    create_file("en/page.md", content)
+    _(linter_output).must_equal @ok
+  end
+end
