Skip to content
Open
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/splitInteger.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,18 @@ const splitInteger = require('./splitInteger');

test(`should split a number into equal parts
if a value is divisible by a numberOfParts`, () => {

expect(splitInteger(6, 2)).toEqual([3, 3]);
});

test(`should return a part equals to a value
when splitting into 1 part`, () => {

expect(splitInteger(10, 1)).toEqual([10]);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

While this test is valid, the task description specifically requires a test for splitInteger(8, 1). Please update this test to use the example provided in the requirements.

});

test('should sort parts ascending if they are not equal', () => {

expect(splitInteger(17, 4)).toEqual([4, 4, 4, 5]);
});

test('should add zeros if value < numberOfParts', () => {

expect(splitInteger(4, 7)).toEqual([0, 0, 0, 1, 1, 1, 1]);
});
Comment on lines 19 to 21
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a good test for an edge case! However, the task description requires a specific test case for splitInteger(32, 6) which is currently missing. Please make sure to add a test for that example to meet all the requirements.