如何使用Mockito测试以静态值作为参数从中调用另一个void函数的函数

时间:2018-12-30 21:33:35

标签: java unit-testing reflection mockito

我的A类是

Class A{
 private static final String ANON_DIR             = "/webapps/worldlingo/data/anonymizer/";
 private static final String NO_ANON             = "noanonymize";

  public String first(String text, String srclang, Map dictTokens) {
      Set<String> noAnonymize = new HashSet<String>();
      second(noAnonymize,ANON_DIR + NO_ANON, "tmpLang","name");

      String value;
      if(noAnonymize.contains("test")){
      value = word;
      }
      else {
         value = "test";
        }

    return value;
}

其中ANON_DIR和NO_ANON是静态最终值。此类具有第一个函数,第二个函数。第一个函数具有一个调用方法,该方法调用第二个函数。第二个函数是void函数,它将静态字段作为参数。

第二个功能只是文件读取功能,提供的路径为

private void second (Set<String> hashSet, String path, String lang , String type) {
    FileReader fr = null;
    BufferedReader br = null;

    try {
      fr = new FileReader(path);
      br = new BufferedReader(fr);
      String Line;
      while ((Line = br.readLine()) != null) {
        hashSet.add(Line);
      }
    } catch (IOException e) {
      log.error("Anonymizer: Unable to load file.", e);

    } finally {
      try {
        if (fr != null) {
          fr.close();
        }
        if (br != null) {
          br.close();
        }
      } catch (IOException e) {
        log.error("Anonymizer : An error occured while closing a resource.", e);
      }
    }
  }

  } 

现在,我尝试首先使用Mockito测试该功能。我正在尝试更改静态参数,并将更改后的静态参数发送为

public void testfirst() throws Exception {
    A anon = new A();


    Field ANON_DIR = A.class.getDeclaredField("ANON_DIR");
    ANON_DIR.setAccessible(true);

    //java relection to change private static final field
    Field modifiersField = Field.class.getDeclaredField("modifiers");
    modifiersField.setAccessible(true);
    modifiersField.setInt(ANON_DIR, ANON_DIR.getModifiers() & ~Modifier.FINAL);
    ANON_DIR.set(null,"test");

    Field NO_ANON = A.class.getDeclaredField("NO_ANON");
    NO_ANON.setAccessible(true);

    Field modifiersField1 = Field.class.getDeclaredField("modifiers");
    modifiersField1.setAccessible(true);
    modifiersField1.setInt(NO_ANON, NO_ANON.getModifiers() & ~Modifier.FINAL);
    NO_ANON.set(null,"/noanonymize");


    Method anonymizeNames = anon.getClass().getDeclaredMethod("first", String.class, String.class , Map.class);
    String srcLang = "MSFT_EN";
    Map mapTokens = new HashMap();
    String result = (String) anonymizeNames.invoke(anon,"I am David",srcLang,mapTokens);


  }

问题:   在这里,我可以使用java反射来更改私有最终静态字段ANON_DIR和NO_ANON,但是更改后的字段不会发送到第二个函数。从第一个函数调用的第二个函数采用原始值而不是更改后的值。即当第二个被称为ANON_DIR “ / webapps / worldlingo / data / anonymizer /”值,而不是“ test”。

由于我将ANON_DIR的值更改为“ test”,因此我希望将相同的“ test”值传递给第二个函数。我该如何测试第二种无效方法。

1 个答案:

答案 0 :(得分:0)

这里的问题是违反了关注点分离原则。

您的受测试的代码(剪切)做得太多,因此您很难找到接缝来替换依赖项。

对单元测试的目的也有误解。您不测试代码,而是测试可观察的公共行为,它是任何返回值与依赖项进行通信 (但不一定是公共的)方法)

所以我的建议是将方法second()移到自己的新类中。剪切获得了作为 constructor参数传递的新类的实例。 然后,很容易用测试中该新类的模拟替换真正的依赖项。

另一方面,您可以使用 PowerMock ...

来简单地屈服于不良设计