Skip to content

Commit

Permalink
Print module in undefined export report
Browse files Browse the repository at this point in the history
  • Loading branch information
lgoettgens committed Sep 12, 2023
1 parent 8bfe0f8 commit cdd63a9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/exports.jl
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ function undefined_exports(m::Module)
undefined = Symbol[]
walkmodules(m) do x
for n in names(x)
isdefined(x, n) || push!(undefined, n)
isdefined(x, n) || push!(undefined, Symbol(join([fullname(x)...; n], '.')))
end
end
return undefined
Expand Down
3 changes: 2 additions & 1 deletion test/test_undefined_exports.jl
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,8 @@ include("preamble.jl")
using PkgWithUndefinedExports

@testset begin
@test Aqua.undefined_exports(PkgWithUndefinedExports) == [:undefined_name]
@test Aqua.undefined_exports(PkgWithUndefinedExports) ==
[Symbol("PkgWithUndefinedExports.undefined_name")]
results = @testtestset begin
Aqua.test_undefined_exports(PkgWithUndefinedExports)
end
Expand Down

0 comments on commit cdd63a9

Please sign in to comment.