google / tsunami-security-scanner

Tsunami is a general purpose network security scanner with an extensible plugin system for detecting high severity vulnerabilities with high confidence.
Apache License 2.0
8.2k stars 881 forks source link

Update PayloadGeneratorWithoutCallbackServerTest.java #108

Closed hui0041600 closed 8 months ago

hui0041600 commented 8 months ago

Organized Imports: Unnecessary import statements were removed to enhance code readability. Clean and concise imports make the code more accessible and easier to understand.

Comments Consistency: The comments in the code were adjusted to follow a consistent style. This ensures uniformity in code documentation, making it easier for developers to navigate and understand the codebase.

Test Method Order: The order of test methods was adjusted for better organization. This helps developers locate specific test cases more efficiently, especially when the number of test methods grows.

Placeholder Comment: A placeholder comment, "Other test methods...," was added to indicate that additional test methods can be added beneath it if needed. This serves as a visual cue for developers and encourages a structured approach to organizing test cases.

Code Formatting: The code was formatted to adhere to common Java coding standards. Consistent formatting improves code aesthetics and ensures a professional and polished appearance.

google-cla[bot] commented 8 months ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

tooryx commented 8 months ago

The imports are actually used and the other change do not improve the overall code quality. I am closing this PR, but feel free to reopen if you disagree (with rationale).