diff options
author | tfarina@chromium.org <tfarina@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-01-02 21:45:03 +0000 |
---|---|---|
committer | tfarina@chromium.org <tfarina@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-01-02 21:45:03 +0000 |
commit | 78e7b4e1b928fa69f672be3c743df6d6c3ecbced (patch) | |
tree | 5bc67e556d8a790498b4700d0aa37495c89bfcd5 /tests/PathOpsBoundsTest.cpp | |
parent | dab032a4ba4a93bff49adae064912e4bb6583784 (diff) |
Get rid of DEFINE_TESTCLASS_SHORT() macro.
Instead tests should be written using DEF_TEST() macro, which is much
nicer and simplifies the process of setting up an unit test.
BUG=None
TEST=skpskgr_test, pathops_unittest
R=mtklein@google.com
Review URL: https://codereview.chromium.org/117863005
git-svn-id: http://skia.googlecode.com/svn/trunk@12870 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tests/PathOpsBoundsTest.cpp')
-rw-r--r-- | tests/PathOpsBoundsTest.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/PathOpsBoundsTest.cpp b/tests/PathOpsBoundsTest.cpp index 3d6091cb14..0e813d2a8e 100644 --- a/tests/PathOpsBoundsTest.cpp +++ b/tests/PathOpsBoundsTest.cpp @@ -7,6 +7,7 @@ #include "PathOpsTestCommon.h" #include "SkPathOpsBounds.h" #include "Test.h" +#include "TestClassDef.h" static const SkRect sectTests[][2] = { {{2, 0, 4, 1}, {4, 0, 6, 1}}, @@ -46,7 +47,7 @@ static const SkRect notReallyEmpty[] = { static const size_t notEmptyTestsCount = SK_ARRAY_COUNT(notReallyEmpty); -static void PathOpsBoundsTest(skiatest::Reporter* reporter) { +DEF_TEST(PathOpsBounds, reporter) { for (size_t index = 0; index < sectTestsCount; ++index) { const SkPathOpsBounds& bounds1 = static_cast<const SkPathOpsBounds&>(sectTests[index][0]); SkASSERT(ValidBounds(bounds1)); @@ -108,6 +109,3 @@ static void PathOpsBoundsTest(skiatest::Reporter* reporter) { (bounds.*SetCurveBounds[3])(curvePts); REPORTER_ASSERT(reporter, bounds == expected); } - -#include "TestClassDef.h" -DEFINE_TESTCLASS_SHORT(PathOpsBoundsTest) |