finished CClipboardTests

src/unittest/synergy/CClipboardTests.cpp

@@ -18,56 +18,385 @@
 #include <gtest/gtest.h>
 #include "CClipboard.h"
 
-TEST(CClipboardTests, empty_wasOpened_returnsTrue)
+TEST(CClipboardTests, empty_openCalled_returnsTrue)
 {
 	CClipboard clipboard;
-	clipboard.open(true);
+	clipboard.open(0);
 
 	bool actual = clipboard.empty();
 
 	EXPECT_EQ(true, actual);
 }
 
+TEST(CClipboardTests, empty_startWithFormats_hasReturnsFalse)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(CClipboard::kText, "synergy rocks!");
+
+	clipboard.empty();
+
+	bool actual = clipboard.has(CClipboard::kText);
+	EXPECT_EQ(false, actual);
+}
+
 TEST(CClipboardTests, add_newValue_valueWasStored)
 {
 	CClipboard clipboard;
-	clipboard.open(true);
+	clipboard.open(0);
-	IClipboard::EFormat format(IClipboard::kText);
 
-	clipboard.add(format, "Hello world!");
+	clipboard.add(IClipboard::kText, "synergy rocks!");
 
-	CString actual = clipboard.get(format);
+	CString actual = clipboard.get(IClipboard::kText);
-	EXPECT_EQ("Hello world!", actual);
+	EXPECT_EQ("synergy rocks!", actual);
 }
 
 TEST(CClipboardTests, add_replaceValue_valueWasStored)
 {
 	CClipboard clipboard;
-	clipboard.open(true);
+	clipboard.open(0);
-	IClipboard::EFormat format(IClipboard::kText);
 
-	clipboard.add(format, "Hello world!");
+	clipboard.add(IClipboard::kText, "synergy rocks!");
-	clipboard.add(format, "Goodbye world!");
+	clipboard.add(IClipboard::kText, "maxivista sucks"); // haha, just kidding.
 
-	CString actual = clipboard.get(format);
+	CString actual = clipboard.get(IClipboard::kText);
-	EXPECT_EQ("Goodbye world!", actual);
+	EXPECT_EQ("maxivista sucks", actual);
 }
 
-TEST(CClipboardTests, open_defaultState_returnsTrue)
+TEST(CClipboardTests, open_timeIsZero_returnsTrue)
 {
 	CClipboard clipboard;
 
-	bool actual = clipboard.open(true);
+	bool actual = clipboard.open(0);
 
 	EXPECT_EQ(true, actual);
 }
 
-TEST(CClipboardTests, getTime_defaultState_returnsZero)
+TEST(CClipboardTests, open_timeIsOne_returnsTrue)
 {
 	CClipboard clipboard;
 
+	bool actual = clipboard.open(1);
+
+	EXPECT_EQ(true, actual);
+}
+
+TEST(CClipboardTests, close_isOpen_noErrors)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+
+	clipboard.close();
+
+	// can't assert anything
+}
+
+TEST(CClipboardTests, getTime_openWithNoEmpty_returnsZero)
+{
+	CClipboard clipboard;
+	clipboard.open(1);
+
 	CClipboard::Time actual = clipboard.getTime();
 
-	CClipboard::Time expected(0);
+	EXPECT_EQ(0, actual);
-	EXPECT_EQ(expected, actual);
+}
+
+TEST(CClipboardTests, getTime_openAndEmpty_returnsOne)
+{
+	CClipboard clipboard;
+	clipboard.open(1);
+	clipboard.empty();
+
+	CClipboard::Time actual = clipboard.getTime();
+
+	EXPECT_EQ(1, actual);
+}
+
+TEST(CClipboardTests, has_formatAdded_returnsTrue)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks!");
+
+	bool actual = clipboard.has(IClipboard::kText);
+
+	EXPECT_EQ(true, actual);
+}
+
+TEST(CClipboardTests, has_noFormatAdded_returnsFalse)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+
+	bool actual = clipboard.has(IClipboard::kText);
+
+	EXPECT_EQ(false, actual);
+}
+
+TEST(CClipboardTests, get_formatNotAdded_returnsEmpty)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+
+	CString actual = clipboard.get(IClipboard::kText);
+
+	EXPECT_EQ("", actual);
+}
+
+TEST(CClipboardTests, get_formatAdded_returnsExpected)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks!");
+
+	CString actual = clipboard.get(IClipboard::kText);
+
+	EXPECT_EQ("synergy rocks!", actual);
+}
+
+TEST(CClipboardTests, marshall_addNotCalled_firstCharIsZero)
+{
+	CClipboard clipboard;
+
+	CString actual = clipboard.marshall();
+
+	// seems to return "\0\0\0\0" but EXPECT_EQ can't assert this,
+	// so instead, just assert that first char is '\0'.
+	EXPECT_EQ(0, (int)actual[0]);
+}
+
+TEST(CClipboardTests, marshall_addText_typeCharIsText)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks!");
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	// string contains other data, but 8th char should be kText.
+	EXPECT_EQ(IClipboard::kText, (int)actual[7]);
+}
+
+TEST(CClipboardTests, marshall_addTextSize14_sizeChar3Is14)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks!"); // 14 chars
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	EXPECT_EQ(14, (int)actual[11]);
+}
+
+// TODO: there's some integer -> char encoding going on here. i find it 
+// hard to believe that the clipboard is the only thing doing this. maybe
+// we should refactor this stuff out of the clipboard.
+TEST(CClipboardTests, marshall_addTextSizeOver255_sizeCharsAreValid)
+{
+	// 285 chars
+	CString data;
+	data.append("Synergy is Free and Open Source Software that lets you ");
+	data.append("easily share your mouse and keyboard between multiple ");
+	data.append("computers, where each computer has it's own display. No ");
+	data.append("special hardware is required, all you need is a local area ");
+	data.append("network. Synergy is supported on Windows, Mac OS X and Linux.");
+
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, data);
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	// 4 asserts here, but that's ok because we're really just asserting 1 
+	// thing. the 32-bit size value is split into 4 chars. if the size is 285
+	// (29 more than the 8-bit max size), the last char "rolls over" to 29 
+	// (this is caused by a bit-wise & on 0xff and 8-bit truncation). each 
+	// char before the last stores a bit-shifted version of the number, each 
+	// 1 more power than the last, which is done by bit-shifting [0] by 24, 
+	// [1] by 16, [2] by 8 ([3] is not bit-shifted).
+	EXPECT_EQ(0, actual[8]); // 285 >> 24 = 285 / (256^3) = 0
+	EXPECT_EQ(0, actual[9]); // 285 >> 16 = 285 / (256^2) = 0
+	EXPECT_EQ(1, actual[10]); // 285 >> 8 = 285 / (256^1) = 1(.11328125)
+	EXPECT_EQ(29, actual[11]); // 285 - 256 = 29
+}
+
+TEST(CClipboardTests, marshall_addHtml_typeCharIsHtml)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kHTML, "html sucks");
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	// string contains other data, but 8th char should be kHTML.
+	EXPECT_EQ(IClipboard::kHTML, (int)actual[7]);
+}
+
+TEST(CClipboardTests, marshall_addHtmlAndText_has2Formats)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks");
+	clipboard.add(IClipboard::kHTML, "html sucks");
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	// the number of formats is stored inside the first 4 chars.
+	// the writeUInt32 function right-aligns numbers in 4 chars,
+	// so if you right align 2, it will be "\0\0\0\2" in a string.
+	// we assert that the char at the 4th index is 2 (the number of
+	// formats that we've added).
+	EXPECT_EQ(2, (int)actual[3]);
+}
+
+TEST(CClipboardTests, marshall_addText_endsWithAdded)
+{
+	CClipboard clipboard;
+	clipboard.open(0);
+	clipboard.add(IClipboard::kText, "synergy rocks!");
+	clipboard.close();
+
+	CString actual = clipboard.marshall();
+
+	// string contains other data, but should end in the string we added.
+	EXPECT_EQ("synergy rocks!", actual.substr(12));
+}
+
+TEST(CClipboardTests, unmarshall_emptyData_hasTextIsFalse)
+{
+	CClipboard clipboard;
+
+	CString data;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0; // 0 formats added
+
+	clipboard.unmarshall(data, 0);
+
+	clipboard.open(0);
+	bool actual = clipboard.has(IClipboard::kText);
+	EXPECT_EQ(false, actual);
+}
+
+TEST(CClipboardTests, unmarshall_textSizeOver255_getTextIsValid)
+{
+	CClipboard clipboard;
+
+	// 285 chars
+	CString text;
+	text.append("Synergy is Free and Open Source Software that lets you ");
+	text.append("easily share your mouse and keyboard between multiple ");
+	text.append("computers, where each computer has it's own display. No ");
+	text.append("special hardware is required, all you need is a local area ");
+	text.append("network. Synergy is supported on Windows, Mac OS X and Linux.");
+
+	CString data;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)1; // 1 format added
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)IClipboard::kText;
+	data += (char)0; // 285 >> 24 = 285 / (256^3) = 0
+	data += (char)0; // 285 >> 16 = 285 / (256^2) = 0
+	data += (char)1; // 285 >> 8 = 285 / (256^1) = 1(.11328125)
+	data += (char)29; // 285 - 256 = 29
+	data += text;
+
+	clipboard.unmarshall(data, 0);
+
+	clipboard.open(0);
+	CString actual = clipboard.get(IClipboard::kText);
+	EXPECT_EQ(text, actual);
+}
+
+TEST(CClipboardTests, unmarshall_textAndHtml_getTextIsValid)
+{
+	CClipboard clipboard;
+	CString data;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)2; // 2 formats added
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)IClipboard::kText;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)14;
+	data += "synergy rocks!";
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)IClipboard::kHTML;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)10;
+	data += "html sucks";
+
+	clipboard.unmarshall(data, 0);
+
+	clipboard.open(0);
+	CString actual = clipboard.get(IClipboard::kText);
+	EXPECT_EQ("synergy rocks!", actual);
+}
+
+TEST(CClipboardTests, unmarshall_textAndHtml_getHtmlIsValid)
+{
+	CClipboard clipboard;
+	CString data;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)2; // 2 formats added
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)IClipboard::kText;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)14;
+	data += "synergy rocks!";
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)IClipboard::kHTML;
+	data += (char)0;
+	data += (char)0;
+	data += (char)0;
+	data += (char)10;
+	data += "html sucks";
+
+	clipboard.unmarshall(data, 0);
+
+	clipboard.open(0);
+	CString actual = clipboard.get(IClipboard::kHTML);
+	EXPECT_EQ("html sucks", actual);
+}
+
+TEST(CClipboardTests, copy_singleText_clipboardsAreEqual)
+{
+	CClipboard clipboard1;
+	clipboard1.open(0);
+	clipboard1.add(CClipboard::kText, "synergy rocks!");
+	clipboard1.close();
+
+	CClipboard clipboard2;
+	CClipboard::copy(&clipboard2, &clipboard1);
+
+	clipboard2.open(0);
+	CString actual = clipboard2.get(CClipboard::kText);
+	EXPECT_EQ("synergy rocks!", actual);
 }