Merge pull request #38 from qdentity/gettext

Add built-in gettext check

README.md

@@ -70,6 +70,7 @@ def deps do
     {:dialyxir, ">= 0.0.0", only: [:dev], runtime: false},
     {:doctor, ">= 0.0.0", only: [:dev], runtime: false},
     {:ex_doc, ">= 0.0.0", only: [:dev], runtime: false},
+    {:gettext, ">= 0.0.0", only: [:dev], runtime: false},
     {:sobelow, ">= 0.0.0", only: [:dev], runtime: false},
     {:mix_audit, ">= 0.0.0", only: [:dev], runtime: false}
   ]

lib/ex_check/config/default.ex

@@ -16,6 +16,7 @@ defmodule ExCheck.Config.Default do
     {:ex_doc, "mix docs", detect: [{:package, :ex_doc}]},
     {:ex_unit, "mix test", detect: [{:file, "test"}], retry: "mix test --failed"},
     {:dialyzer, "mix dialyzer", detect: [{:package, :dialyxir}]},
+    {:gettext, "mix gettext.extract --check-up-to-date", detect: [{:package, :gettext}]},
     {:npm_test, "npm test", cd: "assets", detect: [{:file, "package.json", else: :disable}]}
   ]
 

lib/mix/tasks/check.ex

@@ -36,6 +36,8 @@ defmodule Mix.Tasks.Check do
   - [`:ex_doc`] - compiles the project documentation in order to ensure that there are no issues
     that would make it impossible for docs to get collected and assembled
 
+  - [`:gettext`] - verifies that your POT files are up to date with the current state of the codebase
+
   - [`:mix_audit`] - scans the project's Mix dependencies for known Elixir security vulnerabilities
     based on a GitHub-sourced list of security advisories
 
@@ -253,6 +255,7 @@ defmodule Mix.Tasks.Check do
   [`:doctor`]: https://github.com/akoutmos/doctor
   [`:ex_doc`]: https://hexdocs.pm/ex_doc
   [`:ex_unit`]: https://hexdocs.pm/ex_unit
+  [`:gettext`]: https://hexdocs.pm/gettext
   [`:formatter`]: https://hexdocs.pm/mix/Mix.Tasks.Format.html
   [`:npm_test`]: https://docs.npmjs.com/cli/test.html
   [`:sobelow`]: https://hexdocs.pm/sobelow

test/ex_check/project_cases/config_and_scripts_test.exs

@@ -75,6 +75,7 @@ defmodule ExCheck.ProjectCases.ConfigAndScriptsTest do
     end
 
     refute plain_output =~ "credo skipped due to missing package credo"
+    refute plain_output =~ "gettext skipped due to missing package credo"
     assert plain_output =~ "my_mix_task success"
     assert plain_output =~ "my_elixir_script success"
     assert plain_output =~ "my_shell_script success"

test/ex_check/project_cases/external_tools_test.exs

@@ -6,7 +6,7 @@ defmodule ExCheck.ProjectCases.ExternalToolsTest do
       if Version.match?(System.version(), "< 1.10.0") do
         [:ex_check, :credo, :doctor, :sobelow, :mix_audit]
       else
-        [:ex_check, :credo, :doctor, :ex_doc, :sobelow, :mix_audit]
+        [:ex_check, :credo, :doctor, :ex_doc, :gettext, :sobelow, :mix_audit]
       end
 
     set_mix_deps(project_dir, tools)
@@ -17,6 +17,7 @@ defmodule ExCheck.ProjectCases.ExternalToolsTest do
     assert output =~ "formatter success"
     assert output =~ "ex_unit success"
     assert output =~ "credo success"
+    assert output =~ "gettext success"
 
     if Version.match?(System.version(), ">= 1.8.0") do
       assert output =~ "doctor success"

test/ex_check/project_cases/manifest_test.exs

@@ -31,6 +31,7 @@ defmodule ExCheck.ProjectCases.ManifestTest do
       SKIP doctor
       SKIP sobelow
       SKIP ex_doc
+      SKIP gettext
       SKIP mix_audit
       SKIP dialyzer
       """