well this makes most of the tests pass

This commit is contained in:
Mike Griese 2020-01-23 14:30:36 -06:00
parent c44bc9ed66
commit 2fe6d8e6e4
2 changed files with 11 additions and 6 deletions

View file

@ -539,8 +539,10 @@ void ConptyRoundtripTests::TestResizeHeight()
// The Terminal should be stuck on the bottom of the viewport
const auto thirdTermView = term->GetViewport();
VERIFY_ARE_EQUAL(50 - thirdTermView.Height() + 1, thirdTermView.Top());
VERIFY_ARE_EQUAL(50, thirdTermView.BottomInclusive());
// VERIFY_ARE_EQUAL(50 - thirdTermView.Height() + 1, thirdTermView.Top());
// VERIFY_ARE_EQUAL(50, thirdTermView.BottomInclusive());
VERIFY_ARE_EQUAL(secondTermView.Top(), thirdTermView.Top());
VERIFY_ARE_EQUAL(50 + dy, thirdTermView.BottomInclusive());
verifyHostData(*hostTb, dy);
verifyTermData(*termTb);
@ -555,6 +557,8 @@ void ConptyRoundtripTests::TestResizeHeight()
// The Terminal should be stuck on the bottom of the viewport
const auto fourthTermView = term->GetViewport();
VERIFY_ARE_EQUAL(50 - fourthTermView.Height() + 1, fourthTermView.Top());
VERIFY_ARE_EQUAL(50, fourthTermView.BottomInclusive());
// VERIFY_ARE_EQUAL(50 - fourthTermView.Height() + 1, fourthTermView.Top());
// VERIFY_ARE_EQUAL(50, fourthTermView.BottomInclusive());
VERIFY_ARE_EQUAL(secondTermView.Top(), fourthTermView.Top());
VERIFY_ARE_EQUAL(50 + dy, fourthTermView.BottomInclusive());
}

View file

@ -100,8 +100,9 @@ void TerminalBufferTests::TestResizeHeight()
const auto thirdView = term->GetViewport();
VERIFY_ARE_EQUAL(50 - thirdView.Height() + 1, thirdView.Top());
VERIFY_ARE_EQUAL(50, thirdView.BottomInclusive());
VERIFY_ARE_EQUAL(secondView.Top(), thirdView.Top());
// VERIFY_ARE_EQUAL(50 - thirdView.Height() + 1, thirdView.Top());
VERIFY_ARE_EQUAL(50 + dy, thirdView.BottomInclusive());
verifyBufferContents();
}