Add tests for embed feature

[crisner]

Fixes #313 (<=== Add issue number here)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• PR is descriptively titled 📑 and links the original issue above 🔗
• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
• code is in uniquely-named feature branch and has no merge conflicts 📁
• screenshots/GIFs are attached 📎 in case of UI updation
• ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

[crisner]

I have added some tests for the embed feature. I struggled a bit with this one as I am not used to writing tests. Looking forward to feedback.

[jywarren]

This looks great! One way to be extra confident in tests is to break them intentionally, and see if correct and useful errors result. Especially in Travis, where we'll be seeing them in the future! This looks good to merge but if you'd like to try breaking the tests to confirm they will report in Travis, go ahead and we can revert and merge after!

[crisner]

Great! I'll try breaking the tests to confirm how it reports on Travis.

[crisner]

I did as you suggested and Travis does report the errors when I push broken code. 🎉 :D

[sagarpreet-chadha]

Awesome!!!