From a6200a5600ccddc8e94da6619ac4da38637d79fd Mon Sep 17 00:00:00 2001 From: Michael Kirk Date: Thu, 7 Jun 2018 15:29:37 -0600 Subject: [PATCH] update layout tests --- .../MesssagesBubblesSizeCalculatorTest.swift | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/Signal/test/Models/MesssagesBubblesSizeCalculatorTest.swift b/Signal/test/Models/MesssagesBubblesSizeCalculatorTest.swift index cbcd4af54..30aa5e9b7 100644 --- a/Signal/test/Models/MesssagesBubblesSizeCalculatorTest.swift +++ b/Signal/test/Models/MesssagesBubblesSizeCalculatorTest.swift @@ -10,7 +10,7 @@ import SignalServiceKit * This is a brittle test, which will break if our layout changes. * * It serves mostly as documentation for cases to consider when changing the cell measurement logic. - * Primarly these test cases came out of a bug introduced in iOS10, + * Primarily these test cases came out of a bug introduced in iOS10, * which prevents us from computing proper bounding box for text that uses the UIEmoji font. * * If one of these tests breaks, it should be OK to update the expected value so long as you've tested the result renders @@ -48,70 +48,70 @@ class MesssagesBubblesSizeCalculatorTest: XCTestCase { let text: String? = "" let viewItem = self.viewItemForText(text) let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(42, actual.height) + XCTAssertEqual(36, actual.height) } func testHeightForShort1LineMessage() { let text = "foo" let viewItem = self.viewItemForText(text) let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(42, actual.height) + XCTAssertEqual(36, actual.height) } func testHeightForLong1LineMessage() { let text = "1 2 3 4 5 6 7 8 9 10 11 12 13 14 x" let viewItem = self.viewItemForText(text) let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(64, actual.height) + XCTAssertEqual(58, actual.height) } func testHeightForShort2LineMessage() { let text = "1 2 3 4 5 6 7 8 9 10 11 12 13 14 x 1" let viewItem = self.viewItemForText(text) let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(64, actual.height) + XCTAssertEqual(58, actual.height) } func testHeightForLong2LineMessage() { let text = "1 2 3 4 5 6 7 8 9 10 11 12 13 14 x 1 2 3 4 5 6 7 8 9 10 11 12 13 14 x" let viewItem = self.viewItemForText(text) let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(86, actual.height) + XCTAssertEqual(80, actual.height) } func testHeightForiOS10EmojiBug() { let viewItem = self.viewItemForText("Wunderschönen Guten Morgaaaahhhn 😝 - hast du gut geschlafen ☺️😘") let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(86, actual.height) + XCTAssertEqual(80, actual.height) } func testHeightForiOS10EmojiBug2() { let viewItem = self.viewItemForText("Test test test test test test test test test test test test 😊❤️❤️") let actual = messageBubbleSize(for: viewItem) - XCTAssertEqual(86, actual.height) + XCTAssertEqual(80, actual.height) } func testHeightForChineseWithEmojiBug() { let viewItem = self.viewItemForText("一二三四五六七八九十甲乙丙😝戊己庚辛壬圭咖啡牛奶餅乾水果蛋糕") let actual = messageBubbleSize(for: viewItem) // erroneously seeing 69 with the emoji fix in place. - XCTAssertEqual(86, actual.height) + XCTAssertEqual(80, actual.height) } func testHeightForChineseWithoutEmojiBug() { let viewItem = self.viewItemForText("一二三四五六七八九十甲乙丙丁戊己庚辛壬圭咖啡牛奶餅乾水果蛋糕") let actual = messageBubbleSize(for: viewItem) // erroneously seeing 69 with the emoji fix in place. - XCTAssertEqual(86, actual.height) + XCTAssertEqual(80, actual.height) } func testHeightForiOS10DoubleSpaceNumbersBug() { let viewItem = self.viewItemForText("12345678901234567890") let actual = messageBubbleSize(for: viewItem) // erroneously seeing 51 with emoji fix in place. It's the call to "fix string" - XCTAssertEqual(64, actual.height) + XCTAssertEqual(58, actual.height) } }