Skip to content

Commit

Permalink
(#1335) Add todo to investigate overflow in SumOf
Browse files Browse the repository at this point in the history
  • Loading branch information
@victornoel
victornoel committed May 16, 2021
1 parent 7a66067 commit 510666e
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/test/java/org/cactoos/number/SumOfTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,11 @@
* Test case for {@link SumOf}.
*
* @since 0.9
* @todo #1335:30min Following the rewrite of the Number implementations
* in cactoos, the three methods below prefixed by overflow where adapted
* because they started to give different results. The task is to investigate
* thoroughly what is a "correct" behaviour concerning overflow in the case
* of SumOf and adapt the tests and the code to make this obvious and clear.
* @checkstyle JavadocMethodCheck (500 lines)
* @checkstyle MagicNumberCheck (500 lines)
*/
Expand Down

1 comment on commit 510666e

@0pdd
Copy link
Collaborator

@0pdd 0pdd commented on 510666e Jun 12, 2021

Choose a reason for hiding this comment

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

Puzzle 1335-2793a937 discovered in src/test/java/org/cactoos/number/SumOfTest.java and submitted as #1617. Please, remember that the puzzle was not necessarily added in this particular commit. Maybe it was added earlier, but we discovered it only now.

Please sign in to comment.