{"action":"created","issue":{"url":"https://api.github.com/repos/exercism/exercism.io/issues/2432","labels_url":"https://api.github.com/repos/exercism/exercism.io/issues/2432/labels{/name}","comments_url":"https://api.github.com/repos/exercism/exercism.io/issues/2432/comments","events_url":"https://api.github.com/repos/exercism/exercism.io/issues/2432/events","html_url":"https://github.com/exercism/exercism.io/issues/2432","id":91797354,"number":2432,"title":"Crypto Square tests for Java version conflict with tests for JS version","user":{"login":"matthewmorgan","id":2478531,"avatar_url":"https://avatars.githubusercontent.com/u/2478531?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewmorgan","html_url":"https://github.com/matthewmorgan","followers_url":"https://api.github.com/users/matthewmorgan/followers","following_url":"https://api.github.com/users/matthewmorgan/following{/other_user}","gists_url":"https://api.github.com/users/matthewmorgan/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewmorgan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewmorgan/subscriptions","organizations_url":"https://api.github.com/users/matthewmorgan/orgs","repos_url":"https://api.github.com/users/matthewmorgan/repos","events_url":"https://api.github.com/users/matthewmorgan/events{/privacy}","received_events_url":"https://api.github.com/users/matthewmorgan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-29T12:59:28Z","updated_at":"2015-06-29T13:59:41Z","closed_at":null,"body":"Hi,\r\n\r\nI'm not sure if this is a bug or if I am missing something different about the exercise requirements, but the test suite for the Java version of this problem seems to be expecting different segment sizes than the JS version.  EG\r\n```\r\n//Java test\r\n    @Test\r\n    public void normalizedCipherNotExactlyDivisibleBy5SpillsIntoSmallerSegment() {\r\n        Crypto crypto = new Crypto(\"Madness, and then illumination.\");\r\n        String expectedOutput = \"msemo aanin dninn dlaet ltshu i\";\r\n\r\n        assertEquals(expectedOutput, crypto.getNormalizedCipherText());\r\n    }\r\n```\r\nyields:\r\n```\r\njava.lang.AssertionError: \r\nExpected :msemo aanin dninn dlaet ltshu i\r\nActual   :[msemoa, anindn, inndla, etltsh, ui]\r\n```\r\n\r\nBut the JS test is: \r\n```\r\n//JS test\r\n  xit(\"normalized cipher text\",function() {\r\n    var crypto = new Crypto('Madness, and then illumination.');\r\n    expect(crypto.normalizeCiphertext()).toEqual('msemoa anindn inndla etltsh ui');\r\n  });\r\n```\r\nNote that the 'expected' in the JS test is identical to the result that failed in the Java test, and that the expected segment sizes differ in the two tests.  The README is a little vague, but illustrates that for non-perfect squares, the square size should be R x C, with C>R.  This is in agreement with the JS version tests, but in conflict with the Java version tests.\r\n\r\nThanks!\r\n\r\nEdited: I do see the output type is an array instead of a string, but still, the segment length is mismatched between the tests."},"comment":{"url":"https://api.github.com/repos/exercism/exercism.io/issues/comments/116672961","html_url":"https://github.com/exercism/exercism.io/issues/2432#issuecomment-116672961","issue_url":"https://api.github.com/repos/exercism/exercism.io/issues/2432","id":116672961,"user":{"login":"jtigger","id":470176,"avatar_url":"https://avatars.githubusercontent.com/u/470176?v=3","gravatar_id":"","url":"https://api.github.com/users/jtigger","html_url":"https://github.com/jtigger","followers_url":"https://api.github.com/users/jtigger/followers","following_url":"https://api.github.com/users/jtigger/following{/other_user}","gists_url":"https://api.github.com/users/jtigger/gists{/gist_id}","starred_url":"https://api.github.com/users/jtigger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtigger/subscriptions","organizations_url":"https://api.github.com/users/jtigger/orgs","repos_url":"https://api.github.com/users/jtigger/repos","events_url":"https://api.github.com/users/jtigger/events{/privacy}","received_events_url":"https://api.github.com/users/jtigger/received_events","type":"User","site_admin":false},"created_at":"2015-06-29T13:59:41Z","updated_at":"2015-06-29T13:59:41Z","body":"@kytrinyx I would definitely move to the language-specific track so as to make it clear it's in that track's backlog to prioritize against other stuff happening there (defects like this should be addressed immediately).\n\nLooks like we need (at least) another for the Ruby track, too."}}