Skip to content

Feat/add unit test to TechnitiumLibrary project#20

Closed
zbalkan wants to merge 3 commits intoTechnitiumSoftware:masterfrom
zbalkan:feat/add-unit-test-to-technitiumlibrary
Closed

Feat/add unit test to TechnitiumLibrary project#20
zbalkan wants to merge 3 commits intoTechnitiumSoftware:masterfrom
zbalkan:feat/add-unit-test-to-technitiumlibrary

Conversation

@zbalkan
Copy link
Copy Markdown
Contributor

@zbalkan zbalkan commented Dec 17, 2025

Started small with minimal changes and tests. I'll create PRs project by project after this is accepted.

@zbalkan
Copy link
Copy Markdown
Contributor Author

zbalkan commented Dec 18, 2025

I started small with minimal changes. Whenever an issue is merged, I’ll work on the next one. Until we got 90% coverage, I'll proceed. Then, I'll go on with DnsServer.

@ShreyasZare
Copy link
Copy Markdown
Member

Thanks for the PR. Will review it soon.

@zbalkan
Copy link
Copy Markdown
Contributor Author

zbalkan commented Dec 27, 2025

I have several unit test branches waiting for this review.

@ShreyasZare
Copy link
Copy Markdown
Member

I have several unit test branches waiting for this review.

I need some time since there are too many outstanding issues. So please bear with the delay.

@zbalkan
Copy link
Copy Markdown
Contributor Author

zbalkan commented Dec 29, 2025

I am fine with the timing and delays as long as we are clear it is okay to develop unit tests for robustness. I would not like to spend time on these if you don't want to proceed that way.

@ShreyasZare
Copy link
Copy Markdown
Member

I am fine with the timing and delays as long as we are clear it is okay to develop unit tests for robustness. I would not like to spend time on these if you don't want to proceed that way.

If I had issue with unit tests then I would have already mentioned it.

@zbalkan
Copy link
Copy Markdown
Contributor Author

zbalkan commented Jan 12, 2026

Closing this as I am deploying one PR per supported project.

@zbalkan zbalkan closed this Jan 12, 2026
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.

2 participants