Add e2e tests for icons

[jelbourn]

No description provided.

[jesselpalmer]

Hey @jelbourn, Do you mind if I take a shot? Or are you already working on this?

[jelbourn]

Go ahead, not currently assigned

[jesselpalmer]

How would you feel about leaving e2es out for icons since there is no real behavior to test, at least for the time being. It looks like all of the cases are covered via unit testing. Personally, I always try to limit e2es in favor of unit testing since they tend to be slow and flakey. Here is a good article on the subject if you are interested: http://googletesting.blogspot.com/2015/04/just-say-no-to-more-end-to-end-tests.html. If you feel differently I will go ahead and write tests if you would like.

[jelbourn]

At the very least there should be some smoke-level e2e tests.

[jesselpalmer]

Gotcha

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}