Fehler beim Versuch, eine Karte zu stubben

Question

Ich habe eine Methode, die ich versuche, Einheitstest mit Mocks. Hier ist die Methode:

{... 
     mapMasterMap = this.sysAdminService.getMasterData(); 

     final Map<String, MasterVO> codeMap = (Map<String, MasterVO>) mapMasterMap 
       .get("mvo"); 
     final Map<String, String> sessionMap = (Map<String, String>) mapMasterMap 
       .get("smap"); 
     dataVO.setSessionMap(sessionMap); 
     dataVO.setVO1(codeMap.get("vo1")); 
     dataVO.setVO2(codeMap.get("vo2")); 
     sCommand.setDataVO(dataVO); 
    } catch (final Exception e) { 
     return mav; 
    } 
    return mav; 
} 

Und was ich versuche, die erste Zeile Stummel zu tun, so dass mapMasterMap eine gültige Karte enthält (und die codeMap.gets nicht sprengen) - wie:

{ 
    @Mock 
    private MasterVO masterVO; 
     @Mock 
    private SysAdminService sysAdminService; 


    @InjectMocks 
    private SysAdminController sysAdminController; 

    @Before 
    public void setUp() { 
     MockitoAnnotations.initMocks(this); 
       when(this.sysAdminService.getMasterData()) 
       .thenReturn(new HashMap<String, MasterVO>() {{ 
        this.put("mvo",this.masterVO); 
       }}; 
    } 

    @Test 
    public final void testType(){} 

ich erhalte eine Fehlermeldung:

The method thenReturn(Map<String,Object>) in the type OngoingStubbing<Map<String,Object>> is not applicable for the arguments (new HashMap<String,MasterVO>(){})

Also, erstens - verfolge ich den richtigen Ansatz? Wenn ja, wie kann ich das beheben?

Answer 1

Die Meldung sagt alles: Sie können nicht thenReturn ein HashMap<String, MasterVO> wo Java für eine Map<String,Object> fragt, was Mockito vom Rückgabetyp getMasterData ableitet. Sie müssen eine HashMap<String, Object> oder eine andere Map<String, Object> übergeben, die in Ihrem Fall so einfach ist, wie den Typ in Ihrem Stubbing-Anruf zu ändern.

when(this.sysAdminService.getMasterData()) 
    .thenReturn(new HashMap<String, Object>() {{ 
     this.put("mvo",this.masterVO); 
    }}); 

Warum? In Java, generics are not covariant: Auch wenn MasterVO notwendigerweise Object erweitert, können Sie nicht eines anstelle des anderen verwenden. Sonst könnte man dies ganz einfach:

Map<String, MasterVO> stubMap = new HashMap<String, MasterVO>(); 
when(this.sysAdminService.getMasterData()).thenReturn(stubMap); 
Map<String, Object> returnedMap = this.sysAdminService.getMasterData(); 

// now returnedMap == stubMap, except you've lost some safety 
returnedMap.put("and this is why", "generic types are not covariant"); // legal 
MasterVO exceptItIsnt = stubMap.get("and this is why"); // ClassCastException! 

Hinweis: Wie dimo414 erwähnt, keine Doppel Klammer Syntax. Es erstellt eine unnötige Java-Klasse und kann Speicher verlieren.