Skip to content

test: capture expected deprecation warnings in deprecated-format tests #63

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.

Sign up for GitHub

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

Merged
zachdaniel merged 1 commit into from
Mar 20, 2026
Merged

test: capture expected deprecation warnings in deprecated-format tests #63

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
130 changes: 82 additions & 48 deletions test/mix/tasks/usage_rules.sync_test.exs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

defmodule Mix.Tasks.UsageRules.SyncTest do
use ExUnit.Case, async: true
import ExUnit.CaptureIO
import Igniter.Test

defp sync(igniter, config) do
Expand All @@ -23,6 +24,22 @@ defmodule Mix.Tasks.UsageRules.SyncTest do
|> Rewrite.Source.get(:content)
end

defp capture_stderr_result(fun) do
parent = self()

output =
capture_io(:stderr, fn ->
send(parent, {:captured_result, fun.()})
end)

result =
receive do
{:captured_result, result} -> result
end

{output, result}
end

describe "config validation" do
test "errors when no config is provided" do
project_with_deps()
Expand Down Expand Up @@ -1167,22 +1184,26 @@ defmodule Mix.Tasks.UsageRules.SyncTest do

describe "deprecated {:dep, :reference} format" do
test "build with {:dep, :reference} still works but all packages are references" do
igniter =
project_with_deps(%{
"deps/foo/usage-rules.md" => "# Foo Rules\n\nFoo guidance.",
"deps/bar/usage-rules.md" => "# Bar Rules\n\nBar guidance."
})
|> sync(
skills: [
location: ".claude/skills",
build: [
"my-skill": [usage_rules: [:foo, {:bar, :reference}]]
{output, igniter} =
capture_stderr_result(fn ->
project_with_deps(%{
"deps/foo/usage-rules.md" => "# Foo Rules\n\nFoo guidance.",
"deps/bar/usage-rules.md" => "# Bar Rules\n\nBar guidance."
})
|> sync(
skills: [
location: ".claude/skills",
build: [
"my-skill": [usage_rules: [:foo, {:bar, :reference}]]
]
]
]
)
|> assert_creates(".claude/skills/my-skill/SKILL.md")
|> assert_creates(".claude/skills/my-skill/references/foo.md")
|> assert_creates(".claude/skills/my-skill/references/bar.md")
)
|> assert_creates(".claude/skills/my-skill/SKILL.md")
|> assert_creates(".claude/skills/my-skill/references/foo.md")
|> assert_creates(".claude/skills/my-skill/references/bar.md")
end)

assert output =~ "deprecated in usage_rules skill config"

skill_content = file_content(igniter, ".claude/skills/my-skill/SKILL.md")
# Both foo and bar should be reference links
Expand All @@ -1197,24 +1218,28 @@ defmodule Mix.Tasks.UsageRules.SyncTest do
end

test "build with {~r/.../, :reference} still works but all packages are references" do
igniter =
project_with_deps(%{
"deps/ash/usage-rules.md" => "# Ash Core",
"deps/ash_postgres/usage-rules.md" => "# Ash Postgres",
"deps/ash_json_api/usage-rules.md" => "# Ash JSON API"
})
|> sync(
skills: [
location: ".claude/skills",
build: [
"ash-expert": [usage_rules: [:ash, {~r/^ash_/, :reference}]]
{output, igniter} =
capture_stderr_result(fn ->
project_with_deps(%{
"deps/ash/usage-rules.md" => "# Ash Core",
"deps/ash_postgres/usage-rules.md" => "# Ash Postgres",
"deps/ash_json_api/usage-rules.md" => "# Ash JSON API"
})
|> sync(
skills: [
location: ".claude/skills",
build: [
"ash-expert": [usage_rules: [:ash, {~r/^ash_/, :reference}]]
]
]
]
)
|> assert_creates(".claude/skills/ash-expert/SKILL.md")
|> assert_creates(".claude/skills/ash-expert/references/ash.md")
|> assert_creates(".claude/skills/ash-expert/references/ash_postgres.md")
|> assert_creates(".claude/skills/ash-expert/references/ash_json_api.md")
)
|> assert_creates(".claude/skills/ash-expert/SKILL.md")
|> assert_creates(".claude/skills/ash-expert/references/ash.md")
|> assert_creates(".claude/skills/ash-expert/references/ash_postgres.md")
|> assert_creates(".claude/skills/ash-expert/references/ash_json_api.md")
end)

assert output =~ "deprecated in usage_rules skill config"

skill_content = file_content(igniter, ".claude/skills/ash-expert/SKILL.md")
assert skill_content =~ "[ash](references/ash.md)"
Expand All @@ -1223,13 +1248,17 @@ defmodule Mix.Tasks.UsageRules.SyncTest do
end

test "deps config with {:dep, :reference} still works" do
igniter =
project_with_deps(%{
"deps/foo/usage-rules.md" => "# Foo Rules\n\nFoo content."
})
|> sync(skills: [location: ".claude/skills", deps: [{:foo, :reference}]])
|> assert_creates(".claude/skills/use-foo/SKILL.md")
|> assert_creates(".claude/skills/use-foo/references/foo.md")
{output, igniter} =
capture_stderr_result(fn ->
project_with_deps(%{
"deps/foo/usage-rules.md" => "# Foo Rules\n\nFoo content."
})
|> sync(skills: [location: ".claude/skills", deps: [{:foo, :reference}]])
|> assert_creates(".claude/skills/use-foo/SKILL.md")
|> assert_creates(".claude/skills/use-foo/references/foo.md")
end)

assert output =~ "deprecated in usage_rules skill config"

skill_content = file_content(igniter, ".claude/skills/use-foo/SKILL.md")
assert skill_content =~ "[foo](references/foo.md)"
Expand All @@ -1239,15 +1268,20 @@ defmodule Mix.Tasks.UsageRules.SyncTest do
end

test "deps config with {~r/.../, :reference} still works" do
project_with_deps(%{
"deps/ash_postgres/usage-rules.md" => "# Ash Postgres Rules",
"deps/ash_json_api/usage-rules.md" => "# Ash JSON API Rules"
})
|> sync(skills: [location: ".claude/skills", deps: [{~r/^ash_/, :reference}]])
|> assert_creates(".claude/skills/use-ash_postgres/SKILL.md")
|> assert_creates(".claude/skills/use-ash_postgres/references/ash_postgres.md")
|> assert_creates(".claude/skills/use-ash_json_api/SKILL.md")
|> assert_creates(".claude/skills/use-ash_json_api/references/ash_json_api.md")
{output, _igniter} =
capture_stderr_result(fn ->
project_with_deps(%{
"deps/ash_postgres/usage-rules.md" => "# Ash Postgres Rules",
"deps/ash_json_api/usage-rules.md" => "# Ash JSON API Rules"
})
|> sync(skills: [location: ".claude/skills", deps: [{~r/^ash_/, :reference}]])
|> assert_creates(".claude/skills/use-ash_postgres/SKILL.md")
|> assert_creates(".claude/skills/use-ash_postgres/references/ash_postgres.md")
|> assert_creates(".claude/skills/use-ash_json_api/SKILL.md")
|> assert_creates(".claude/skills/use-ash_json_api/references/ash_json_api.md")
end)

assert output =~ "deprecated in usage_rules skill config"
end

test "all packages appear in search docs section" do
Expand Down
Loading