[ZLI][UX]: Support = in parameters, and better error messages (#24)

[zhiningli]

This PR addresses issue #24 by adding support for the --flag=value syntax in the ZLI command-line interface.

Changes:

• tools/arg/arg_parser.cpp: Updated processLongFlag to parse = syntax, extracting flag name and value separately. Added validation to reject =value for boolean flags that don't accept values.
• tools/arg/tests/arg_parser_test.cpp : Added 9 unit tests covering various scenarios.
• tools/arg/tests/BUCK : Added Buck build configuration for the new tests.
• tools/arg/CMakeLists.txt : Added CMake test configuration using gtest.
• cli/zli.cpp : Updated usage message to document the = syntax.

Before:

zli compress --profile=u32 tmp# Error: Unknown option: --profile=u32

After:

zli compress --profile=u32 tmp# ✅ Works correctly!

Both syntaxes are now supported:

--profile=u32 (new)
--profile u32 (existing)

Test Plan:

Run unit tests with CMake:

# Configure with tests enabled
cmake -B build -DOPENZL_BUILD_TESTS=ON -DOPENZL_BUILD_CLI=ON# Build the arg parser testscmake --build build --target test_arg --config Release# Run the tests./build/tools/arg/Release/test_arg.exe   # Windows

Expected output:

$ cd C:\Users\Zhining\source\repos\openzl\openzl; .\build\tools\arg\Release\test_arg.exe
Running main() from C:\Users\Zhining\source\repos\openzl\openzl\build\_deps\googletest-src\googletest\src\gtest_main.cc
[==========] Running 9 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 9 tests from ArgParserTest
[ RUN      ] ArgParserTest.LongFlagWithEqualsSign
[       OK ] ArgParserTest.LongFlagWithEqualsSign (0 ms)
[ RUN      ] ArgParserTest.LongFlagWithSpaceSeparatedValue
[       OK ] ArgParserTest.LongFlagWithSpaceSeparatedValue (0 ms)
[ RUN      ] ArgParserTest.BothSyntaxesProduceSameResult
[       OK ] ArgParserTest.BothSyntaxesProduceSameResult (0 ms)
[ RUN      ] ArgParserTest.EqualSyntaxWithMultipleValues
[       OK ] ArgParserTest.EqualSyntaxWithMultipleValues (0 ms)
[ RUN      ] ArgParserTest.MixedSyntax
[       OK ] ArgParserTest.MixedSyntax (0 ms)
[ RUN      ] ArgParserTest.BooleanFlagWithoutValue
[       OK ] ArgParserTest.BooleanFlagWithoutValue (0 ms)
[ RUN      ] ArgParserTest.ValueContainingSpecialCharacters
[       OK ] ArgParserTest.ValueContainingSpecialCharacters (0 ms)
[ RUN      ] ArgParserTest.BooleanFlagWithEqualsValueThrows
[       OK ] ArgParserTest.BooleanFlagWithEqualsValueThrows (0 ms)
[ RUN      ] ArgParserTest.UnknownFlagWithEquals
[       OK ] ArgParserTest.UnknownFlagWithEquals (0 ms)
[----------] 9 tests from ArgParserTest (0 ms total)

[----------] Global test environment tear-down
[==========] 9 tests from 1 test suite ran. (0 ms total)
[  PASSED  ] 9 tests.
$ 

[meta-codesync]

@zhiningli has imported this pull request. If you are a Meta employee, you can view this in D89766246.

[meta-codesync]

@zhiningli has exported this pull request. If you are a Meta employee, you can view the originating Diff in D89766246.