c++ - cppunit 只接受返回值零

Question

我是CPPUnit测试的新手，已经用各种测试用例编写了登录认证功能的测试。但它只接受返回值零。

如果预期和实际相同，第一个测试用例应返回 1，但它仅在我更改为零时有效。任何建议表示赞赏。谢谢。

void POSUnitTest::testAuthenticate()
{        
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser","password"),1); // valid username and password    
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser1","password"),0); //invalid username, valid password             
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser","password1"),0); //valid username, invalid password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser1","password1"),0); // invalid username and password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate(" ","password"),0); // Empty username, valid password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser",""),0); // Valid username, empty password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("",""),0); // Empty username and password 
        cout << "Test 3) testAuthenticate successful.\n\n";   
}

这是我在 POSConsole 类中的 Authenticate 函数。

int POSConsole::Authenticate(string _userID, string _password)
{   
    bool failedLogin=false;
    int returnValue=0;

    _password = Encrypt (_password);     //Encrypt user enter password to compare with vector  

        for (int index = 0; index < cashierVector.size(); index++)
        {
                if ( (_userID == cashierVector[index].getUserID()) && 
                 (_password == cashierVector[index].getPassword()))
                {
                        returnValue=1;                                                     
            system("clear");
                        POSMenu();                                
            break;                                
                }
                else
                {
                    returnValue=0;
                    cout << "Invalid user login information. Please try again.\n";           
                    failCount++;                   
                    break;
                }
        }

        return returnValue;   

        if (failCount == 3)                
         {         
              cout << "You have used maximum attempts to login. Your account has been locked." << endl;
              exit (0);                       
         }

    }

编辑:我想我已经找到了问题所在。当我运行 CPPUnit 测试时，我的 vector 大小显示为零。我有一个函数可以将文本文件数据读取为 vector 。所以我从 CPPUnit testAuthenticate 函数中调用了该函数。现在所有测试用例都通过了，但其他单元测试功能变得不可见。如果有意义的话，编译器看不到它们。我总共有 10 个测试函数，但只有两个正在处理。我根本没有从其他测试用例中得到任何输出，甚至没有失败错误。

请问如何解决？感谢您的帮助，因为我已经坚持了好几天了。

void POSUnitTest::testAuthenticate()
{       
        console.readData();
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser","password"),1); // valid username and password    
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser1","password"),0); //invalid username, valid password             
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser","password1"),0); //valid username, invalid password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser1","password1"),0); // invalid username and password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate(" ","password"),0); // Empty username, valid password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("testuser",""),0); // Valid username, empty password
        CPPUNIT_ASSERT_EQUAL (console.Authenticate("",""),0); // Empty username and password */
        cout << "Test 3) testAuthenticate successful.\n\n";   
}

修改后的函数:

int POSConsole::Authenticate(string _userID, string _password)
{   
    bool validUser=false;
    bool failedLogin=false;
    int returnValue=0;

    _password = Encrypt (_password);        

        int vectorzie = cashierVector.size();
        cout << "\nVector size" << vectorzie << endl;
        for (int index = 0; index < cashierVector.size(); index++)
        {
                if ( (_userID == cashierVector[index].getUserID()) && 
                 (_password == cashierVector[index].getPassword()))
                {
            validUser = true;
                        returnValue=1;       
                        cout << "\nOk Login Return Value: " << returnValue; //this show 1
            system("clear");
                        POSMenu();                                
            break;                                
                }                
        }
        if (validUser=false)  //I have changed to this from else statement
                {
                    failedLogin=true;
                    returnValue=2;
                    cout << "\nfail Login Return Value: " << returnValue;
                    cout << "\nInvalid user login information. Please try again.\n";           
                    failCount++;                   
                    cout << "\nfail count: " << failCount << endl; 

                }

        cout << "Final Login Return Value: " << returnValue; 

        if (failCount == 3)                
         {         
              cout << "You have used maximum attempts to login. Your account has been locked." << endl;
              exit (0);                       
         }

        return returnValue;                 
    }

Answer 1

对于 if ... else 的两种情况，您都有一个 break 语句。因此，您的循环将始终在与 cashierVector 的第一个元素进行比较后结束。

如果您删除两个 break，那么您的循环将继续运行，即使已经找到用户并且 returnValue 将重置为 0再次。

您只需要删除 else block 中的 break。事实上，只有在 cashierVector 中没有匹配的用户/密码组合时，才应执行您的 else block 。所以它应该放在循环之后，而不是循环中。

函数永远不会到达return returnValue 之后的 block ，因为它已经在该点返回。您需要在 return 语句之前移动它。但是，如果您这样做，那么您的单元测试将永远不会成功，因为在第三次使用错误密码调用 Authenticate 之后(就像您在测试中所做的那样)，程序将通过 exit( 0)。

在侧节点上:这是一个 map 而不是 vector 的用例。如果您有很多用户，那么搜索将花费很长时间。在 map 中，这要快得多，而且您不需要循环。

编辑后:

如果 (validUser=false)。这里将false赋值给validUser，然后测试返回，是false。因此这个 if block 永远不会被执行。您应该将其更改为 if(validUser==false) 或更好的 if(!valid_user)。您还缺少该 block 中的 return 语句。如果它被执行，你不希望下面的部分也被执行。