Unit tests

config_parser.cc

@@ -210,14 +210,20 @@ bool NginxConfigParser::Parse(std::istream* config_file, NginxConfig* config) {
         // Error.
         break;
       }
-      return true;
+      if( config_stack.size()==1){
+	return true;
+      }else 
+      {
+	break;
+      }
     } else {
       // Error. Unknown token.
       break;
     }
+
     last_token_type = token_type;
   }
-
+  
   printf ("Bad transition from %s to %s\n",
           TokenTypeAsString(last_token_type),
           TokenTypeAsString(token_type));

config_parser_test.cc

@@ -1,11 +1,51 @@
+#include <sstream>
+#include <string>
 #include "gtest/gtest.h"
 #include "config_parser.h"
 
-TEST(NginxConfigParserTest, SimpleConfig) {
+TEST(NginxConfigTest, ToString)
+{
+  NginxConfigStatement statement;
+  statement.tokens_.push_back("foo");
+  statement.tokens_.push_back("bar");
+  EXPECT_EQ(statement.ToString(0), "foo bar;\n");
+}
+//blah
+TEST(NginxConfigParserTest, SimpleConfig)
+{
+  std::string config_string = "foo bar;";
+  std::stringstream config_stream(config_string);
   NginxConfigParser parser;
-  NginxConfig out_config;
+  NginxConfig config;
+  EXPECT_TRUE(parser.Parse(&config_stream, &config));
+  EXPECT_EQ(1, config.statements_.size())
+    << "Config has one statements";
+    EXPECT_EQ("foo", config.statements_.at(0)->tokens_.at(0));
+}
 
-  bool success = parser.Parse("example_config", &out_config);
+class NginxStringConfigTest : public :: testing :: Test {
+protected:
+  bool ParseString(const std::string config_string) {
+    std::stringstream config_stream(config_string);
+    return parser_.Parse(&config_stream, &out_config_);
+  }
+  NginxConfigParser parser_;
+  NginxConfig out_config_;
+  //NginxConfigStatement statements;
+};
+
+TEST_F(NginxStringConfigTest, AnotherSimpleConfig) 
+{
+  EXPECT_TRUE(ParseString("foo bar;"));
+  EXPECT_EQ(1, out_config_.statements_.size())
+    <<"Config has one statements";
+   EXPECT_EQ("foo", out_config_.statements_.at(0)->tokens_.at(0));
+}
 
-  EXPECT_TRUE(success);
+TEST_F(NginxStringConfigTest, InvalidConfig) 
+{
+  EXPECT_FALSE(ParseString("foo bar"));
 }
+
+
+