Ad
  • Custom User Avatar

    how does the tester expected them to be spelled?

  • Custom User Avatar

    I don't know whether it was intentional but the "encript" and "decript" function names are not spelled correctly.

  • Custom User Avatar

    Also note that the test cases work, when run separately by hitting the "TEST" button. When I define "alphabet = []" and hit the "ATTEMPT" button, the encript and chained fixed tests fail and the decript random tests also fail. In all cases however, your kata is reporting that the decript tests are successful.

  • Custom User Avatar

    Based on the current rating (you are looking to get at least 80 % satisfaction rating), it might be worthwhile to incorporate a small feature that adds a bit of extra flair to the challenge.

  • Custom User Avatar

    As long as there's agreement on what the "cool" letters are. :-) I certainly don't want to do all 26.

    And there is already a kata that asks you to print multiple ascii art letters horizontally. It doesn't have the scaling factor, but that doesn't seem worth adding.

  • Custom User Avatar

    that would be a world better than expecting to one day have literally dozens of near duplicate kata

  • Custom User Avatar
    ReferenceError: alphabet is not defined
    
        at /workspace/node/test.js:25:32
        at Object.<anonymous> (/workspace/node/test.js:105:3)
        at Module._compile (node:internal/modules/cjs/loader:1159:14)
        at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
        at Module.load (node:internal/modules/cjs/loader:1037:32)
        at Module._load (node:internal/modules/cjs/loader:878:12)
        at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29)
        at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
        at async Promise.all (index 0)
        at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
        at async importModuleDynamicallyWrapper (node:internal/vm/module:438:15)
        at async formattedImport (/workspace/node/node_modules/.pnpm/mocha@10.2.0/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
        at async exports.requireOrImport (/workspace/node/node_modules/.pnpm/mocha@10.2.0/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
        at async exports.loadFilesAsync (/workspace/node/node_modules/.pnpm/mocha@10.2.0/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
        at async singleRun (/workspace/node/node_modules/.pnpm/mocha@10.2.0/node_modules/mocha/lib/cli/run-helpers.js:125:3)
        at async exports.handler (/workspace/node/node_modules/.pnpm/mocha@10.2.0/node_modules/mocha/lib/cli/run.js:370:5)
    
  • Custom User Avatar

    Can't you combine all cool letters into one kata, the current one?

  • Custom User Avatar

    Actually, I was going to do M next, which is more interesting. But B could be done. How about this?

    BBBB

    B B

    BBBB

    B B

    BBBB

  • Custom User Avatar

    If you could identify what was confusing, or have any suggestions how to clarify it, please share.

  • Custom User Avatar

    Now let's create a kata for B... oh, wait

  • Custom User Avatar

    Pretty confusing description.

  • Custom User Avatar

    Thanks - fixed. In the first version of the kata the size of the figure was 4xsize, but I changed it later to 5xsize. Forgot to update the figures.

  • Custom User Avatar

    The width of the "C" is 4 in description but 5 in tests.

  • Custom User Avatar
  • Loading more items...