1314497 - Convert intl/lwbrk/tests/TestLineBreak.cpp to a gtest

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Nicholas Nethercote [inactive]]

intl/lwbrk/tests/TestLineBreak.cpp is a horrible old test that also happens to be a GeckoCppUnitTest and so its presence blocks bug 1313141. I will change it to a gtest.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: (part 1) - Convert intl/lwbrk/tests/TestLineBreak.cpp to a gtest

This commit does the following.

- Converts the test to a gtest, including renaming the directory.

- Factors out the duplicated checking code into a separate Check() function.

- Avoids printing any output unless a failure occurs. (The gtest harness prints
  out minimal output, like the name of the current test.)

- Removes SampleWordBreakUsage(), which prints some output but doesn't actually
  test anything, and so isn't useful.

Comment 2

[Nicholas Nethercote [inactive]]

Attachment: (part 2) - Reformat intl/lwbrk/tests/TestLineBreak.cpp

This patch makes no functional changes.

Comment 4

[Masatoshi Kimura [:emk]]

Comment on attachment 8806575 [details] [diff] [review]
(part 1) - Convert intl/lwbrk/tests/TestLineBreak.cpp to a gtest

Review of attachment 8806575 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the following comment resolved.

::: intl/lwbrk/gtest/TestLineBreak.cpp
@@ -314,5 @@
> -static const char wb3[] =  "ernationali";
> -static const char wb4[] =  "zation work.";
> -
> -static const char* wb[] = {wb0,wb1,wb2,wb3,wb4};
> -void SampleWordBreakUsage()

Could you leave this function (and sub-functions) intact? I'll convert them to automated tests later on.

Originally TestLineBreak.cpp was a manual test. Bug 754176 changed the file to a CppUnitTest, but the bug didn't convert this function. It's the reason why this function only output the result.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b4e3d21a447d
(part 1) - Convert intl/lwbrk/tests/TestLineBreak.cpp to a gtest. r=emk.
https://hg.mozilla.org/integration/mozilla-inbound/rev/50d2cc0d4cbd
(part 2) - Reformat intl/lwbrk/tests/TestLineBreak.cpp.

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/b4e3d21a447d
https://hg.mozilla.org/mozilla-central/rev/50d2cc0d4cbd