python new test framework is required. updated in this fork

C#:

• No random tests

• CollectionAssert.AreEqual or equivalent assertions without the need to convert arrays to strings should be used

Ok suic. I have approved now. It wasn't showig the full details earlier. You are right, all the tests are there.

Normally the author of the original kata is the approver.

I'm sorry, I don't understand your two comments.

1. The details section is shared between translations. I didn't modify the original description.
2. Ad tests: The randomized tests cover all the currencies. There's one test for each currency but in shuffled order.

Regards,

suic

Thanks Suic. I don't know Python. Am I the only person who can approve it?

Also

• the detail section is missing some important details describing the currency strength
• Could do with few more tests for the full tests.

Python translation kumited.

Please review and approve.

Regards,

suic

Great job!

Hi Donald,

I have made the changes and tests are now run randomly. That was a good one. Thanks.

Great! Very good!!

Hi Donald

The "Allow Contributors" was always checked. Not sure why it won't let you edit. If it is still not possible then please share the code and I will add it :-).

@Donald: Better tell/write him, how to do it himself. It is his kata! Perhaps write him an example or describe how to do it on his own. He wants to learn it.

Hi @kmlprtsng,

It seems like you unchecked the "Allow Contributors" box so I cannot edit your Kata. Can you please tick that box again so I can help you add randomized tests?

Thanks a lot :D

Cheers,
donaldsebleung

You could do the 8 test-calls in random-order.

Hi Donald

I am not sure which randomize tests can be added. Can you help me with that and I'll try to see if I can get them in? Thanks.

Thank you Steffen. Really appreciate your kind words.