1.2.5 angular-mocks copy change breaks tests using XML datatype

[matthughes]

Apparently the XML datatype in Chrome/Firefox (and others?) contains pointers to itself. The change here (f69dc16) copies the HTTP response.

If you are mocking an HTTP service with XML datatype as the return type, copy() never terminates.

[matthughes]

One solution would be to make angular.copy support recursive data structures. I made this change locally, but it is not enough.

If you just take this approach to copying an XMLDocument, you will be left with just the properties of the document, none of the methods that exist on the object's prototype. And then when someone tries to use one of those prototype methods, your test blows up.

So... we could add a special case for XMLDocument in copy and use cloneNode.

        } else if (source.cloneNode) {
          destination = source.cloneNode();
        }

But is that too specific to one type? Ideally users would be able to extend this copy method to support any type out there.

[matthughes]

Here are my fixes I described above:

matthughes@205637e