addMockModules permits duplicate module names?

[kbaltrinic]

I have not experimented with this but I was looking through the protractor code base and I see what looks like a bug at protractor.js Line 841 and following. It appears entirely possible to add two mock modules with the same name, have them both applied to the browser and then need to call removeMockModule twice with that name to remove them both. Am I reading this right? Is this correct behavior?

[kbaltrinic]

Looking at this more, there is also a bug in removeMockModule such that if you pass in a module name that does not exist, it does not validate validate the argument. The result being that it will incorrectly remove the most recently added module.

[kbaltrinic]

So at this point I have written unit tests that confirm both behaviors malfunction as I suspected and have submitted a pull request containing the confirming tests and associated fixes. Someone may want to review my unit tests. I think there may be a better approach.