Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update link tests #79

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
2 changes: 1 addition & 1 deletion _includes/blocks/tabs.html
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<ul>
{% for tab in include.tabs %}
<li class="tab" id="{{ tab.label | slugify }}-heading" onclick="openTab('{{ tab.label | slugify }}')">
<h3><a>{{ tab.label }}</a></h3>
<h3><span>{{ tab.label }}</span></h3>
</li>
{% endfor %}
</ul>
Expand Down
2 changes: 1 addition & 1 deletion _includes/head.html
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
{%- include misc/google-analytics.html -%}
{%- endif -%}

{% if page.hero.image %}
{% if page.hero.image and page.hero.image != "" %}
<link rel="preload" as="image" href="{{ page.hero.image | absolute_url }}">
{% endif %}

Expand Down
2 changes: 1 addition & 1 deletion _layouts/default.html
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
</section>
{%- include footer.html -%}
<script src="https://code.jquery.com/jquery-3.6.0.min.js" integrity="sha256-/xUj+3OJU5yExlq6GSYGSHk7tPXikynS7ogEvDej/m4=" crossorigin="anonymous"></script>
<script type="text/javascript" src="//cdn.jsdelivr.net/npm/[email protected]/slick/slick.min.js"></script>
<script type="text/javascript" src="https://cdn.jsdelivr.net/npm/[email protected]/slick/slick.min.js"></script>
<script src="https://kit.fontawesome.com/2c3e99340f.js" crossorigin="anonymous"></script>
<script type="text/javascript" src="{{ 'assets/js/carouselInit.js' | absolute_url }}"></script>

Expand Down
42 changes: 19 additions & 23 deletions lib/tasks/test.rake
Original file line number Diff line number Diff line change
@@ -1,16 +1,10 @@
require 'iiifc'

namespace :test do
desc 'Check html'
desc 'Check Images and Scripts'
task :html do
opts = {
check_html: true,
assume_extension: true,
validation: {
report_mismatched_tags: true,
report_invalid_tags: true
},
checks_to_ignore: ['LinkCheck']
checks: ['Images', 'Scripts']
}
HTMLProofer.check_directory(SITE_DIR, opts).run
end
Expand All @@ -20,33 +14,35 @@ namespace :test do
task :internal do
puts 'Checking for internal link errors'
opts = {
checks_to_ignore: ['ImageCheck', 'HtmlCheck', 'ScriptCheck'],
checks: ['Links'],
disable_external: true,
internal_domains: ['localhost:4000']
enforce_https: false,
internal_domains: ['localhost:4000'],
}
HTMLProofer.check_directory(SITE_DIR, opts).run
end

desc 'Check for *iiif.io* link errors'
task :iiif do
puts 'Checking for link errors in *iiif.io* sites'

desc 'Check for external link rot'
task :external do
puts 'Checking for external link errors'
opts = {
checks_to_ignore: ['ImageCheck', 'HtmlCheck', 'ScriptCheck'],
url_ignore: [/^((?!iiif\.io).)*$/, 'github'] # temporarily ignore iiif.io github repo errors
enforce_https: false,
ignore_status_codes: [429],
only_4xx: true,
checks: ['Links','Images'],
url_ignore: [/.*iiif\.io.*/],
ignore_files: [/.*news\/.*/]
}
HTMLProofer.check_directory(SITE_DIR, opts).run
end

desc 'Check for external link rot'
task :external do
puts 'Checking for external link errors'
desc 'Check for https link errors'
task :https do
puts 'Checking for https link errors'
opts = {
external_only: true,
http_status_ignore: [429],
enforce_https: true,
only_4xx: true,
checks_to_ignore: ['ImageCheck', 'HtmlCheck', 'ScriptCheck'],
url_ignore: [/.*iiif\.io.*/]
checks: ['Links','Images'],
}
HTMLProofer.check_directory(SITE_DIR, opts).run
end
Expand Down