Question

我正在使用java为java编写自定义声纳规则。我遇到了一个无法轻易修复的断言错误。我确定源代码是正确的。但测试用例无法通过。我想知道在使用TDD过程时我应该关心什么，以及如何解决它。

public class logTCheckFile {
    private static Logger logger = Logger.getLogger(logTCheckFile.class);
    public void loggingWithID(String nonsense) throws myException{
        logger.error("errorID:20160801 this is an error");
        return;
    }

    public void loggingWithoutID(String nonsens){
        try{
            logger.error("this is an error");
        }catch(NullPointerException e){
            logger.error("what",e);
        }
        return;
    }

    public void specific(){
        logger.error("only the logger");
        try{
            logger.error("this is an error");
        }catch(NullPointerException e){
            logger.error("without an exception");
        }
        return;
    }
}

我正在测试上面的文件，我编写了一条规则来测试是否在记录器中打印了未抛出的异常。

消息是AssertionError: Unexpected at [20]（这是故障堆栈跟踪的图片）

我为检查文件而编写的代码如下：

public class logTCheck extends  IssuableSubscriptionVisitor {


Logger log = Logger.getLogger(logTCheck.class);

@Override
public List<Kind> nodesToVisit() {
    return ImmutableList.of(Kind.METHOD);
}

@Override
public void visitNode(Tree tree){
    MethodTree method = (MethodTree) tree;
    if(method.throwsClauses().size()==0){
        log.info("this method does not have a throw clause");
        BlockTree bt = method.block();
        for(StatementTree st:bt.body()){
            if(st.is(Kind.TRY_STATEMENT)){
                TryStatementTree tst = (TryStatementTree) st;
                for(CatchTree ct:tst.catches()){
                    for(StatementTree state:ct.block().body()){
                        ExpressionStatementTree ex = (ExpressionStatementTree)state;
                        MethodInvocationTree mit = (MethodInvocationTree) ex.expression();
                        if(mit.arguments().size()!=2){
                            log.error(method.simpleName());
                            reportIssue(method.simpleName(), "you didn't print the exception in the log");
                        }
                    }
                }
            }
        }
    };
 }
}

Answer 1

消息是AssertionError：在[20]
时意外

这意味着测试数据的第20行包含违反您正在检查的规则的行为。

您需要告知验证人此违规行为是故意的。一种简单的方法是在违规行为之前的行上添加这样的注释：

// Noncompliant@+1 {{the violation message}}

@+1表示违规发生在下一行。适当调整数字。

评论必须在该行的开头，或者//前面可能有空格。

{{...}}中包含的违规消息是可选的，但强烈建议。在做TDD时，输入正确消息的简单方法是添加{{x}}之类的内容，这会导致测试失败，然后您可以将测试输出中的消息复制到测试文件中进行修复。

Answer 2

我终于从迈克尔的另一个答案中找到了问题。我没有告诉测试人员问题应该在哪里。我应该使用评论// Noncompliant来标出问题。

在为java编写自定义声纳规则时如何处理assertionError

2 个答案: