Skip to content

Commit f936bcf

Browse files
committed
mix format
1 parent 00f2d15 commit f936bcf

File tree

8 files changed

+106
-37
lines changed

8 files changed

+106
-37
lines changed

config/release.exs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +0,0 @@
1-

lib/cldr/locale_display.ex

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,13 @@ defmodule Cldr.LocaleDisplay do
120120

121121
match_fun = &language_match_fun(&1, &2, display_names.language)
122122

123-
case first_match(language_tag, match_fun, @omit_script_if_only_one, language_display, prefer) do
123+
case first_match(
124+
language_tag,
125+
match_fun,
126+
@omit_script_if_only_one,
127+
language_display,
128+
prefer
129+
) do
124130
{language_name, matched_tags} ->
125131
language_tag = merge_extensions_and_private_use(language_tag)
126132

@@ -140,7 +146,9 @@ defmodule Cldr.LocaleDisplay do
140146
{:ok, format_display_name(language_name, subtag_names, extension_names, display_names)}
141147

142148
nil ->
143-
{:error, {Cldr.DisplayName.NoDataError, "The locale #{inspect in_locale} has no display name data."}}
149+
{:error,
150+
{Cldr.DisplayName.NoDataError,
151+
"The locale #{inspect(in_locale)} has no display name data."}}
144152
end
145153
end
146154
end
@@ -244,9 +252,10 @@ defmodule Cldr.LocaleDisplay do
244252
@reinstate_subtags [:territory, :script]
245253

246254
defp first_match(language_tag, match_fun, omit_script_if_only_one?, :standard, prefer) do
247-
language_tag = Enum.reduce(@reinstate_subtags, language_tag, fn key, tag ->
248-
Map.put(tag, key, nil)
249-
end)
255+
language_tag =
256+
Enum.reduce(@reinstate_subtags, language_tag, fn key, tag ->
257+
Map.put(tag, key, nil)
258+
end)
250259

251260
case Cldr.Locale.first_match(language_tag, match_fun, omit_script_if_only_one?) do
252261
{language_name, matched_tags} ->

lib/cldr/locale_display/t.ex

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,13 @@ defmodule Cldr.LocaleDisplay.T do
5252
end
5353
end
5454

55-
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer) when is_binary(value) do
55+
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer)
56+
when is_binary(value) do
5657
replace_parens_with_brackets(value)
5758
end
5859

59-
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer) when is_atom(value) do
60+
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer)
61+
when is_atom(value) do
6062
value
6163
|> to_string()
6264
|> replace_parens_with_brackets()

lib/cldr/locale_display/u.ex

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ defmodule Cldr.LocaleDisplay.U do
3333
# Returns the localised value for the key. If there is
3434
# no available key name then just return the value.
3535

36-
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer) when is_binary(value) do
36+
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer)
37+
when is_binary(value) do
3738
replace_parens_with_brackets(value)
3839
end
3940

40-
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer) when is_atom(value) do
41+
defp display_value(_key, nil, value, _transform, _in_locale, _display_names, _prefer)
42+
when is_atom(value) do
4143
value
4244
|> to_string()
4345
|> replace_parens_with_brackets()
@@ -165,8 +167,9 @@ defmodule Cldr.LocaleDisplay.U do
165167
zone_name = String.replace(city, "_", " ")
166168
Cldr.Substitution.substitute([zone_name], territory_format)
167169
end
168-
else _other ->
169-
zone
170+
else
171+
_other ->
172+
zone
170173
end
171174
end
172175

mix.exs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,8 @@ defmodule Cldr.LocaleDisplay.MixProject do
4141
[
4242
# {:ex_cldr, "~> 2.37"},
4343
{:ex_cldr, path: "../cldr", override: true},
44-
4544
{:ex_cldr_currencies, "~> 2.12"},
4645
{:ex_cldr_territories, "~> 2.4"},
47-
4846
{:jason, "~> 1.0", optional: true},
4947
{:ex_doc, "~> 0.18", only: [:dev, :release], runtime: false, optional: true},
5048
{:dialyxir, "~> 1.0", only: [:test, :dev], runtime: false},
@@ -81,7 +79,8 @@ defmodule Cldr.LocaleDisplay.MixProject do
8179
def links do
8280
%{
8381
"GitHub" => "https://github.com/elixir-cldr/cldr_locale_display",
84-
"Readme" => "https://github.com/elixir-cldr/cldr_locale_display/blob/v#{@version}/README.md",
82+
"Readme" =>
83+
"https://github.com/elixir-cldr/cldr_locale_display/blob/v#{@version}/README.md",
8584
"Changelog" =>
8685
"https://github.com/elixir-cldr/cldr_locale_display/blob/v#{@version}/CHANGELOG.md"
8786
}

test/locale_display_name_test.exs

Lines changed: 73 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -5,40 +5,80 @@ defmodule Cldr.LocaleDisplayName.Test do
55
doctest MyApp.Cldr.LocaleDisplay
66

77
# These tests have issues:
8-
# * 1566, 1569, 1686, 1699 seem to have the icorrect test result in the data
8+
# * 1566, 1569, 1686, 1699 seem to have the incorrect test result in the data
99
# * 344, 345 appear to have the wrong timezone returned
1010
# * 21,41 includes a field for a -u- tag that is invalid
1111
@invalid_test_results [21, 41, 344, 345, 1556, 1569, 1686, 1699]
1212

1313
# These have test results with [] but I think they should remain ()
1414
# since there are no subtags or extensions.
1515
@results_should_be_parens [
16-
396, 474, 500, 786, 864, 942, 1124, 1203, 1254, 1358, 1384, 1538, 1928, 2008,
17-
2188, 2214, 2242, 2254, 2268, 2294, 2500, 2554, 2580, 2684, 2708, 2942, 2968,
18-
2994, 3020, 3022, 3126, 3152, 3178
16+
396,
17+
474,
18+
500,
19+
786,
20+
864,
21+
942,
22+
1124,
23+
1203,
24+
1254,
25+
1358,
26+
1384,
27+
1538,
28+
1928,
29+
2008,
30+
2188,
31+
2214,
32+
2242,
33+
2254,
34+
2268,
35+
2294,
36+
2500,
37+
2554,
38+
2580,
39+
2684,
40+
2708,
41+
2942,
42+
2968,
43+
2994,
44+
3020,
45+
3022,
46+
3126,
47+
3152,
48+
3178
1949
]
2050

2151
# The test results for these are what would be generated with
2252
# language_display: :standard, but the test specifies language_tag: :dialect
2353
@wrong_language_display [
24-
2370, 2374, 2375, 2371
54+
2370,
55+
2374,
56+
2375,
57+
2371
2558
]
2659

2760
@except_lines @invalid_test_results ++ @results_should_be_parens ++ @wrong_language_display
2861
@locales [:en, :fr, :de, :it, :es, :zh, :"zh-Hans", :"zh-Hant", :ja]
2962

30-
for [line, locale, language_display, from, to] <- Cldr.LocaleDisplayNameGenerator.data(), line not in @except_lines do
31-
test "##{line} Locale #{inspect(from)} becomes #{inspect(to)} in locale #{inspect locale}" do
32-
assert Cldr.LocaleDisplay.display_name!(unquote(from), locale: unquote(locale), language_display: unquote(language_display)) ==
63+
for [line, locale, language_display, from, to] <- Cldr.LocaleDisplayNameGenerator.data(),
64+
line not in @except_lines do
65+
test "##{line} Locale #{inspect(from)} becomes #{inspect(to)} in locale #{inspect(locale)}" do
66+
assert Cldr.LocaleDisplay.display_name!(unquote(from),
67+
locale: unquote(locale),
68+
language_display: unquote(language_display)
69+
) ==
3370
unquote(to)
3471
end
3572
end
3673

3774
for [line, _locale, language_display, from, _to] <- Cldr.LocaleDisplayNameGenerator.data(),
3875
locale <- @locales,
3976
line not in @except_lines && locale != :und do
40-
test "##{line} Language tag #{inspect(from)} in locale #{inspect locale} renders" do
41-
assert Cldr.LocaleDisplay.display_name!(unquote(from), locale: unquote(locale), language_display: unquote(language_display))
77+
test "##{line} Language tag #{inspect(from)} in locale #{inspect(locale)} renders" do
78+
assert Cldr.LocaleDisplay.display_name!(unquote(from),
79+
locale: unquote(locale),
80+
language_display: unquote(language_display)
81+
)
4282
end
4383
end
4484

@@ -55,8 +95,13 @@ defmodule Cldr.LocaleDisplayName.Test do
5595
assert Cldr.display_name(~l(en)u) == "English"
5696
assert Cldr.display_name(~l(zh-Hans)u, language_display: :dialect) == "Simplified Chinese"
5797
assert Cldr.display_name(~l(zh-Hant)u, language_display: :dialect) == "Traditional Chinese"
58-
assert Cldr.display_name(~l(zh-Hant), language_display: :dialect) == "Traditional Chinese (Taiwan)"
59-
assert Cldr.display_name(~l(zh-Hans), language_display: :dialect) == "Simplified Chinese (China)"
98+
99+
assert Cldr.display_name(~l(zh-Hant), language_display: :dialect) ==
100+
"Traditional Chinese (Taiwan)"
101+
102+
assert Cldr.display_name(~l(zh-Hans), language_display: :dialect) ==
103+
"Simplified Chinese (China)"
104+
60105
assert Cldr.display_name(~l(zh-Hant)u) == "Chinese (Traditional)"
61106
assert Cldr.display_name(~l(zh-Hans)u) == "Chinese (Simplified)"
62107
assert Cldr.display_name(~l(zh-Hant)) == "Chinese (Traditional, Taiwan)"
@@ -66,13 +111,21 @@ defmodule Cldr.LocaleDisplayName.Test do
66111
test "More complex language tags" do
67112
import Cldr.LanguageTag.Sigil
68113

69-
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi"u, locale: "zh-Hant")
70-
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi-h0-hybrid"u,
71-
locale: "zh-Hant")
114+
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi"u,
115+
locale: "zh-Hant"
116+
)
72117

73-
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi"u, locale: "zh-Hans")
74-
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi-h0-hybrid"u,
75-
locale: "zh-Hans")
76-
end
118+
assert Cldr.display_name(
119+
~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi-h0-hybrid"u,
120+
locale: "zh-Hant"
121+
)
122+
123+
assert Cldr.display_name(~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi"u,
124+
locale: "zh-Hans"
125+
)
77126

78-
end
127+
assert Cldr.display_name(
128+
~l"fr-CA-u-ca-gregory-nu-arab-cu-usd-cf-account-ms-uksystem-t-hi-h0-hybrid"u,
129+
locale: "zh-Hans"
130+
)
131+
end

test/support/locale_display_name_generator.ex

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,19 @@ defmodule Cldr.LocaleDisplayNameGenerator do
2222
{acc, _, _} =
2323
Enum.reduce(list, {[], nil, nil}, fn
2424
[line, locale, display], {acc, test_locale, language_display} ->
25-
{[[line, test_locale, language_display, locale, display] | acc], test_locale, language_display}
25+
{[[line, test_locale, language_display, locale, display] | acc], test_locale,
26+
language_display}
2627

2728
[_line, option], {acc, locale, language_display} ->
2829
case String.split(option, "=") do
2930
["@locale", locale] ->
3031
{acc, locale, language_display}
32+
3133
["@languageDisplay", language_display] ->
3234
{acc, locale, String.to_atom(language_display)}
3335
end
3436
end)
37+
3538
acc
3639
end
3740
end

test/support/test_backend.ex

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@ require Cldr.LocaleDisplay
22

33
defmodule MyApp.Cldr do
44
use Cldr,
5-
locales: :all, # ["en", "fr", "de", "zh"],
5+
# ["en", "fr", "de", "zh"],
6+
locales: :all,
67
default_locale: "en",
78
providers: [Cldr.Territory, Cldr.LocaleDisplay, Cldr.Currency]
89
end

0 commit comments

Comments
 (0)