Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2832)

Unified Diff: breakpad/linux/line_reader_unittest.cc

Issue 115526: Linux: Add Breakpad client. (Closed)
Patch Set: ... Created 11 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « breakpad/linux/line_reader.h ('k') | breakpad/linux/linux_dumper.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: breakpad/linux/line_reader_unittest.cc
diff --git a/breakpad/linux/line_reader_unittest.cc b/breakpad/linux/line_reader_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0f9276494caecd176af06800dad7395cb3b61b68
--- /dev/null
+++ b/breakpad/linux/line_reader_unittest.cc
@@ -0,0 +1,184 @@
+// Copyright (c) 2009, Google Inc.
+// All rights reserved.
+//
+// Redistribution and use in source and binary forms, with or without
+// modification, are permitted provided that the following conditions are
+// met:
+//
+// * Redistributions of source code must retain the above copyright
+// notice, this list of conditions and the following disclaimer.
+// * Redistributions in binary form must reproduce the above
+// copyright notice, this list of conditions and the following disclaimer
+// in the documentation and/or other materials provided with the
+// distribution.
+// * Neither the name of Google Inc. nor the names of its
+// contributors may be used to endorse or promote products derived from
+// this software without specific prior written permission.
+//
+// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+#include <stdlib.h>
+#include <unistd.h>
+#include <sys/types.h>
+
+#include "breakpad/linux/line_reader.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using namespace google_breakpad;
+
+static int TemporaryFile() {
+ static const char templ[] = "/tmp/line-reader-unittest-XXXXXX";
+ char templ_copy[sizeof(templ)];
+ memcpy(templ_copy, templ, sizeof(templ));
+ const int fd = mkstemp(templ_copy);
+ if (fd >= 0)
+ unlink(templ_copy);
+
+ return fd;
+}
+
+namespace {
+typedef testing::Test LineReaderTest;
+}
+
+TEST(LineReaderTest, EmptyFile) {
+ const int fd = TemporaryFile();
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
+
+TEST(LineReaderTest, OneLineTerminated) {
+ const int fd = TemporaryFile();
+ write(fd, "a\n", 2);
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'a');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
+
+TEST(LineReaderTest, OneLine) {
+ const int fd = TemporaryFile();
+ write(fd, "a", 1);
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'a');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
+
+TEST(LineReaderTest, TwoLinesTerminated) {
+ const int fd = TemporaryFile();
+ write(fd, "a\nb\n", 4);
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'a');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'b');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
+
+TEST(LineReaderTest, TwoLines) {
+ const int fd = TemporaryFile();
+ write(fd, "a\nb", 3);
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'a');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, 1);
+ ASSERT_EQ(line[0], 'b');
+ ASSERT_EQ(line[1], 0);
+ reader.PopLine(len);
+
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
+
+TEST(LineReaderTest, MaxLength) {
+ const int fd = TemporaryFile();
+ char l[LineReader::kMaxLineLen - 1];
+ memset(l, 'a', sizeof(l));
+ write(fd, l, sizeof(l));
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_TRUE(reader.GetNextLine(&line, &len));
+ ASSERT_EQ(len, sizeof(l));
+ ASSERT_TRUE(memcmp(l, line, sizeof(l)) == 0);
+ ASSERT_EQ(line[len], 0);
+
+ close(fd);
+}
+
+TEST(LineReaderTest, TooLong) {
+ const int fd = TemporaryFile();
+ char l[LineReader::kMaxLineLen];
+ memset(l, 'a', sizeof(l));
+ write(fd, l, sizeof(l));
+ lseek(fd, 0, SEEK_SET);
+ LineReader reader(fd);
+
+ const char *line;
+ unsigned len;
+ ASSERT_FALSE(reader.GetNextLine(&line, &len));
+
+ close(fd);
+}
« no previous file with comments | « breakpad/linux/line_reader.h ('k') | breakpad/linux/linux_dumper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698