From 0deec44e7dfc2c23bf9cae9af01fc68f3ed72c42 Mon Sep 17 00:00:00 2001 From: alecpl <alec@alec.pl> Date: Fri, 06 May 2011 04:46:22 -0400 Subject: [PATCH] - Fixed handling of folder with name "0" in folder selector --- tests/mailfunc.php | 35 ++++++++++++++++++++++++++++++----- 1 files changed, 30 insertions(+), 5 deletions(-) diff --git a/tests/mailfunc.php b/tests/mailfunc.php index cc26f77..9d70bef 100644 --- a/tests/mailfunc.php +++ b/tests/mailfunc.php @@ -19,7 +19,7 @@ $RCMAIL->imap_init(false); $IMAP = $RCMAIL->imap; - require_once 'steps/mail/func.inc'; + require_once INSTALL_PATH . 'program/steps/mail/func.inc'; $GLOBALS['EMAIL_ADDRESS_PATTERN'] = $EMAIL_ADDRESS_PATTERN; } @@ -66,9 +66,8 @@ $this->assertPattern('/<style [^>]+>/', $html2, "Allow styles in safe mode"); $this->assertPattern('#src="http://evilsite.net/mailings/ex3.jpg"#', $html2, "Allow external images in HTML (safe mode)"); $this->assertPattern("#url\('?http://evilsite.net/newsletter/image/bg/bg-64.jpg'?\)#", $html2, "Allow external images in CSS (safe mode)"); - - $css = '<link rel="stylesheet" type="text/css" href="?_task=utils&_action=modcss&u='.urlencode('http://anysite.net/styles/mail.css').'&c=foo"'; - $this->assertPattern('#'.preg_quote($css).'#', $html2, "Filter external styleseehts with bin/modcss.php"); + $css = '<link rel="stylesheet" .+_u=tmp-[a-z0-9]+\.css.+_action=modcss'; + $this->assertPattern('#'.$css.'#Ui', $html2, "Filter (anonymized) external styleseehts with utils/modcss.inc"); } /** @@ -122,7 +121,7 @@ function test_mailto() { $part = $this->get_html_part('src/mailto.txt'); - + // render HTML in normal mode $html = rcmail_html4inline(rcmail_print_body($part, array('safe' => false)), 'foo'); @@ -132,4 +131,30 @@ $this->assertPattern('|'.preg_quote($mailto, '|').'|', $html, "Extended mailto links"); } + /** + * Test the elimination of HTML comments + */ + function test_html_comments() + { + $part = $this->get_html_part('src/htmlcom.txt'); + $washed = rcmail_print_body($part, array('safe' => true)); + + // #1487759 + $this->assertPattern('|<p>test1</p>|', $washed, "Buggy HTML comments"); + // but conditional comments (<!--[if ...) should be removed + $this->assertNoPattern('|<p>test2</p>|', $washed, "Conditional HTML comments"); + } + + /** + * Test URI base resolving in HTML messages + */ + function test_resolve_base() + { + $html = file_get_contents(TESTS_DIR . 'src/htmlbase.txt'); + $html = rcmail_resolve_base($html); + + $this->assertPattern('|src="http://alec\.pl/dir/img1\.gif"|', $html, "URI base resolving [1]"); + $this->assertPattern('|src="http://alec\.pl/dir/img2\.gif"|', $html, "URI base resolving [2]"); + $this->assertPattern('|src="http://alec\.pl/img3\.gif"|', $html, "URI base resolving [3]"); + } } -- Gitblit v1.9.1