Skip to content

Add unit tests for get_commit_timestamp_utc & name_hash - #595

Open
NikitaJangidBMW wants to merge 3 commits into
mainfrom
test/html_report_namehash-commit-timestamp
Open

Add unit tests for get_commit_timestamp_utc & name_hash#595
NikitaJangidBMW wants to merge 3 commits into
mainfrom
test/html_report_namehash-commit-timestamp

Conversation

@NikitaJangidBMW

Copy link
Copy Markdown
Contributor

test_html_report.py file

test(html_report): add unit tests for submodule timestamp and name_hash

  • Added test for get_commit_timestamp_utc() when commit is found only in submodule
  • Added parameterized tests for name_hash() covering empty string, ASCII, and German Unicode input
  • Verified consistency and uniqueness of MD5 hashes

…tml_report.py file

test(html_report): add unit tests for submodule timestamp and name_hash

- Added test for get_commit_timestamp_utc when commit is found only in submodule
- Added parameterized tests for name_hash covering empty string, ASCII, and German Unicode input
- Verified consistency and uniqueness of MD5 hashes
@NikitaJangidBMW
NikitaJangidBMW requested a review from a team as a code owner June 26, 2026 07:05
@kedarnn kedarnn added the internal Affects the CI, tests or refactorings only, not relevant to the end-user label Jul 6, 2026
@kedarnn
kedarnn force-pushed the test/html_report_namehash-commit-timestamp branch from dc03b6c to 2436afe Compare July 6, 2026 07:04
Comment thread tests_unit/lobster_html_report/test_html_report.py Outdated
Comment thread tests_unit/lobster_html_report/test_html_report.py Outdated
Comment thread tests_unit/lobster_html_report/test_html_report.py Outdated
@NikitaJangidBMW
NikitaJangidBMW force-pushed the test/html_report_namehash-commit-timestamp branch from 2436afe to 3fd21c6 Compare July 7, 2026 13:23
@NikitaJangidBMW
NikitaJangidBMW requested a review from kedarnn July 24, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Affects the CI, tests or refactorings only, not relevant to the end-user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants