Utils.get_app_name/1 now understands the application name expressed via module attribute.

Author: Aleksei Matiushkin

lib/sobelow/utils.ex

@@ -912,10 +912,26 @@ defmodule Sobelow.Utils do
       ast = ast(filepath)
       {_, project_block} = Macro.prewalk(ast, [], &extract_project_block/2)
       {_, app_name} = Macro.prewalk(project_block, [], &extract_app_name/2)
-      if is_atom(app_name), do: Atom.to_string(app_name), else: app_name
+      binarize_app_name(app_name, ast)
     end
   end
 
+  defp binarize_app_name(app_name, _) when is_binary(app_name), do: app_name
+  defp binarize_app_name(app_name, _) when is_atom(app_name), do: Atom.to_string(app_name)
+
+  defp binarize_app_name({:@, _, [{module_attribute, _, _}]}, ast) do
+    ast
+    |> Macro.prewalk([], fn
+      {:@, _, [{^module_attribute, _, [name]}]}, [] ->
+        {[], name}
+
+      ast, acc ->
+        {ast, acc}
+    end)
+    |> elem(1)
+    |> binarize_app_name(ast)
+  end
+
   defp extract_project_block({:def, _, [{:project, _, _}, [do: block]]} = ast, _) do
     {ast, block}
   end

test/fixtures/utils/mix.exs

@@ -0,0 +1,10 @@
+defmodule FooBar.Mixfile do
+  use Mix.Project
+
+  @app :foo_bar
+  def project do
+    [
+      app: @app
+    ]
+  end
+end

test/utils_test.exs

@@ -0,0 +1,7 @@
+defmodule SobelowTest.UtilsTest do
+  use ExUnit.Case
+
+  test "Utils.get_app_name/1 understands module attributes" do
+    assert Sobelow.Utils.get_app_name("./test/fixtures/utils/mix.exs") == "foo_bar"
+  end
+end