search

exercism / coffeescript

Exercism exercises in CoffeeScript.
https://exercism.org/tracks/coffeescript
MIT License
19 stars 50 forks source link

fixed protein-translation.spec.coffee #266

Closed ladokp closed 5 months ago

ladokp commented 5 months ago

rewrote the last test to check for an exception and not a string like seen in other exercise tests.

Related to #259

github-actions[bot] commented 5 months ago

Hello. Thanks for opening a PR on Exercism 🙂

We ask that all changes to Exercism are discussed on our Community Forum before being opened on GitHub. To enforce this, we automatically close all PRs that are submitted. That doesn't mean your PR is rejected but that we want the initial discussion about it to happen on our forum where a wide range of key contributors across the Exercism ecosystem can weigh in.

You can use this link to copy this into a new topic on the forum. If we decide the PR is appropriate, we'll reopen it and continue with it, so please don't delete your local branch.

If you're interested in learning more about this auto-responder, please read this blog post.

Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it.

ladokp commented 5 months ago

@ErikSchierboom: Opened a new PR with my current try to solve the broken test for protein-translation.

ErikSchierboom commented 5 months ago

Are we using an outdated version of the matchers library or something?

ladokp commented 5 months ago

Maybe, let's stick to the "toThrow()" function for now? It's also used in other exercises.

ErikSchierboom commented 5 months ago

I guess we have to

ErikSchierboom commented 5 months ago

Thanks!

ladokp commented 5 months ago

Thank you too, I hope the exercise is solvable now.