Beta

Spy Agent

22e-imad
Description
Loading description...
Cryptography
Algorithms
Strings
Puzzles
View
AllIssues3QuestionsSuggestionsShow Resolved
  • Please sign in or sign up to leave a comment.
  • saudiGuy Avatar

    Issue :

    • python new test frame work is required.

    Suggestion :

    • remove unecceary comments in full tests suite
    • assertion messages for tests
  • Blind4Basics Avatar

    ("...yet another Caesar cipher kata...")

    Well, not so "yet another". At least there is a twist. ;)

    Some issues, though:

    • as said below: don't use test.expect (some readings here) (note: seemed to be already addressed?)
    • you have to provide some tests with an odd length (both in the fixed aand the sample tests)
    • lengths of the inputs in the random tests are ridiculously big!! There's no point to this, so you should reduce their length to something like 100 or 200 characters max (100 would be far enough, IMO).
    • e-imad Avatar
      • For the sample test cases i modified some of them to have an odd number.
      • I rededuced the size of input and increased the number of random tests.
      • Thank you for you input.
      Issue marked resolved by e-imad 8 years ago
  • Avanta Avatar

    This comment has been hidden.