From a2fe91cc5f27b2ccf69ff44367111dee77358277 Mon Sep 17 00:00:00 2001 From: Andrew Moore Date: Sun, 16 Apr 2023 19:07:35 -0600 Subject: [PATCH] fix tests --- lib/mix/tasks/excessibility.ex | 7 ------- test/excessibility_test.exs | 18 +++++++----------- test_fail/test/test_helper.exs | 2 +- 3 files changed, 8 insertions(+), 19 deletions(-) diff --git a/lib/mix/tasks/excessibility.ex b/lib/mix/tasks/excessibility.ex index caf7ddd..a2e3297 100644 --- a/lib/mix/tasks/excessibility.ex +++ b/lib/mix/tasks/excessibility.ex @@ -2,7 +2,6 @@ defmodule Mix.Tasks.Excessibility do @moduledoc "Library to aid in testing your application for WCAG compliance automatically using Pa11y and Wallaby." @shortdoc "Runs pally against generated snapshots" @requirements ["app.config"] - @assets_task Application.compile_env(:excessibility, :assets_task, "assets.deploy") @pally_path Application.compile_env( :excessibility, :pa11y_path, @@ -20,12 +19,6 @@ defmodule Mix.Tasks.Excessibility do @impl Mix.Task def run(_args) do - Mix.Task.run(@assets_task) - - File.mkdir_p("#{@ex_assets_path}/css/") - File.mkdir_p("#{@ex_assets_path}/js/") - File.mkdir_p("#{@snapshots_path}/") - File.ls!(@snapshots_path) |> filter_dirs() |> run_pa11y() diff --git a/test/excessibility_test.exs b/test/excessibility_test.exs index f74e1c0..76039a1 100644 --- a/test/excessibility_test.exs +++ b/test/excessibility_test.exs @@ -3,19 +3,16 @@ defmodule ExcessibilityTest do import ExUnit.CaptureIO - @output_path "test/excessibility" - @snapshots_path "#{@output_path}/html_snapshots" - @ex_assets_path "#{@output_path}/assets" + @snapshots_path "test/excessibility/html_snapshots" describe "run/1" do - @describetag :flipper test "when all pa11y checks pass" do Mix.Project.in_project(:test_pass, "test_pass/", fn _ -> - assert capture_io(fn -> + refute capture_io(fn -> Mix.Task.run("excessibility") - end) =~ + end) + |> IO.inspect() =~ "Error:" - |> IO.inspect() end) end @@ -23,16 +20,15 @@ defmodule ExcessibilityTest do Mix.Project.in_project(:test_fail, "test_fail/", fn _ -> assert capture_io(fn -> Mix.Task.run("excessibility") - end) =~ + end) + |> IO.inspect() =~ "Error:" - |> IO.inspect() end) end test "creates necessary directories for assets" do Mix.Project.in_project(:test_pass, "test_pass/", fn _ -> - assert File.exists?("#{@ex_assets_path}/css") - assert File.exists?("#{@ex_assets_path}/js") + assert File.exists?("#{@snapshots_path}") end) end end diff --git a/test_fail/test/test_helper.exs b/test_fail/test/test_helper.exs index 78b0817..073c2fb 100644 --- a/test_fail/test/test_helper.exs +++ b/test_fail/test/test_helper.exs @@ -1,2 +1,2 @@ -ExUnit.start(exclude: [:flipper]) +ExUnit.start() Ecto.Adapters.SQL.Sandbox.mode(TestFail.Repo, :manual)