From 3c444517cf5940c9d6e9fb2cc33523528e0551f9 Mon Sep 17 00:00:00 2001 From: Nick Bolton Date: Sat, 23 Apr 2011 19:55:55 +0000 Subject: [PATCH] finished CClipboardTests --- src/unittest/synergy/CClipboardTests.cpp | 365 +++++++++++++++++++++-- 1 file changed, 347 insertions(+), 18 deletions(-) diff --git a/src/unittest/synergy/CClipboardTests.cpp b/src/unittest/synergy/CClipboardTests.cpp index 1fb74435..71f653e6 100644 --- a/src/unittest/synergy/CClipboardTests.cpp +++ b/src/unittest/synergy/CClipboardTests.cpp @@ -18,56 +18,385 @@ #include #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); - IClipboard::EFormat format(IClipboard::kText); + clipboard.open(0); - clipboard.add(format, "Hello world!"); + clipboard.add(IClipboard::kText, "synergy rocks!"); - CString actual = clipboard.get(format); - EXPECT_EQ("Hello world!", actual); + CString actual = clipboard.get(IClipboard::kText); + EXPECT_EQ("synergy rocks!", actual); } TEST(CClipboardTests, add_replaceValue_valueWasStored) { CClipboard clipboard; - clipboard.open(true); - IClipboard::EFormat format(IClipboard::kText); + clipboard.open(0); - clipboard.add(format, "Hello world!"); - clipboard.add(format, "Goodbye world!"); + clipboard.add(IClipboard::kText, "synergy rocks!"); + clipboard.add(IClipboard::kText, "maxivista sucks"); // haha, just kidding. - CString actual = clipboard.get(format); - EXPECT_EQ("Goodbye world!", actual); + CString actual = clipboard.get(IClipboard::kText); + 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(expected, actual); + EXPECT_EQ(0, 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); }