Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

test(ngSanitize): enhance test regarding the xlink:href attribute #9770

Closed
wants to merge 1 commit into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Oct 24, 2014

(Related to #9751 (comment) .)

@caitp
Copy link
Contributor

caitp commented Oct 24, 2014

Please rebase this, it should pass now i think

@caitp
Copy link
Contributor

caitp commented Oct 24, 2014

(I guess you had the idea from my comment on the other PR before I did :p)

@@ -246,6 +246,15 @@ describe('HTML', function() {
it('should sanitize SVG xmlns:xlink attribute values', function() {
expectHTML('<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"><a xmlns:xhref="javascript:alert()"></a></svg>')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test doesn't really fit into this test case. We should either leave it out, or move it to a test case named accordingly.

@gkalpak gkalpak deleted the $sanitize-enhance-test branch October 24, 2014 06:20
christianliebel pushed a commit to christianliebel/angular.js that referenced this pull request Oct 25, 2014
SVG attribute test splitted, descriptions changed accordingly

Related to angular#9770
petebacondarwin pushed a commit that referenced this pull request Oct 25, 2014
SVG attribute test splitted, descriptions changed accordingly

Related to #9770
Closes #9787
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants