Specs cleanup

_includes/toggle-color-scheme.html

@@ -20,8 +20,8 @@
   <button type="button"
     class="btn js-unset-color-scheme btn-outline px-2"
     style="margin-left: -3px; border-top-left-radius: 0; border-bottom-left-radius: 0;"
-    aria-label="Reset color scheme"
-    >⎌</button>
+    aria-label="Automatically switch theme"
+    ><i class="fa-solid fa-rotate" title="Automatically switch theme"></i></button>
 </span>
 
 <script>

home.md

@@ -1,11 +1,11 @@
 ---
 layout: page
-title: Just the Class
+title: Welcome!
 nav_exclude: true
-permalink: /:path/
+permalink: /
 seo:
   type: Course
-  name: Just the Class
+  name: Berkeley Class Site
 ---
 
 # UC Berkeley Class Site Template

spec/accessibility_spec.rb

@@ -30,24 +30,40 @@
 puts "Running tests on #{ALL_PAGES.count} pages."
 puts "  - #{ALL_PAGES.join("\n  - ")}\n\n"
 
+RSpec.shared_examples 'a11y tests' do
+  it 'meets WCAG 2.1' do
+    expect(page).to be_axe_clean
+      .according_to(*required_a11y_standards)
+      .skipping(*skipped_rules)
+      .excluding(*excluded_elements)
+  end
+
+  it 'meets WCAG 2.2' do
+    expect(page).to be_axe_clean
+      .according_to(*complete_a11y_standards)
+      .skipping(*skipped_rules)
+      .excluding(*excluded_elements)
+  end
+end
+
 ALL_PAGES.each do |path|
   describe "#{path} is accessible", :js, type: :feature do
-    before do
-      visit(path)
-    end
+    context 'when light mode' do
+      before do
+        visit(path)
+        page.execute_script('jtd.setTheme("light")')
+      end
 
-    it 'meets WCAG 2.1' do
-      expect(page).to be_axe_clean
-        .according_to(*required_a11y_standards)
-        .skipping(*skipped_rules)
-        .excluding(*excluded_elements)
+      include_context 'a11y tests'
     end
 
-    it 'meets WCAG 2.2' do
-      expect(page).to be_axe_clean
-        .according_to(*complete_a11y_standards)
-        .skipping(*skipped_rules)
-        .excluding(*excluded_elements)
+    context 'when dark mode' do
+      before do
+        visit(path)
+        page.execute_script('jtd.setTheme("dark")')
+      end
+
+      include_context 'a11y tests'
     end
   end
 end

spec/spec_helper.rb

@@ -57,9 +57,15 @@
 end
 
 Capybara::Screenshot.register_filename_prefix_formatter(:rspec) do |example|
-  page = example.example_group.top_level_description.gsub(%r{^/}, '').gsub('/', '-').gsub(' is accessible', '')
-  standards = example.description.split.last
-  "tmp/capybara/screenshot_#{page}_#{standards}"
+  # Highly specific to a11y specs: path-mode-wcag-version
+  # TODO: Find a nice way to name "index" pages, or consider using Capybara.page.title
+  page = example.example_group.top_level_description.gsub(' is accessible', '')
+  mode = example.example_group.description # i.e. light mode / dark mode
+  standard = example.description.split.last # i.e "meets WCAG 2.1"
+  test_case = "#{page}_#{mode}_#{standard}"
+  test_case = test_case.gsub(%r{^/}, '').gsub(%r{[/\s+]}, '-')
+
+  "tmp/capybara/screenshot_#{test_case}"
 end
 
 Capybara::Screenshot.autosave_on_failure = true