See https://status.codewars.com/incidents/ymg0y1r172sl

We don't consider this an issue. It's a trade-off for providing a more natural experience to the users.

In fact, there are many ways to cheat the system because of the flexibility we provide and it's well known.

We can prevent this specific case, but you can always find ways around it until we lock the system up enough to harm the user experience. So our policy is to only fix something that can accidentally cause the tests to pass and this one doesn't apply.

Note that this "attack" only harms the "attacker" by ruining their learning opportunity. You'll learn more by solving a kata yourself.
We also ban users who heavily cheats or plagiarizes because we don't want them in our community.

Can you improve the test case names? Too much redundancy.

Maybe change the describe to describe("Roman numerals encoder for 1 <= n < 4000", function() {, and remove everything outside the parens for test case names.

The sample tests have accounts.

Can you also add a semiolon to the solution setup so it compiles?

 string greeting = ""; // Change this

                     ^

Please make the following changes:

• Remove accounts parameter in describe('HelloWorld', function(accounts) {
• Move tests and randomNum (const randomNum = Math.floor(Math.random() * tests.length);) inside the random test case
• Rewrite tests using async/await to match other kata

I forgot we need to set Chai's truncateThreshold to 0 for assert.deepEqual to avoid output like expected [ Array(13) ] to deeply equal [ Array(13) ].

const { assert, config } = require("chai");
config.truncateThreshold = 0;

Should be good to merge after this.

GetVolumeOfCubiod  old
GetVolumeOfCuboid  new
              ^^

Please change countAndConvert helper to

  // Note: We must convert `BigNumber` to `Number` to use `assert.deepEqual`.
  const countAndConvert = async (c, x, n) => {
    return (await c.countBy(x, n)).map(b => b.toNumber());
  };

• Pass the contract instance instead of deploying it each time
• Use .map(b => b.toNumber()) method instead of .toString().split(',').map(Number)
• Remove unnecessary await

Use it like

  it('should pass basic tests', async () => {    
    const c = await loadFixture(deployFixture);
    assert.deepEqual(await countAndConvert(c,1,5), [1,2,3,4,5])  
  });

Also, please fix the globals (i, x, n) in the random tests.

Please change the random test assertion to

assert.strictEqual(await d.checkForFactor(b, f), exp, `base = ${b}, factor = ${f}`);

and remove console.log to avoid unnecessary outputs.