95d35d38da
* Use `Minitest::Test`
`Test::Unit` is deprecated, and we can switch to `Minitest::Test` with
almost no side effects.
This commit does all of the work required to make Minitest tests run
with Gollum's existing test helpers.
* Change Minitest output format
- The `DefaultReporter` seems to have cleaner output than what we had
before.
- `color: true` ensures things are colorized. It's pretty nice.
* Tweak test formatting; fix order-dependent failure
The order-dependent failure has been been commented in code.
After manually bisecting the test suite, I was able to determine that
the template cascade tests were leaving the `@@template_priority_path`
set to an overridden value in tests run afterward.
* Tweak setting initialization
I could not see a meaningful reason behind calling `forbid` so early in
the settings initialization block. But moving the `forbid` call until
later resolved this issue.
In the real world, I don't see how this would cause issues. But I found
that calling `forbid` when `wiki_options[:allow_editing]` was set to
false caused order-dependent test errors where Sprockets would end up
being badly configured and left without an initialized
`Sprockets::Environment` object, which is required by the
`sprockets-helpers` gem to resolve asset paths.
The test that would cause errors is also in this commit diff. I've
updated it to be a bit more readable.
I also took this opportunity to review and clean up the `@allow_editing`
assignment, which seemed a bit obfuscated.
* Update migration script to test run warnings
When running the entire test suite, many warnings would be output:
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: already initialized constant HYPHENATE
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: previous definition of HYPHENATE was here
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: already initialized constant PAGE_FILE_DIR
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: previous definition of PAGE_FILE_DIR was here
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:91: warning: already initialized constant REPO
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:91: warning: previous definition of REPO was here
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:236: warning: already initialized constant TREE
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:236: warning: previous definition of TREE was here
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: already initialized constant PAGE_FILE_DIR
/Users/bw/Projects/gollum/test/test_migrate.rb:37: warning: previous definition of PAGE_FILE_DIR was here
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:91: warning: already initialized constant REPO
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:91: warning: previous definition of REPO was here
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:236: warning: already initialized constant TREE
/Users/bw/Projects/gollum/bin/gollum-migrate-tags:236: warning: previous definition of TREE was here
While it's unlikely that end users would ever see these warnings, as
they'd only be running the migration script once, they will always be
shown in our local test runs and CI run output.
So instead of using constants in our migration script, I change the
script to use class variables instead. This should not effect the
functionality of the migration script whatsoever.
* Use `File.exist?` instead of `File.exists?`
In Ruby 3.x, `File.exists?` is deprecated and outputs a warning.
* Improve "allow editing" tests
While making changes to the test suite, I ran into some issues with
these tests failing on occasion.
I added some setup and teardown to help with this.
But one thing I did notice is that the word "Upload" appears in the
response body whether uploading is enabled or not, so I made these
assertions more specific to the HTML rather than other places the word
"Upload" might appear (which is related to Critic Markup).
* Do not attempt to modify frozen strings
Using `#gsub!` here now results in an error. I am not entirely sure why
this hasn't happened before, but the error is straightforward:
FrozenError: can't modify frozen String: "Author %{author} is from %{location}"
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:45:in `gsub!'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:45:in `fill_argument_content'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:37:in `block in autofill'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `each'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `map'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `autofill'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:35:in `block in autofill'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `each'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `map'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:33:in `autofill'
/home/runner/work/gollum/gollum/lib/gollum/views/helpers/locale_helpers.rb:21:in `t'
/home/runner/work/gollum/gollum/test/gollum/views/test_locale_helper.rb:95:in `block (4 levels) in <top (required)>'
`#gsub!` attempts to modify a string in-place, which does not work when
a string is frozen. It seems that in some Ruby environments, strings
from YAML files are frozen.
* Fix another order-dependent test failure
Very occasionally, this test fails due to `Precious::App` settings set
in previous tests. You can reproduce this failure using this seed:
bundle exec rake TESTOPTS="--seed=42898"
229 lines
7.1 KiB
Ruby
229 lines
7.1 KiB
Ruby
# ~*~ encoding: utf-8 ~*~
|
|
require File.expand_path(File.join(File.dirname(__FILE__), 'helper'))
|
|
require File.expand_path '../../lib/gollum/views/page', __FILE__
|
|
|
|
|
|
context "Precious::Views::Page" do
|
|
setup do
|
|
@path = cloned_testpath('examples/empty.git')
|
|
@wiki = Gollum::Wiki.new(@path)
|
|
end
|
|
|
|
teardown do
|
|
FileUtils.rm_rf(@path)
|
|
end
|
|
|
|
test "breadcrumbs guard against malicious input" do
|
|
malicious_path = '<script>alert("malicious-content");/Very Bad'
|
|
@wiki.write_page(malicious_path, :markdown, 'Is Bilbo a hobbit? Why certainly!')
|
|
page = @wiki.page(malicious_path)
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, false
|
|
|
|
refute_includes @view.breadcrumb, malicious_path
|
|
assert_includes @view.breadcrumb, "><script>alert("malicious-content");</a>"
|
|
end
|
|
|
|
test "breadcrumbs retain unicode and ASCII characters" do
|
|
path = "数学 📘/Age of Bilbo"
|
|
@wiki.write_page(path, :markdown, "How old is Bilbo?")
|
|
page = @wiki.page(path)
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, false
|
|
|
|
assert_includes @view.breadcrumb, "数学 📘"
|
|
end
|
|
|
|
test 'page <title> is the page header from content, if present' do
|
|
page_title = 'Page header from content'
|
|
@wiki.write_page(page_title, :markdown, 'Contents', commit_details)
|
|
|
|
@view = Precious::Views::Page.new.tap do |view|
|
|
view.instance_variable_set :@page, @wiki.page(page_title)
|
|
view.instance_variable_set :@h1_title, true
|
|
end
|
|
|
|
assert_equal @view.title, 'Page header from content'
|
|
end
|
|
|
|
test 'page <title> is URL path title if no h1 present' do
|
|
@wiki.write_page('dir/My path title', :markdown, 'Contents', commit_details)
|
|
page = @wiki.page('dir/My path title')
|
|
|
|
@view = Precious::Views::Page.new.tap do |view|
|
|
view.instance_variable_set :@page, page
|
|
view.instance_variable_set :@h1_title, false
|
|
end
|
|
|
|
assert_equal @view.title, 'My path title'
|
|
end
|
|
|
|
test "page header retains unicode and ASCII characters" do
|
|
title = "数学 📘"
|
|
@wiki.write_page(title, :markdown, "How old is Bilbo?")
|
|
page = @wiki.page(title)
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, false
|
|
|
|
assert @view.page_header, "数学 📘"
|
|
end
|
|
|
|
test "h1 title sanitizes correctly" do
|
|
title = 'H1'
|
|
@wiki.write_page(title, :markdown, '# 1 & 2 <script>alert("js")</script>' + "\n # 3", commit_details)
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, true
|
|
|
|
# Test page_header_from_content(@content)
|
|
assert @view.page_header, "1 & 2"
|
|
end
|
|
|
|
test "page header uses filename when h1_title is false" do
|
|
title = "H1"
|
|
contents = <<~TEXT
|
|
# First H1 header
|
|
# Second H1 header
|
|
TEXT
|
|
|
|
@wiki.write_page(title, :markdown, contents, commit_details)
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, false
|
|
|
|
assert_equal @view.page_header, "H1"
|
|
end
|
|
|
|
test "page header uses filename when h1_title is true" do
|
|
contents = <<~TEXT
|
|
# First H1 header
|
|
# Second H1 header
|
|
TEXT
|
|
|
|
@wiki.write_page("H1", :markdown, contents, commit_details)
|
|
page = @wiki.page("H1")
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
@view.instance_variable_set :@h1_title, true
|
|
|
|
assert_equal @view.page_header, "First H1 header"
|
|
end
|
|
|
|
|
|
|
|
test "metadata is rendered into a table" do
|
|
title = 'metadata test'
|
|
@wiki.write_page(title, :markdown, "---\nsome: metadata\nhere: for you\n---\n# Some markdown\nIn this doc")
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
|
|
assert_equal @view.rendered_metadata, <<-EOS
|
|
<table>
|
|
<tr>
|
|
<th>some</th>
|
|
<th>here</th>
|
|
</tr>
|
|
<tr>
|
|
<td>metadata</td>
|
|
<td>for you</td>
|
|
</tr>
|
|
</table>
|
|
EOS
|
|
end
|
|
|
|
test "allow numbered headings based on metadata" do
|
|
title = 'header enumeration test'
|
|
@wiki.write_page(title, :markdown, "---\nheader_enum: true\n---\n# Some markdown\nIn this doc")
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
|
|
assert_equal @view.header_enum?, true
|
|
assert_equal @view.header_enum_style, 'decimal'
|
|
|
|
title = 'header_enum test2'
|
|
@wiki.write_page(title, :markdown, "---\nheader_enum: 'lower-roman'\n---\n# Some markdown\nIn this doc")
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
|
|
assert_equal @view.header_enum?, true
|
|
assert_equal @view.header_enum_style, 'lower-roman'
|
|
|
|
# With invalid style
|
|
title = 'header_enum test3'
|
|
@wiki.write_page(title, :markdown, "---\nheader_enum: 'roman'\n---\n# Some markdown\nIn this doc")
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
|
|
assert_equal @view.header_enum?, true
|
|
assert_equal @view.header_enum_style, 'decimal'
|
|
end
|
|
|
|
test 'page has sha id' do
|
|
title = 'test'
|
|
@wiki.write_page(title, :markdown, 'Test' + "\n # 3", commit_details)
|
|
page = @wiki.page(title)
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
assert_equal "594e928cc5dcb6d833dfb86bb36076fd4a84eea7", @view.id
|
|
end
|
|
|
|
test "breadcrumbs" do
|
|
@wiki.write_page('subdir/BC Test 1', :markdown, 'Test', commit_details)
|
|
page = @wiki.page('subdir/BC Test 1')
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
assert_equal @view.breadcrumb, "<nav aria-label=\"Breadcrumb\"><ol>\n<li class=\"breadcrumb-item\"><a href=\"/gollum/overview/subdir/\">subdir</a></li>\n</ol></nav>"
|
|
|
|
|
|
# No breadcrumb on unnested page
|
|
@wiki.write_page('BC Test 2', :markdown, 'Test', commit_details)
|
|
page = @wiki.page('BC Test 2')
|
|
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@page, page
|
|
@view.instance_variable_set :@content, page.formatted_data
|
|
assert_equal @view.breadcrumb, ''
|
|
end
|
|
|
|
test "body_side is 'right' by default" do
|
|
@view = Precious::Views::Page.new
|
|
assert_equal @view.body_side, "right"
|
|
end
|
|
|
|
test "body_side is 'left' if bar_side side is 'right'" do
|
|
@view = Precious::Views::Page.new
|
|
@view.instance_variable_set :@bar_side, :right
|
|
assert_equal @view.body_side, "left"
|
|
end
|
|
|
|
test "links to pages containing ?" do
|
|
@view = Precious::Views::Page.new
|
|
assert_equal @view.page_route("Page?"), '/Page%3F'
|
|
end
|
|
end
|