Format generated code only when clang-format is available

lib/unifex/interface_IO.ex

@@ -54,10 +54,19 @@ defmodule Unifex.InterfaceIO do
     File.write!("#{out_base_path}.c", source)
     File.write!("#{out_base_path}.cpp", source)
 
-    Mix.shell().cmd(
-      "clang-format -style=\"{BasedOnStyle: llvm, IndentWidth: 2}\" -i " <>
-        "#{out_base_path}.h #{out_base_path}.c #{out_base_path}.cpp"
-    )
+    # Format generated code only when clang-format is available
+    if Unifex.Utils.clang_format_installed?() do
+      System.cmd(
+        "clang-format",
+        [
+          "-style={BasedOnStyle: llvm, IndentWidth: 2}",
+          "-i",
+          "#{out_base_path}.h",
+          "#{out_base_path}.c",
+          "#{out_base_path}.cpp"
+        ]
+      )
+    end
 
     :ok
   end

lib/unifex/utils.ex

@@ -0,0 +1,6 @@
+defmodule Unifex.Utils do
+  @moduledoc false
+
+  @spec clang_format_installed?() :: boolean()
+  def clang_format_installed?(), do: System.find_executable("clang-format") != nil
+end

mix.exs

@@ -1,7 +1,7 @@
 defmodule Unifex.MixProject do
   use Mix.Project
 
-  @version "1.0.1"
+  @version "1.0.2"
   @github_url "https://github.com/membraneframework/unifex"
 
   def project do

test/test_helper.exs

@@ -1 +1,7 @@
-ExUnit.start()
+# clang-format is required to format the generated code
+# it won't match the reference files otherwise
+if Unifex.Utils.clang_format_installed?() do
+  ExUnit.start()
+else
+  raise "clang-format has to be installed on the system to run Unifex tests."
+end

test/unifex/integration_test.exs

@@ -44,10 +44,6 @@ defmodule Unifex.IntegrationTest do
   defp do_test_project(project, interface, language) do
     run_projects_tests(project, language)
 
-    # clang-format is required to format the generated code
-    # it won't match the reference files otherwise
-    assert {_output, 0} = System.cmd("clang-format", ~w(--version))
-
     test_common(project)
     test_particular(project, interface)
   end