diff options
author | Ms2ger <ms2ger@gmail.com> | 2014-12-27 14:47:44 +0100 |
---|---|---|
committer | Ms2ger <ms2ger@gmail.com> | 2014-12-27 16:14:32 +0100 |
commit | aa18c84f0c73bd67774a091087f963db8aa57d20 (patch) | |
tree | 7b357ecaab16d641bb49dcf8699ad76d15a51d6d | |
parent | 9857ea26cb9ee262654bee97322dbbf373486bff (diff) | |
download | servo-aa18c84f0c73bd67774a091087f963db8aa57d20.tar.gz servo-aa18c84f0c73bd67774a091087f963db8aa57d20.zip |
Re-enable two transform_text unit tests.
Note that I had to change some expectations to make them pass.
-rw-r--r-- | components/gfx/text/util.rs | 90 |
1 files changed, 49 insertions, 41 deletions
diff --git a/components/gfx/text/util.rs b/components/gfx/text/util.rs index d39119935d7..67c17c2a066 100644 --- a/components/gfx/text/util.rs +++ b/components/gfx/text/util.rs @@ -196,63 +196,71 @@ fn test_transform_discard_newline() { } } -/* FIXME: Fix and re-enable #[test] fn test_transform_compress_whitespace() { - let test_strs : ~[String] = ~[" foo bar".into_string(), - "foo bar ".into_string(), - "foo\n bar".into_string(), - "foo \nbar".into_string(), - " foo bar \nbaz".into_string(), - "foo bar baz".into_string(), - "foobarbaz\n\n".into_string()]; - - let oracle_strs : ~[String] = ~[" foo bar".into_string(), - "foo bar ".into_string(), - "foo\n bar".into_string(), - "foo \nbar".into_string(), - " foo bar \nbaz".into_string(), - "foo bar baz".into_string(), - "foobarbaz\n\n".into_string()]; + let test_strs = vec![ + " foo bar", + "foo bar ", + "foo\n bar", + "foo \nbar", + " foo bar \nbaz", + "foo bar baz", + "foobarbaz\n\n", + ]; + + let oracle_strs = vec![ + "foo bar", + "foo bar ", + "foo\n bar", + "foo \nbar", + "foo bar \nbaz", + "foo bar baz", + "foobarbaz\n\n", + ]; assert_eq!(test_strs.len(), oracle_strs.len()); - let mode = CompressWhitespace; + let mode = CompressionMode::CompressWhitespace; - for i in range(0, test_strs.len()) { - let mut new_line_pos = ~[]; - let (trimmed_str, _out) = transform_text(test_strs[i], mode, true, &mut new_line_pos); - assert_eq!(&trimmed_str, &oracle_strs[i]) + for (test, oracle) in test_strs.iter().zip(oracle_strs.iter()) { + let mut new_line_pos = vec![]; + let mut trimmed_str = String::new(); + transform_text(*test, mode, true, &mut trimmed_str, &mut new_line_pos); + assert_eq!(&*trimmed_str, *oracle) } } #[test] fn test_transform_compress_whitespace_newline() { - let test_strs : ~[String] = ~[" foo bar".into_string(), - "foo bar ".into_string(), - "foo\n bar".into_string(), - "foo \nbar".into_string(), - " foo bar \nbaz".into_string(), - "foo bar baz".into_string(), - "foobarbaz\n\n".into_string()]; - - let oracle_strs : ~[String] = ~["foo bar".into_string(), - "foo bar ".into_string(), - "foo bar".into_string(), - "foo bar".into_string(), - " foo bar baz".into_string(), - "foo bar baz".into_string(), - "foobarbaz ".into_string()]; + let test_strs = vec![ + " foo bar", + "foo bar ", + "foo\n bar", + "foo \nbar", + " foo bar \nbaz", + "foo bar baz", + "foobarbaz\n\n" + ]; + + let oracle_strs = vec![ + "foo bar", + "foo bar ", + "foo bar", + "foo bar", + "foo bar baz", + "foo bar baz", + "foobarbaz " + ]; assert_eq!(test_strs.len(), oracle_strs.len()); let mode = CompressionMode::CompressWhitespaceNewline; - for i in range(0, test_strs.len()) { - let mut new_line_pos = ~[]; - let (trimmed_str, _out) = transform_text(test_strs[i], mode, true, &mut new_line_pos); - assert_eq!(&trimmed_str, &oracle_strs[i]) + for (test, oracle) in test_strs.iter().zip(oracle_strs.iter()) { + let mut new_line_pos = vec![]; + let mut trimmed_str = String::new(); + transform_text(*test, mode, true, &mut trimmed_str, &mut new_line_pos); + assert_eq!(&*trimmed_str, *oracle) } } -*/ #[test] fn test_transform_compress_whitespace_newline_no_incoming() { |