Commit cb74f421 authored by Jérome Perrin's avatar Jérome Perrin

testXHTML: display jsl errors from all scripts

do not limit to the first script with errors
parent 200bb85b
...@@ -189,6 +189,7 @@ class TestXHTMLMixin(ERP5TypeTestCase): ...@@ -189,6 +189,7 @@ class TestXHTMLMixin(ERP5TypeTestCase):
portal_skins_path = self.portal.getId() + '/portal_skins/' portal_skins_path = self.portal.getId() + '/portal_skins/'
args = ('jsl', '-stdin', '-nologo', '-nosummary', '-conf', args = ('jsl', '-stdin', '-nologo', '-nosummary', '-conf',
os.path.join(os.path.dirname(__file__), 'jsl.conf')) os.path.join(os.path.dirname(__file__), 'jsl.conf'))
error_list = []
for path in path_list: for path in path_list:
check_path = portal_skins_path + path check_path = portal_skins_path + path
body = self.publish(check_path).getBody() body = self.publish(check_path).getBody()
...@@ -197,7 +198,11 @@ class TestXHTMLMixin(ERP5TypeTestCase): ...@@ -197,7 +198,11 @@ class TestXHTMLMixin(ERP5TypeTestCase):
close_fds=True).communicate(body) close_fds=True).communicate(body)
except OSError, e: except OSError, e:
raise OSError, '%r\n%r' % (os.environ, e) raise OSError, '%r\n%r' % (os.environ, e)
self.assertEqual(stdout, '', 'jsl result of %s : %s' % (check_path, stdout)) if stdout:
error_list.append((check_path, stdout))
if error_list:
message = '\n'.join(["%s\n%s\n" % error for error in error_list])
self.fail(message)
def test_html_file(self): def test_html_file(self):
path_list = os.environ.get('CGI_PATH', path_list = os.environ.get('CGI_PATH',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment