From 04894a04a517df97eaff3bfb8df2c33d9bee97a5 Mon Sep 17 00:00:00 2001 From: tian-lt Date: Thu, 1 Jul 2021 14:54:10 +0800 Subject: [PATCH] fixes up merging flaws --- .../CalculatorManagerTest.cpp | 76 ------------------- .../LocalizationSettingsUnitTests.cpp | 14 ++-- 2 files changed, 7 insertions(+), 83 deletions(-) diff --git a/src/CalculatorUnitTests/CalculatorManagerTest.cpp b/src/CalculatorUnitTests/CalculatorManagerTest.cpp index 81054c72..4837e3f7 100644 --- a/src/CalculatorUnitTests/CalculatorManagerTest.cpp +++ b/src/CalculatorUnitTests/CalculatorManagerTest.cpp @@ -1151,80 +1151,4 @@ namespace CalculatorManagerTest Command::Command4, Command::CommandMUL, Command::Command5, Command::CommandMUL, Command::CommandNULL }; TestDriver::Test(L"120", L"120 \x00D7 ", commands24); } - - void CalculatorManagerTest::CalculatorManagerTestStandardOrderOfOperations() - { - Command commands1[] = { Command::Command1, Command::CommandREC, Command::CommandNULL }; - TestDriver::Test(L"1", L"1/(1)", commands1); - - Command commands2[] = { Command::Command4, Command::CommandSQRT, Command::CommandNULL }; - TestDriver::Test(L"2", L"\x221A(4)", commands2); - - Command commands3[] = { Command::Command1, Command::CommandADD, Command::Command4, Command::CommandSQRT, Command::CommandNULL }; - TestDriver::Test(L"2", L"1 + \x221A(4)", commands3); - - Command commands4[] = { Command::Command1, Command::CommandADD, Command::Command4, Command::CommandSQRT, Command::CommandSUB, Command::CommandNULL }; - TestDriver::Test(L"3", L"3 - ", commands4); - - Command commands5[] = { Command::Command2, Command::CommandMUL, Command::Command4, Command::CommandREC, Command::CommandNULL }; - TestDriver::Test(L"0.25", L"2 \x00D7 1/(4)", commands5); - - Command commands6[] = { Command::Command5, Command::CommandDIV, Command::Command6, Command::CommandPERCENT, Command::CommandNULL}; - TestDriver::Test(L"0.06", L"5 \x00F7 0.06", commands6); - - Command commands7[] = { Command::Command4, Command::CommandSQRT, Command::CommandSUB, Command::CommandNULL }; - TestDriver::Test(L"2", L"\x221A(4) - ", commands7); - - Command commands8[] = { Command::Command7, Command::CommandSQR, Command::CommandDIV, Command::CommandNULL }; - TestDriver::Test(L"49", L"sqr(7) \x00F7 ", commands8); - - Command commands9[] = { Command::Command8, Command::CommandSQR, Command::CommandSQRT, Command::CommandNULL }; - TestDriver::Test(L"8", L"\x221A(sqr(8))", commands9); - - Command commands10[] = { Command::Command1, Command::Command0, Command::CommandADD, Command::Command2, Command::CommandSUB, Command::CommandNULL }; - TestDriver::Test(L"12", L"12 - ", commands10); - - Command commands11[] = { Command::Command3, Command::CommandMUL, Command::Command4, Command::CommandDIV, Command::CommandNULL }; - TestDriver::Test(L"12", L"12 \x00F7 ", commands11); - - Command commands12[] = { Command::Command6, Command::CommandDIV, Command::Command3, Command::CommandSUB, Command::CommandADD, Command::CommandNULL }; - TestDriver::Test(L"2", L"2 + ", commands12); - - Command commands13[] = { Command::Command7, Command::CommandSUB, Command::Command4, Command::CommandDIV, Command::CommandMUL, Command::CommandNULL }; - TestDriver::Test(L"3", L"3 \x00D7 ", commands13); - - Command commands14[] = { Command::Command8, Command::CommandMUL, Command::Command2, Command::CommandADD, Command::CommandSQRT, Command::CommandNULL }; - TestDriver::Test(L"4", L"16 + \x221A(16)", commands14); - - Command commands15[] = { Command::Command9, Command::CommandADD, Command::Command0, Command::CommandMUL, Command::CommandSIGN, Command::CommandNULL }; - TestDriver::Test(L"-9", L"9 \x00D7 negate(9)", commands15); - - Command commands16[] = { Command::Command9, Command::CommandSIGN, Command::Command0, Command::CommandADD, Command::CommandMUL, Command::CommandNULL }; - TestDriver::Test(L"-90", L"-90 \x00D7 ", commands16); - - Command commands17[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandEQU, Command::CommandNULL }; - TestDriver::Test(L"3", L"1 + 2=", commands17); - - Command commands18[] = { Command::Command2, Command::Command0, Command::CommandMUL, Command::Command0, Command::Command2, Command::CommandEQU, Command::CommandNULL }; - TestDriver::Test(L"40", L"20 \x00D7 2=", commands18); - - Command commands19[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandADD, Command::CommandBACK, Command::CommandNULL }; - TestDriver::Test(L"3", L"3 + ", commands19); - - Command commands20[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandADD, Command::CommandCLEAR, Command::CommandNULL }; - TestDriver::Test(L"0", L"", commands20); - - Command commands21[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandADD, Command::CommandCENTR, Command::CommandNULL }; - TestDriver::Test(L"0", L"3 + ", commands21); - - Command commands22[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandCLEAR, Command::CommandNULL }; - TestDriver::Test(L"0", L"", commands22); - - Command commands23[] = { Command::Command1, Command::CommandADD, Command::Command2, Command::CommandCENTR, Command::CommandNULL }; - TestDriver::Test(L"0", L"1 + ", commands23); - - Command commands24[] = { Command::Command1, Command::CommandMUL, Command::Command2, Command::CommandMUL, Command::Command3, Command::CommandMUL, Command::Command4, Command::CommandMUL, Command::Command5, Command::CommandMUL, Command::CommandNULL }; - TestDriver::Test(L"120", L"120 \x00D7 ", commands24); - } - } /* namespace CalculationManagerUnitTests */ diff --git a/src/CalculatorUnitTests/LocalizationSettingsUnitTests.cpp b/src/CalculatorUnitTests/LocalizationSettingsUnitTests.cpp index 64c68329..650f809b 100644 --- a/src/CalculatorUnitTests/LocalizationSettingsUnitTests.cpp +++ b/src/CalculatorUnitTests/LocalizationSettingsUnitTests.cpp @@ -52,13 +52,13 @@ TEST_METHOD(TestGetEnglishValueFromLocalizedDigits) TEST_METHOD(TestIsEnUsDigit) { - auto& settings = LocalizationSettings::GetInstance(); - VERIFY_IS_FALSE(settings.IsEnUsDigit(L'/')); - VERIFY_IS_TRUE(settings.IsEnUsDigit(L'0')); - VERIFY_IS_TRUE(settings.IsEnUsDigit(L'1')); - VERIFY_IS_TRUE(settings.IsEnUsDigit(L'8')); - VERIFY_IS_TRUE(settings.IsEnUsDigit(L'9')); - VERIFY_IS_FALSE(settings.IsEnUsDigit(L':')); + auto settings = LocalizationSettings::GetInstance(); + VERIFY_IS_FALSE(settings->IsEnUsDigit(L'/')); + VERIFY_IS_TRUE(settings->IsEnUsDigit(L'0')); + VERIFY_IS_TRUE(settings->IsEnUsDigit(L'1')); + VERIFY_IS_TRUE(settings->IsEnUsDigit(L'8')); + VERIFY_IS_TRUE(settings->IsEnUsDigit(L'9')); + VERIFY_IS_FALSE(settings->IsEnUsDigit(L':')); } TEST_METHOD(TestIsLocalizedDigit)