当我使用 try 和 catch 块时，Junit 测试失败。为什么？

Question

我想使用 Junit 测试框架测试我的 java 代码。在我的源代码中，我使用 try 和 catch 块来处理异常。在我的测试方法中，我想测试这种方法。如果我的源代码返回一些异常并且它在我的测试方法中匹配，我的测试将通过，否则失败。但是我的代码没有通过这个测试。

我找到了解决方法，但我想使用 try 和 catch 块。我也来分享一下。

不工作 我的源代码（我想测试这个方法）

    public boolean isCreateAccount(String amount) {
    
    try { 
        
        Double balance = Double.parseDouble(amount);
        
        if(balance < 0)
             throw new Exception();

    
    } catch (Exception e) { 
        System.out.println("WARNING: Invalid amount!");
    } 
    
         return false;
    
    }

这是我在 JUnit 中的测试方法

@Rule
public ExpectedException expectedException = ExpectedException.none();

@Test
public void testCreateAccountInvalidAmount() {

    expectedException.expect(Exception.class);
    accountServices.isCreateAccount("-100");    
    
}

工作中 这种方式可行，但我想使用 try 和 catch 块 我的源代码（我想测试这个方法）

    public void isCreateCheckingAccount(String amount) {
            
        Double balance = Double.parseDouble(amount); 

        if(balance < 0)
             throw new Exception();

    
     }

这是我在 JUnit 中的测试方法

@Rule
public ExpectedException expectedException = ExpectedException.none();

@Test
public void testCreateAccountInvalidAmount() {

    expectedException.expect(Exception.class);
    accountServices.isCreateAccount("-100");    
    
}

Answer 1

我想通了这个问题。谢谢你们的帮助。我在这里分享我的解决方案。如果您遇到此类问题，希望对您有所帮助。

//这个代码在我的服务层...

@Override
public boolean isCreateAccount(String amount) {
    
    try { 
            
        isValidAmount(amount);
        double balance = Double.parseDouble(amount);    
        createAccount(balance);
        return true;
        
    } catch(NullPointerException e) {
        log.warn("WARNING: Empty Amount!");
        System.out.println("\nWARNING: Empty Amount!");
        
    } catch (NumberFormatException e) { 
        log.warn("WARNING: Invalid amount!");
        System.out.println("\nWARNING: Invalid amount!");
        
    } catch (NegativeAmountException e) {
        log.warn("WARNING: Amount must have a positive number!");       
        System.out.println("\nWARNING: Amount must have a positive number!");
        
    } catch (InsufficientAmountException e) {
        log.warn("WARNING: Not enough money to open a checking account!");
        System.out.println("\nWARNING: Not enough money to open a checking 
    account!");
    }
    
    return false;
}

//这个代码在我的服务层...

@Override
public void isValidAmount(String money) {
    
    if(money == null || money.length() == 0) {
        throw new NullPointerException("Empty Amount");
    }
    
    double amount = 0.0D;
    
    try {
        amount = Double.parseDouble(money);         
    } catch (NumberFormatException e) {
        throw new NumberFormatException("Invalid Amount");
    } 
        
    if(amount < 0.0D) {
        throw new NegativeAmountException("Negative Amount");
    }       
    
    if(amount < 25.0D) {
        throw new InsufficientAmountException("Insufficient Amount");
    }

}

//这个代码在我的Junit测试类中

@Test
public void testCreateCheckingAccountNullAmount() {
    expectedException.expect(NullPointerException.class);
    expectedException.expectMessage("Empty Amount");
    accountServices.isValidAmount("");              
}

@Test
public void testCreateCheckingAccountInvalidAmount() {
    
    expectedException.expect(NumberFormatException.class);
    expectedException.expectMessage("Invalid Amount");
    accountServices.isValidAmount("a100");          
}

@Test
public void testCreateCheckingAccountNegativeAmount() {
    expectedException.expect(NegativeAmountException.class);
    expectedException.expectMessage("Negative Amount");
    accountServices.isValidAmount("-10");
}

@Test
public void testCreateCheckingAccountInsufficientAmount() {
    expectedException.expect(InsufficientAmountException.class);
    expectedException.expectMessage("Insufficient Amount");
    accountServices.isValidAmount("10");
}

Answer 2

这会做你想做的事：

public boolean isCreateAccount(String amount) throws Exception {

    Double balance = Double.parseDouble(amount);

    if (balance < 0) {
        System.out.println("WARNING: Invalid amount!");
        throw new Exception();
    }

    return false;
}

虽然返回 false 没有多大意义。

如果你真的想要那个 try catch 块：

public boolean isCreateAccount(String amount) throws Exception {

    try {
        Double balance = Double.parseDouble(amount);
        if (balance < 0) {
            System.out.println("WARNING: Invalid amount!");
            throw new Exception();
        }
    } catch (Exception e) {
        throw e;
    }

    return false;
}

这些基本测试适用于两个版本：

public class AccountTest {

    @Rule
    public ExpectedException expectedException = ExpectedException.none();

    @Test
    public void testCreateAccountInvalidAmount() throws Exception {
        Account accountServices = new Account();
        expectedException.expect(Exception.class);
        accountServices.isCreateAccount("-100");
    }

    @Test
    public void testCreateAccountZeroAmount() throws Exception {
        Account accountServices = new Account();
        assertFalse(accountServices.isCreateAccount("0"));
    }

    @Test
    public void testCreateAccountValidAmount() throws Exception {
        Account accountServices = new Account();
        assertFalse(accountServices.isCreateAccount("1"));
    }
}

Answer 3

看看你的方法。它具有以下行为：

• 如果金额为负数或不是数字，它会向 sysout 打印一些内容（这很糟糕；您不应该像这样混合职责；您的帐户工具肯定不是您的 UI 层，但这是次要问题）。否则它什么也不打印。

• 然后，不管你传入什么，它都会打印 false。

所以，这个方法的可观察行为是它返回 false，并且它打印一些东西到 sysout。

测试它是否返回 false 是微不足道的； assertFalse(services.isCreateAccount("100") 会做到这一点（不过，大概您的方法总是返回 false 的事实实际上是一个错误）。

测试它是否打印到 sysout 是很复杂的。这是因为你不应该在那里打印任何东西的根本问题。如果输出是要点的一部分，那么不要打印到System.out，而是将一些输出抽象（可能是java.io.PrintWriter）传递给AccountServices，这样您就可以传递一些变体作为测试的一部分；可以配置为检查是否在此处显示某些预期输出的变体。

您似乎认为这个方法会抛出一些东西。它没有 - 它捕捉一些东西。在junit中，说：“我期望这个异常”意味着您期望它从方法中消失，即相反：该方法没有捕获该异常，甚至该方法显式抛出该异常。