r215639 - Split a large unit-test, so that it doesn't exceed stack frame size in certain test environments

[Alexander Kornienko]

Author: alexfh
Date: Thu Aug 14 08:07:35 2014
New Revision: 215639

URL: http://llvm.org/viewvc/llvm-project?rev=215639&view=rev
Log:
Split a large unit-test, so that it doesn't exceed stack frame size in certain 
test environments

Modified:
    cfe/trunk/unittests/Format/FormatTest.cpp

Modified: cfe/trunk/unittests/Format/FormatTest.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Format/FormatTest.cpp?rev=215639&r1=215638&r2=215639&view=diff
==============================================================================
--- cfe/trunk/unittests/Format/FormatTest.cpp (original)
+++ cfe/trunk/unittests/Format/FormatTest.cpp Thu Aug 14 08:07:35 2014
@@ -8225,11 +8225,6 @@ TEST_F(FormatTest, GetsCorrectBasedOnSty
   EXPECT_ALL_STYLES_EQUAL(Styles);
 }
 
-#define CHECK_PARSE(TEXT, FIELD, VALUE)                                        
\
-  EXPECT_NE(VALUE, Style.FIELD);                                               
\
-  EXPECT_EQ(0, parseConfiguration(TEXT, &Style).value());                      
\
-  EXPECT_EQ(VALUE, Style.FIELD)
-
 #define CHECK_PARSE_BOOL(FIELD)                                                
\
   Style.FIELD = false;                                                         
\
   EXPECT_EQ(0, parseConfiguration(#FIELD ": true", &Style).value());           
\
@@ -8237,7 +8232,7 @@ TEST_F(FormatTest, GetsCorrectBasedOnSty
   EXPECT_EQ(0, parseConfiguration(#FIELD ": false", &Style).value());          
\
   EXPECT_FALSE(Style.FIELD);
 
-TEST_F(FormatTest, ParsesConfiguration) {
+TEST_F(FormatTest, ParsesConfigurationBools) {
   FormatStyle Style = {};
   Style.Language = FormatStyle::LK_Cpp;
   CHECK_PARSE_BOOL(AlignEscapedNewlinesLeft);
@@ -8266,7 +8261,18 @@ TEST_F(FormatTest, ParsesConfiguration)
   CHECK_PARSE_BOOL(SpacesInContainerLiterals);
   CHECK_PARSE_BOOL(SpacesInCStyleCastParentheses);
   CHECK_PARSE_BOOL(SpaceBeforeAssignmentOperators);
+}
+
+#undef CHECK_PARSE_BOOL
 
+#define CHECK_PARSE(TEXT, FIELD, VALUE)                                        
\
+  EXPECT_NE(VALUE, Style.FIELD);                                               
\
+  EXPECT_EQ(0, parseConfiguration(TEXT, &Style).value());                      
\
+  EXPECT_EQ(VALUE, Style.FIELD)
+
+TEST_F(FormatTest, ParsesConfiguration) {
+  FormatStyle Style = {};
+  Style.Language = FormatStyle::LK_Cpp;
   CHECK_PARSE("AccessModifierOffset: -1234", AccessModifierOffset, -1234);
   CHECK_PARSE("ConstructorInitializerIndentWidth: 1234",
               ConstructorInitializerIndentWidth, 1234u);
@@ -8462,7 +8468,6 @@ TEST_F(FormatTest, ParsesConfigurationWi
 }
 
 #undef CHECK_PARSE
-#undef CHECK_PARSE_BOOL
 
 TEST_F(FormatTest, UsesLanguageForBasedOnStyle) {
   FormatStyle Style = {};


_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits