[#197] Fix #197: Organize Tests (#238)

test/otp_formatter_SUITE.erl

@@ -10,7 +10,8 @@ test_app(_Config) ->
     {ok, State1} =
         rebar3_format:init(
             rebar_state:new()),
-    Files = {files, ["src/*.app.src", "src/*.sh", "src/*.erl", "include/*.hrl"]},
+    Files =
+        {files, ["src/*.app.src", "src/*.sh", "src/*.erl", "src/*/*.erl", "include/*.hrl"]},
     Formatter = {formatter, otp_formatter},
     IgnoredFiles =
         case string:to_integer(

test/test_app_SUITE.erl

@@ -10,7 +10,8 @@ test_app(_Config) ->
     {ok, State1} =
         rebar3_format:init(
             rebar_state:new()),
-    Files = {files, ["src/*.app.src", "src/*.sh", "src/*.erl", "include/*.hrl"]},
+    Files =
+        {files, ["src/*.app.src", "src/*.sh", "src/*.erl", "src/*/*.erl", "include/*.hrl"]},
     IgnoredFiles =
         case string:to_integer(
                  erlang:system_info(otp_release))

test_app/after/src/brackets_ignore.erl

@@ -1,3 +1,4 @@
+%% This module should not be formatted since it's in the list of ignored modules
 -module(brackets_ignore).
 
 -format(#{paper => 50, inline_items => all}).

test_app/after/src/break.erl

@@ -2,18 +2,20 @@
 
 -compile(export_all).
 
+%% receive...after...end is never a one-liner
 break_on_receive() ->
     receive
-        {this, will, be, indented} ->
-            but_this:is(short)
+        x ->
+            y
     after 1000 ->
-        this:too(should, go, below)
+        z
     end.
 
+%% try...of...after...end is never a one-liner
 break_on_try() ->
-    try this:short(statement) of
-        {things, cant, be} ->
-            {a, <<"one-liner">>}
+    try w of
+        x ->
+            y
     after
-        this:neither()
+        z
     end.

test_app/rebar.config

@@ -19,7 +19,7 @@
 {project_plugins, [rebar3_format]}.
 
 {format, [
-    {files, ["src/*.erl", "include/*.hrl"]},
+    {files, ["src/*.erl", "src/*/*.erl", "include/*.hrl"]},
     {ignore, ["src/*_ignore.erl", "src/ignored_file_config.erl"]},
     {options, #{dummy => option}}
 ]}.

test_app/src/brackets_ignore.erl

@@ -1,3 +1,4 @@
+%% This module should not be formatted since it's in the list of ignored modules
 -module(brackets_ignore).
 
 -format(#{paper => 50, inline_items => all}).

test_app/src/break.erl

@@ -2,13 +2,11 @@
 
 -compile(export_all).
 
+%% receive...after...end is never a one-liner
 break_on_receive() ->
-    receive
-      {this, will, be, indented} -> but_this:is(short)
-      after 1000 -> this:too(should, go, below)
-    end.
+    receive x -> y after 1000 -> z end.
 
+%% try...of...after...end is never a one-liner
 break_on_try() ->
-    try this:short(statement) of
-      {things, cant, be} -> {a, <<"one-liner">>} after this:neither() end.
+    try w of x -> y after z end.