Commitc35d84c

committed

Fix#217: Fully remove element in removeChild in etree treebuilder (#259)

This adds a test here because we still fail the upstream one, asour implementation of AAA is outdated.

1 parent2d37673 commitc35d84cCopy full SHA for c35d84c

File tree

-1

lines changed

-1

lines changed

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`from .importsupport# noqa`
`8`	`8`
`9`	`9`	`fromhtml5lib.constantsimportnamespaces`
`10`		`-fromhtml5libimportparse,HTMLParser`
	`10`	`+fromhtml5libimportparse,parseFragment,HTMLParser`
`11`	`11`
`12`	`12`
`13`	`13`	`# tests that aren't autogenerated from text files`
`@@ -88,3 +88,8 @@ def test_debug_log():`
`88`	`88`	`expected[i]=tuple(log)`
`89`	`89`
`90`	`90`	`assertparser.log==expected`
	`91`	`+`
	`92`	`+`
	`93`	`+deftest_no_duplicate_clone():`
	`94`	`+frag=parseFragment("<b><em><foo><foob><fooc><aside></b></em>")`
	`95`	`+assertlen(frag)==2`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -100,6 +100,7 @@ def insertBefore(self, node, refNode):`
`100`	`100`	`node.parent=self`
`101`	`101`
`102`	`102`	`defremoveChild(self,node):`
	`103`	`+self._childNodes.remove(node)`
`103`	`104`	`self._element.remove(node._element)`
`104`	`105`	`node.parent=None`
`105`	`106`

Comments

(0)