Description

• the arrays will be always a string.

  4 inputs given are string, not array, the terminology used is wrong...

• return the sum of arr1 and arr2, sum of arr3 and arr5 ...

  should be

  return the sum of first and second string's length, then multiply it by the sum of third and fourth string's length

• return the concat of the arrays, by (plus sign) and return the word capatalized

  should be

  return the concatenation of strings, then capitalize them

• The expected return type is not clearly specified --> tuples / arrays / ...

Test Cases

• No sample tests

• No random tests

• Function name should use snake_case

• Missing it block for fixed tests

Overall, this is not how to write a good bug-fixing kata, specifically where all the bugs are just spelling typos that do not represent any meaning in programming terms...

Also, this is your 5th bug fixing kata it seems, and previous ones all got retired, so I suggest you go through the documentation before coming up with new katas , make sure they comfront to the current standards

Self approved after 4 years :-)

Approved

• Why use a Wrapper class Integer whilst a primitive int can do the same thing?

• I suggest to add annotations for better name representation using @DisplayName and ordering using @Order

Forked and approved

Fork with following changes:

• Enabled Java 17.0 with JUnit5

• Made initial solution setup compile (also changed the method signature to be unanimous across languages)

• Scrutinized tests with annotations

• Removed reference solution

• Added useful assertion messages

Java translation

Fork with following changes:

• Enabled Java 17.0 with JUnit5

• Made initial solution setup compile and added input type

• Scrutinized tests with order and name annotations

• Added useful assertion messages

• Made random tests generate empty strings -> addressing this issue

• Added fixed tests to address this issue

Random tests of all languages do not generate empty strings

Rejected:

• NO random tests

• The imports are wrong in actual test suite -> which is why it says "Failed Tests"

import org.junit.Test;
import static org.junit.Assert.assertEquals;

• JUnit5 with Java 17.0 should be enabled

• Missing useful assertion messages (a MUST for an 8kyu to aid beginners in debugging)

• Initial solution setup has wrong class name, also it should return 0; to make it compile

Kindly read the translating guidelines before publishing more translations ^^

Approved

Approved

Fork with following changes:

• Scrutinized test with order and name annotations

• Added useful assertion messages

• Updated to Java 17.0

• Made initial solution setup compile, plus changed output type signature from String to String[]

• Made random tests generate 1 to 20 words, each containing 1 to 20 chars. (Original one only generates 1 word)

• Modified reference solution to be different from author's solution

• Discarded RepeatedTest annotation because it will show all assertions upon failure, and not just the first one, which makes it hard to debug with the long wall of text.

Fork with following changes:

• Scrutinized test with order and name annotations (Removed original logs)

• Added more useful assertion messages

• Updated to Java 17.0

• Made initial solution setup compile

• Made random tests generation range smaller to not always generate drink whisky