Skip to content

[ZEPPELIN-6485] Add unit tests for PropertiesUtil - #5493

Open
uommou wants to merge 1 commit into
apache:masterfrom
uommou:fix/ZEPPELIN-6485
Open

uommou wants to merge 1 commit into
apache:masterfrom
uommou:fix/ZEPPELIN-6485

Conversation

@uommou

@uommou uommou commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

What is this PR for?

This PR adds unit tests for PropertiesUtil in the zeppelin-interpreter module.

The tests cover the behavior of getString, getLong, and getInt, including missing properties, empty and whitespace-only values, successful numeric parsing, parsing failures, and numeric boundary values.

No production code is changed.

What type of PR is it?

Improvement

Todos

  • Add tests for missing properties
  • Add tests for empty and whitespace-only values
  • Add tests for valid positive and negative numeric values
  • Add tests for numeric boundary values
  • Add tests for invalid and out-of-range numeric values

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6485

How should this be tested?

Run the new unit tests:

./mvnw test -pl zeppelin-interpreter -Dtest=PropertiesUtilTest

Expected result:

Tests run: 6, Failures: 0, Errors: 0, Skipped: 0
BUILD SUCCESS

Screenshots (if appropriate)

This PR does not change the user interface.

Questions:

  • Does the license files need to update? No.
  • Is there breaking changes for older versions? No.
  • Does this needs documentation? No.

@voidmatcha voidmatcha left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants