Skip to content

Conversation

@Alokzh
Copy link
Collaborator

@Alokzh Alokzh commented Jul 2, 2025

Changes:

  • Methods added: do:, collect:, select:, detect:, anySatisfy:, asArray, copy, clear, addAll:, removeAll:, elementsFrom:to:, elementsGreaterThan:, elementsLessThan:, findMin, findMax, predecessorOf:, successorOf:, remove:, remove:ifAbsent:, preOrderDo:, postOrderDo:, validateBSTProperty
  • Added comprehensive tests covering all new methods
  • Added proper description for all classes

@coveralls
Copy link

coveralls commented Jul 2, 2025

Pull Request Test Coverage Report for Build 16015258858

Details

  • 299 of 316 (94.62%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+9.8%) to 89.849%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Containers-BinarySearchTree/CTBinarySearchTree.class.st 133 138 96.38%
src/Containers-BinarySearchTree/CTBSTNillNode.class.st 37 49 75.51%
Totals Coverage Status
Change from base Build 15801971608: 9.8%
Covered Lines: 416
Relevant Lines: 463

💛 - Coveralls

@Alokzh Alokzh merged commit 90820ce into pharo-containers:main Jul 2, 2025
12 checks passed
@Alokzh Alokzh deleted the binary-search-tree-implementation branch July 2, 2025 03:20
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