Skip to content

Commit

Permalink
test_web.py: fix test_GET_DIRECTORY_html_filenode_encoding for Twiste…
Browse files Browse the repository at this point in the history
…d==12.3.0. fixes #1922

twisted.web.html.escape was used to produce html-encoded string (to then look
it up in "value" attribute), but behavior of that function has changed between
Twisted 12.2.0 (simple custom implementation) and 12.3.0 (imported from stdlib
cgi module).
  • Loading branch information
mk-fg committed Feb 27, 2013
1 parent cdf1752 commit 0f49987
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/allmydata/test/test_web.py
@@ -1,12 +1,12 @@
import os.path, re, urllib, time
import os.path, re, urllib, time, cgi
import simplejson
from StringIO import StringIO

from twisted.application import service
from twisted.trial import unittest
from twisted.internet import defer, reactor
from twisted.internet.task import Clock
from twisted.web import client, error, http, html
from twisted.web import client, error, http
from twisted.python import failure, log

from foolscap.api import fireEventually, flushEventualQueue
Expand Down Expand Up @@ -324,8 +324,8 @@ def _then(res):
self._htmlname_raw = self._htmlname_unicode.encode('utf-8')
self._htmlname_urlencoded = urllib.quote(self._htmlname_raw, '')
self._htmlname_escaped = escapeToXML(self._htmlname_raw)
self._htmlname_escaped_attr = html.escape(self._htmlname_raw)
self._htmlname_escaped_double = escapeToXML(html.escape(self._htmlname_raw))
self._htmlname_escaped_attr = cgi.escape(self._htmlname_raw, quote=True)
self._htmlname_escaped_double = escapeToXML(cgi.escape(self._htmlname_raw, quote=True))
self.HTMLNAME_CONTENTS, n, self._htmlname_txt_uri = self.makefile(0)
foo.set_uri(self._htmlname_unicode, self._htmlname_txt_uri, self._htmlname_txt_uri)

Expand Down

0 comments on commit 0f49987

Please sign in to comment.