Ad
  • Custom User Avatar

    In the C# version of this kata, one of the hidden test cases (TestException) appears to access Current after MoveNext() has returned false. According to .NET conventions, this should result in an InvalidOperationException, but the kata description does not mention this requirement.

    This behavior is easy to overlook, especially since most other test cases pass without issue. It would be helpful if the kata description explicitly stated that:

    Accessing Current after MoveNext() returns false must throw an exception.

    Adding this note would make the expectations clearer and help prevent confusion for users implementing a valid IEnumerator in C#.

  • Custom User Avatar

    Python, probably others:

    • No random tests
    • Incomplete coverage of spec
  • Custom User Avatar

    Python new test framework required

  • Custom User Avatar

    the preloaded section emits warnings to STDERR:

    Note: src/main/java/ClassHelper.java uses unchecked or unsafe operations.

    This seems to be caused by this line:

    class SpecialFileManager extends ForwardingJavaFileManager {
    

    ForwardingJavaFileManager is a generic class:

    Class ForwardingJavaFileManager<M extends JavaFileManager>
    

    so the subclass SpecialFileManager is a raw type, hence the warnings. It should either be parameterized itself, or should extend a parameterized version of ForwardingJavaFileManager.

  • Custom User Avatar

    Ruby version kumited, please check and comment or approve.

  • Custom User Avatar

    Missing random tests in Java.

  • Custom User Avatar

    Missing random tests in C#.

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    typo in the 'Instruction set' section:

    Instructions for same ...

    should be:

    Instructions for Sam ...

  • Custom User Avatar

    TypeScript translation pending, feel free to leave feedback (if there is anything I can do better) or approve if you think it's ready.

  • Custom User Avatar

    I understand almost nothing from the description of this Kata and the examples given just can't make up for the 75% missing explanation. Can someone please explain me what I even have to do for this Kata?

  • Custom User Avatar

    in python, C# and Java, you'll have to manage null/None input too

    This is not true for Python. IMO, this requirement should be removed from other languages if it's actually present there.

  • Custom User Avatar

    "Connecting linked lists" does not feel quite right. To me, that sounds like solver has to connect two linked lists.

    Would you consider "Find shared node in linked lists" or something like that?

  • Custom User Avatar

    Python version is broken.

    Cases:
    'snakecase' should equal 'snake_case'
    'urlrouter' should equal 'url_router'
    'impl0test' should equal 'impl0_test'
    'copyonwritearraylist' should equal 'copy_on_write_array_list'
    'impl0test1inst0' should equal 'impl0_test1_inst0'

    How is it possible to implement an algorithm for these?

    In addition to that, these tests are not specified in the specs.

  • Custom User Avatar

    The error output string should be changed. I was recieving a lot of these types of errors when submitting tokens with integer (rather than the expected string) values:

    [Token(chars='12345', type='integer')] should equal [Token(chars='12345', type='integer')]
    

    This made it much more difficult to debug than it needed to be for what I think are obvious reasons.

    Cheers

  • Loading more items...