Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add more units tests for Arrays category #38

Open
delphidabbler opened this issue Jan 7, 2025 · 2 comments
Open

Add more units tests for Arrays category #38

delphidabbler opened this issue Jan 7, 2025 · 2 comments
Assignees
@delphidabbler
Labels
completed Issue completed and committed to develop. To be closed on next release enhancement New feature or request
Milestone
Next Release

Comments

@delphidabbler
Copy link
Owner

Only TArrayUtils, ByteArraysEqual and ReverseByteArray currently have unit tests. More are needed.
@delphidabbler delphidabbler self-assigned this Jan 7, 2025
@delphidabbler delphidabbler added enhancement New feature or request accepted Issue will be actioned labels Jan 7, 2025
@github-project-automation github-project-automation bot moved this to Considering in Code Snippets Jan 7, 2025
@delphidabbler delphidabbler moved this from Considering to Accepted in Code Snippets Jan 7, 2025
@delphidabbler
Copy link
Owner Author

⚠️ Note that although ByteArraysEqual has unit tests, arrays.ini does not record that fact.

@delphidabbler delphidabbler added completed Issue completed and committed to develop. To be closed on next release and removed accepted Issue will be actioned labels Apr 5, 2025
@delphidabbler delphidabbler moved this from Accepted to Completed in Code Snippets Apr 5, 2025
@delphidabbler delphidabbler added this to the Next Release milestone Apr 5, 2025
delphidabbler added a commit that referenced this issue Apr 5, 2025
@delphidabbler
Merge branch 'feature/38-arrays-category-tests' into develop
34d945a 
Fixes #38
@delphidabbler
Copy link
Owner Author

Added unit tests for all snippets that didn't already have them and updated arrays.ini accordingly.

Implemented by merge commit 34d945a.

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

@delphidabbler delphidabbler

Labels
completed Issue completed and committed to develop. To be closed on next release enhancement New feature or request
Projects
Code Snippets
Status: Completed
Milestone
Next Release
Development

No branches or pull requests
1 participant
@delphidabbler