Apply suggestions from code review

Author: josevalim

lib/elixir/lib/option_parser.ex

@@ -900,7 +900,7 @@ defmodule OptionParser do
     type = get_type(option, opts, types)
 
     with :regex <- type,
-         {:error, {reason, position}} <- Regex.compile(value) do
+         {:error, {reason, position}} <- Regex.compile(value, "u") do
       "#{option} : Invalid regular expression #{inspect(value)}: #{reason} at position #{position}"
     else
       _ -> "#{option} : Expected type #{type}, got #{inspect(value)}"

lib/elixir/test/elixir/option_parser_test.exs

@@ -409,20 +409,6 @@ defmodule OptionParserTest do
 
       assert OptionParser.parse(["--pattern", "(?invalid)"], opts) ==
                {[], [], [{"--pattern", "(?invalid)"}]}
-
-      # Valid regex patterns should compile successfully
-      assert {[pattern: regex], [], []} =
-               OptionParser.parse(["--pattern", "valid.*pattern"], opts)
-
-      assert Regex.match?(regex, "valid test pattern")
-      refute Regex.match?(regex, "invalid")
-
-      # Test complex valid patterns
-      assert {[pattern: regex], [], []} =
-               OptionParser.parse(["--pattern", "\\d{3}-\\d{2}-\\d{4}"], opts)
-
-      assert Regex.match?(regex, "123-45-6789")
-      refute Regex.match?(regex, "abc-de-fghi")
     end
 
     test "parse! raises an exception for invalid regex patterns" do

lib/mix/lib/mix/tasks/test.ex

@@ -157,7 +157,7 @@ defmodule Mix.Tasks.Test do
     * `--max-requires` - sets the maximum number of test files to compile in parallel.
       Setting this to 1 will compile test files sequentially.
 
-    * `-t`, `--name-pattern` *(since v1.19.0)* - only run tests which the name match
+    * `-n`, `--name-pattern` *(since v1.19.0)* - only run tests which the name matching
       the given regular expression
 
     * `--no-archives-check` - does not check archives