Ah another improvement for excluding could be an `—except` option for `mix do` then you could exclude apps that way
On Sat, 21 Oct 2023, at 1:45 PM, Daniel Kukula wrote: > Thanks Jon. It's helpful. My main concern with adding the tags was to exclude > single app from tests. > When I'm changing something in one of the childs and everything is finally > green in this one I wan't to run tests for all the rest to check if nothing > depends on my changes. (it takes few minutes to run all the tests in this > particular app and I ran the tests just a minute ago) > On Saturday, October 21, 2023 at 11:19:30 AM UTC+1 ma...@jonrowe.co.uk wrote: >> __ >> For 1) this behaviour is actually the same as non umbrella apps, if you run >> `mix test` you will run the default test path in both both cases (for an >> umbrella this runs all apps tests). If you do `mix test some_path` in a >> normal app it will similarly misbehave (in my phoenix app `mix test .` for >> example tries to run tests from node_modules 😂) >> >> For 2) are you aware of `mix cmd?` you can filter that to apps with `mix do >> —app app cmd` so you can test specific apps with: `mix do —app app1 —app >> app2 cmd mix test` >> >> I actually run `mix cmd mix test` on umbrella apps quite regularly because >> unlike `mix test` it runs the tests in each app seperately which can be >> helpful to isolate dependency issues. >> >> Hope that helps >> Cheers >> Jon >> >> On Thu, 19 Oct 2023, at 2:34 PM, Daniel Kukula wrote: >>> Hi all, I have 2 proposals when running exunit inside umbrella: >>> >>> 1) Automatically add `/test` at the end of path running tests in umbrella: >>> `mix test apps/child_app` >>> currently this does not run tests, but it also does not print any warnings >>> about not tests found - I have to specify the test folder in the child app: >>> `mix test apps/child_app/test` >>> but I don't have to do this when running tests in standard app - this just >>> works: >>> `mix test` >>> >>> 2) Add umbrella app name as a tag so this works and runs only tests in >>> child app >>> `mix test --include child_app` >>> `mix test --exclude child_app` >>> This may be backwards incompatible for some of us so maybe >>> `include_app`, `exclude_app` modifiers make sense ? >>> >>> >>> -- >>> You received this message because you are subscribed to the Google Groups >>> "elixir-lang-core" group. >>> To unsubscribe from this group and stop receiving emails from it, send an >>> email to elixir-lang-co...@googlegroups.com. >>> To view this discussion on the web visit >>> https://groups.google.com/d/msgid/elixir-lang-core/396c12a4-c34b-4859-a4d7-1b4d4efa8aa3n%40googlegroups.com >>> >>> <https://groups.google.com/d/msgid/elixir-lang-core/396c12a4-c34b-4859-a4d7-1b4d4efa8aa3n%40googlegroups.com?utm_medium=email&utm_source=footer>. >> > > > -- > You received this message because you are subscribed to the Google Groups > "elixir-lang-core" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to elixir-lang-core+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/elixir-lang-core/acfa7531-ad6b-499d-9be4-feea7e4a7dc6n%40googlegroups.com > > <https://groups.google.com/d/msgid/elixir-lang-core/acfa7531-ad6b-499d-9be4-feea7e4a7dc6n%40googlegroups.com?utm_medium=email&utm_source=footer>. -- You received this message because you are subscribed to the Google Groups "elixir-lang-core" group. To unsubscribe from this group and stop receiving emails from it, send an email to elixir-lang-core+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/elixir-lang-core/bb60f3f0-63b8-4418-ac58-a736c873e8b4%40app.fastmail.com.
