Raathigesh / majestic

⚡ Zero config GUI for Jest
MIT License
7.49k stars 173 forks source link

Majestic stops when a `it` is a `todo` #212

Closed C0ZEN closed 4 years ago

C0ZEN commented 4 years ago

Is this a bug report or a feature request?

Majestic does not handle the .todo for a it and just stops. This is annoying and I do not see any usefulness of this.

Version Info

Reproduction Repo

StackTrace:

return finalText.concat(`\`\$\{${path.node.arguments[0].expressions[currentExpressionIndex++].name}\}\``);
TypeError: Cannot read property 'name' of undefined
rbarbazz commented 4 years ago

After doing some testing it appears that any empty template will fail, for instance: test(``) or it(``).

I can make a PR to address this.

C0ZEN commented 4 years ago

Still waiting for the PR to be merged. @rbarbazz get the job done a while ago.