Skip to content

Commit

Permalink
Correct known_territory_subdivisions typespec and add test (#24)
Browse files Browse the repository at this point in the history
  • Loading branch information
zorbash authored Oct 20, 2022
1 parent 8c2a43a commit cc16e56
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/cldr/territory.ex
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ defmodule Cldr.Territory do
"gbdal", "gbdby", "gbden", ...]}
"""
@spec known_territory_subdivisions(atom_binary_tag(), Cldr.backend()) ::
{:ok, binary()} | {:error, error()}
{:ok, binary() | nil} | {:error, error()}
def known_territory_subdivisions(territory_code, backend) do
module = Module.concat(backend, Territory)
module.known_territory_subdivisions(territory_code)
Expand Down
13 changes: 13 additions & 0 deletions test/territory_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -1032,4 +1032,17 @@ defmodule Cldr.TerritoryTest do
end
end

describe "known_territory_subdivisions/2" do
test "with valid params" do
assert {:ok, ["usak", "usal" | _]} =
Territory.known_territory_subdivisions(:US, TestBackend.Cldr)
end

test "with invalid params" do
assert {:ok, nil} = Territory.known_territory_subdivisions(:EU, TestBackend.Cldr)

assert {:error, {Cldr.UnknownTerritoryError, "The territory :NOPE is unknown"}} =
Territory.known_territory_subdivisions(:NOPE, TestBackend.Cldr)
end
end
end

0 comments on commit cc16e56

Please sign in to comment.