Ok, republished with test import removed from preloaded and added to Sample Test Cases.

The only code that should go in preloaded is code that the solver might need in some way (or, to set up restrictions). There is no reason to have the import there, since the solver does not use it.

I think for a simple import it doesn't actually affect anything (either way it gets imported) but I think best practice is to import in the sample tests themselves.

I thought it was the right place to put it...
Are you suggesting to remove it from preloaded ?
Then I guess I should add it at the beginning of sample test cases.

Why are you importing codewars_test in preloaded? That should be in the sample tests I think.

Bump,

I'd like to contibute by translating more katas, some of them in less popular languages, but on the other hand, I dont want to pollute CW with low quality translations, so I'd like this one to be OK and approved first.

Since this is my first translation, it would not be legitimate to approve it myself.

Would someone be kind enough to re-check :).

Thank you for you quick answer, edited conforming to your advice and re-published.

Edit and Republish it, BTW you should use it blocks too.

Another question, for the modifications to take effect, should I fork the current translation or just edit it ?

Thank you for pointing this doc.
Here is the modified version :

• Corrected function name in Sample Test Cases.
• Grouped the tests conforming to the doc.

Check this doc: https://docs.codewars.com/languages/python/codewars-test

It's done for the python translation.

Nice one !