WIP: feat: minimal implementation of possible bdd test suite#492
WIP: feat: minimal implementation of possible bdd test suite#492kimadactyl wants to merge 1 commit intomainfrom
Conversation
Deploying the-trans-dimension with
|
| Latest commit: |
2a5371a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://ff0bbfa1.the-trans-dimension.pages.dev |
| Branch Preview URL: | https://add-bdd-tests.the-trans-dimension.pages.dev |
|
I worry it might introduce another "thing to maintain" but am not opposed to trying it out and assessing the benefits after a period of time rather than hard no. In terms of testing, I think the most important aspects are ensuring the displays of filtered data are working as expected. |
mateusfpleite
left a comment
There was a problem hiding this comment.
When trying to test the PR linked above, I found it difficult to test the features in a more meaningful way due to some limitations in the Elm-pages architecture. In addition to making it clearer when the acceptance criteria are being met, I think this type of test can make it easier to test more complex flows like the filtering mentioned by @katjam, so I think it's worth a try!
|
My main concern with this approach is I'm unsure how to do meaningful testing that shows the API input vs "in browser" output. In Ruby I'd prob use something like Does anyone know what they're doing if we do go this route? Also is this the right JS idioms? |
A proof of concept for a possible cucumber / bdd test suite to compliment the elm unit tests.
What do people think to this direction? If you like it I can write out a proper issue ticket.