FrostyHec opened a new pull request, #718:
URL: https://github.com/apache/hugegraph-toolchain/pull/718
## Purpose of the PR
This pull request introduces a new option to bypass required argument checks
when building a `HugeClientBuilder`, allowing for more flexible client
initialization. The main changes include updating constructors and builder
methods to support the `skipRequiredChecks` flag, modifying validation logic,
and adding unit tests to verify this new behavior.
## Main Changes
Enhancements to client builder flexibility:
* Added a new `skipRequiredChecks` boolean parameter to the
`HugeClientBuilder` constructor and the corresponding `HugeClient.builder`
static method, allowing users to bypass mandatory checks for `url` and `graph`
arguments when needed.
* Updated validation logic in both the constructor and the `build()` method
of `HugeClientBuilder` to conditionally skip argument checks based on the
`skipRequiredChecks` flag.
Testing:
* Introduced a new test class `HugeClientBuilderTest` with tests to ensure
that the builder behaves correctly with and without the `skipRequiredChecks`
flag, including cases where required arguments are missing.
## Verifying these changes
<!-- Please pick the proper options below -->
- [ ] Trivial rework / code cleanup without any test coverage. (No Need)
- [ ] Already covered by existing tests, such as *(please modify tests
here)*.
- [x] Need tests and can be verified as follows:
- add new tests.
## Does this PR potentially affect the following parts?
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [x] Nope
- [ ] Dependencies (add/update license info) <!-- Don't forget to
add/update the info in "LICENSE" & "NOTICE" files (both in root & dist module)
-->
- [ ] Modify configurations
- [ ] The public API
- [ ] Other affects (typed here)
## Documentation Status
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `Doc - TODO` <!-- Your PR changes impact docs and you will update
later -->
- [ ] `Doc - Done` <!-- Related docs have been already added or updated -->
- [x] `Doc - No Need` <!-- Your PR changes don't impact/need docs -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
