DevWiki/calculator
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixing CalculatorManagerTests by setting calculator mode so CCalcEngine is set. (#940)

Browse Source
This commit is contained in:
Scott Freeman 2020-01-17 23:19:17 -05:00 committed by Matt Cooley
parent 1947a25ff8
commit e77823b97c
1 changed files with 57 additions and 57 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

114
src/CalculatorUnitTests/CalculatorManagerTest.cpp
View File

@ -195,10 +195,10 @@ namespace CalculatorManagerTest
TEST_METHOD(CalculatorManagerNumberFormattingUtils_GetNumberDigitsWholeNumberPart); TEST_METHOD(CalculatorManagerNumberFormattingUtils_GetNumberDigitsWholeNumberPart);
TEST_METHOD(CalculatorManagerNumberFormattingUtils_RoundSignificantDigits); TEST_METHOD(CalculatorManagerNumberFormattingUtils_RoundSignificantDigits);
TEST_METHOD(CalculatorManagerNumberFormattingUtils_ToScientificNumber); TEST_METHOD(CalculatorManagerNumberFormattingUtils_ToScientificNumber);
// TODO re-enable when cause of failure is determined. Bug 20226670
// TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived); TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived);
// TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived_Multiple); TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived_Multiple);
// TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived_LongInput); TEST_METHOD(CalculatorManagerTestBinaryOperatorReceived_LongInput);
TEST_METHOD_CLEANUP(Cleanup); TEST_METHOD_CLEANUP(Cleanup);
@ -1008,72 +1008,72 @@ namespace CalculatorManagerTest
VERIFY_ARE_EQUAL(result, L"-3.432432e-09"); VERIFY_ARE_EQUAL(result, L"-3.432432e-09");
} }
// TODO re-enable when cause of failure is determined. Bug 20226670 void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived()
// void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived() {
// { CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// ExecuteCommands({ m_calculatorManager->SetStandardMode();
// Command::Command1, ExecuteCommands({
// Command::CommandADD Command::Command1,
// }); Command::CommandADD
});
// wstring display = pCalculatorDisplay->GetPrimaryDisplay(); wstring display = pCalculatorDisplay->GetPrimaryDisplay();
// VERIFY_ARE_EQUAL(wstring(L"1"), display); VERIFY_ARE_EQUAL(L"1", display);
// // Verify BinaryOperatorReceived // Verify BinaryOperatorReceived
// VERIFY_ARE_EQUAL(1, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(1, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// } }
// TODO re-enable when cause of failure is determined. Bug 20226670 void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived_Multiple()
// void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived_Multiple() {
// { CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// ExecuteCommands({ m_calculatorManager->SetStandardMode();
// Command::Command1, ExecuteCommands({
// Command::CommandADD, Command::Command1,
// Command::CommandSUB, Command::CommandADD,
// Command::CommandMUL Command::CommandSUB,
// }); Command::CommandMUL
});
// wstring display = pCalculatorDisplay->GetPrimaryDisplay(); wstring display = pCalculatorDisplay->GetPrimaryDisplay();
// VERIFY_ARE_EQUAL(wstring(L"1"), display); VERIFY_ARE_EQUAL(L"1", display);
// // Verify BinaryOperatorReceived // Verify BinaryOperatorReceived
// VERIFY_ARE_EQUAL(3, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(3, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// } }
// TODO re-enable when cause of failure is determined. Bug 20226670 void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived_LongInput()
// void CalculatorManagerTest::CalculatorManagerTestBinaryOperatorReceived_LongInput() {
// { CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// CalculatorManagerDisplayTester* pCalculatorDisplay = (CalculatorManagerDisplayTester *)m_calculatorDisplayTester.get();
// VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(0, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// ExecuteCommands({ m_calculatorManager->SetStandardMode();
// Command::Command1, ExecuteCommands({
// Command::CommandADD, Command::Command1,
// Command::Command2, Command::CommandADD,
// Command::CommandMUL, Command::Command2,
// Command::Command1, Command::CommandMUL,
// Command::Command0, Command::Command1,
// Command::CommandSUB, Command::Command0,
// Command::Command5, Command::CommandSUB,
// Command::CommandDIV, Command::Command5,
// Command::Command5, Command::CommandDIV,
// Command::CommandEQU Command::Command5,
// }); Command::CommandEQU
});
// wstring display = pCalculatorDisplay->GetPrimaryDisplay(); wstring display = pCalculatorDisplay->GetPrimaryDisplay();
// VERIFY_ARE_EQUAL(wstring(L"5"), display); VERIFY_ARE_EQUAL(L"5", display);
// // Verify BinaryOperatorReceived // Verify BinaryOperatorReceived
// VERIFY_ARE_EQUAL(4, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount()); VERIFY_ARE_EQUAL(4, pCalculatorDisplay->GetBinaryOperatorReceivedCallCount());
// } }
} /* namespace CalculationManagerUnitTests */ } /* namespace CalculationManagerUnitTests */