how to Mockup map class and mock return map.size

I'm improving my coverage rate. So I try to enter a special condition by mock a map size, but I fail to do so. Does this method working with the original type such as map.class because I used the same method on my other class and they work fine?

Item i = new Item();

ItemTbl tbl = new ItemTbl();

tbl.setMerchantId(key.getMerchantId());

tbl.setBaseSku(key.getBaseSku().toUpperCase());

tbl.setOrigBaseSku(key.getBaseSku());

i.setItemTbl(tbl);

final Map<String, Item> existingItems = prepareExistingData(Arrays.asList(i), false, false);



Item existingItem;

existingItem = existingItems.get(tbl.getBaseSku());



if (existingItem == null) {

    return null; //original item api return 200 if item not found

}



if (existingItem.getShopItemMap() != null) {   

    if (existingItems.size() == 1) {

        //the thing I want to cover

    }

}

Junit test:

new MockUp<Map<String, Item>>() {

    @Mock

    public int size() {

        return 2;

    };

};



fixture.delete(entity);

I notice the existingItems is set as final, but I tried to delete final, but it still not work.

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

Which mocking framework are you using?

– Raedwald
Nov 22 '18 at 11:34

Why do you need to mock a Map? Cannot you just return a HashMap with two elements from your (mocked version of) prepareExistingData ?

– Thilo
Nov 22 '18 at 11:40

@Raedwald I use mockit currently. I tried mockito, but still no luck

– Ian Zhang
Nov 23 '18 at 1:12

@Thilo Because the data are too complicated to makeup which I still need to use in if statement, and I am wonder why it doesn't work.

– Ian Zhang
Nov 23 '18 at 1:20

Rather than clarify you question in comments, edit your question to include the extra information.

– Raedwald
Nov 23 '18 at 9:35

add a comment |

Item i = new Item();

ItemTbl tbl = new ItemTbl();

tbl.setMerchantId(key.getMerchantId());

tbl.setBaseSku(key.getBaseSku().toUpperCase());

tbl.setOrigBaseSku(key.getBaseSku());

i.setItemTbl(tbl);

final Map<String, Item> existingItems = prepareExistingData(Arrays.asList(i), false, false);



Item existingItem;

existingItem = existingItems.get(tbl.getBaseSku());



if (existingItem == null) {

    return null; //original item api return 200 if item not found

}



if (existingItem.getShopItemMap() != null) {   

    if (existingItems.size() == 1) {

        //the thing I want to cover

    }

}

Junit test:

new MockUp<Map<String, Item>>() {

    @Mock

    public int size() {

        return 2;

    };

};



fixture.delete(entity);

I notice the existingItems is set as final, but I tried to delete final, but it still not work.

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

Which mocking framework are you using?

– Raedwald
Nov 22 '18 at 11:34

Why do you need to mock a Map? Cannot you just return a HashMap with two elements from your (mocked version of) prepareExistingData ?

– Thilo
Nov 22 '18 at 11:40

@Raedwald I use mockit currently. I tried mockito, but still no luck

– Ian Zhang
Nov 23 '18 at 1:12

@Thilo Because the data are too complicated to makeup which I still need to use in if statement, and I am wonder why it doesn't work.

– Ian Zhang
Nov 23 '18 at 1:20

Rather than clarify you question in comments, edit your question to include the extra information.

– Raedwald
Nov 23 '18 at 9:35

add a comment |

Item i = new Item();

ItemTbl tbl = new ItemTbl();

tbl.setMerchantId(key.getMerchantId());

tbl.setBaseSku(key.getBaseSku().toUpperCase());

tbl.setOrigBaseSku(key.getBaseSku());

i.setItemTbl(tbl);

final Map<String, Item> existingItems = prepareExistingData(Arrays.asList(i), false, false);



Item existingItem;

existingItem = existingItems.get(tbl.getBaseSku());



if (existingItem == null) {

    return null; //original item api return 200 if item not found

}



if (existingItem.getShopItemMap() != null) {   

    if (existingItems.size() == 1) {

        //the thing I want to cover

    }

}

Junit test:

new MockUp<Map<String, Item>>() {

    @Mock

    public int size() {

        return 2;

    };

};



fixture.delete(entity);

I notice the existingItems is set as final, but I tried to delete final, but it still not work.

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

Item i = new Item();

ItemTbl tbl = new ItemTbl();

tbl.setMerchantId(key.getMerchantId());

tbl.setBaseSku(key.getBaseSku().toUpperCase());

tbl.setOrigBaseSku(key.getBaseSku());

i.setItemTbl(tbl);

final Map<String, Item> existingItems = prepareExistingData(Arrays.asList(i), false, false);



Item existingItem;

existingItem = existingItems.get(tbl.getBaseSku());



if (existingItem == null) {

    return null; //original item api return 200 if item not found

}



if (existingItem.getShopItemMap() != null) {   

    if (existingItems.size() == 1) {

        //the thing I want to cover

    }

}

Junit test:

new MockUp<Map<String, Item>>() {

    @Mock

    public int size() {

        return 2;

    };

};



fixture.delete(entity);

I notice the existingItems is set as final, but I tried to delete final, but it still not work.

java junit

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

edited Nov 22 '18 at 11:33

deHaar

2,50851628

edited Nov 22 '18 at 11:33

deHaar

2,50851628

edited Nov 22 '18 at 11:33

deHaar

2,50851628

asked Nov 22 '18 at 9:01

Ian Zhang

asked Nov 22 '18 at 9:01

Ian Zhang

asked Nov 22 '18 at 9:01

Ian Zhang

Which mocking framework are you using?

– Raedwald
Nov 22 '18 at 11:34

Why do you need to mock a Map? Cannot you just return a HashMap with two elements from your (mocked version of) prepareExistingData ?

– Thilo
Nov 22 '18 at 11:40

@Raedwald I use mockit currently. I tried mockito, but still no luck

– Ian Zhang
Nov 23 '18 at 1:12

@Thilo Because the data are too complicated to makeup which I still need to use in if statement, and I am wonder why it doesn't work.

– Ian Zhang
Nov 23 '18 at 1:20

Rather than clarify you question in comments, edit your question to include the extra information.

– Raedwald
Nov 23 '18 at 9:35

add a comment |

Which mocking framework are you using?

– Raedwald
Nov 22 '18 at 11:34

Why do you need to mock a Map? Cannot you just return a HashMap with two elements from your (mocked version of) prepareExistingData ?

– Thilo
Nov 22 '18 at 11:40

@Raedwald I use mockit currently. I tried mockito, but still no luck

– Ian Zhang
Nov 23 '18 at 1:12

@Thilo Because the data are too complicated to makeup which I still need to use in if statement, and I am wonder why it doesn't work.

– Ian Zhang
Nov 23 '18 at 1:20

Rather than clarify you question in comments, edit your question to include the extra information.

– Raedwald
Nov 23 '18 at 9:35

Which mocking framework are you using?

– Raedwald
Nov 22 '18 at 11:34

Why do you need to mock a Map? Cannot you just return a HashMap with two elements from your (mocked version of) prepareExistingData ?

– Thilo
Nov 22 '18 at 11:40

@Raedwald I use mockit currently. I tried mockito, but still no luck

– Ian Zhang
Nov 23 '18 at 1:12

@Thilo Because the data are too complicated to makeup which I still need to use in if statement, and I am wonder why it doesn't work.

– Ian Zhang
Nov 23 '18 at 1:20

Rather than clarify you question in comments, edit your question to include the extra information.

– Raedwald
Nov 23 '18 at 9:35

add a comment |

1 Answer
1

active

oldest

votes

I suggest you mock the behavior of size() on existingItems specifically.
It would look something like this:

when(existingItems.size()).thenReturn(2);

answered Nov 22 '18 at 9:36

Gy. Timi

161

add a comment |

Your Answer

StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});

}
});

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Post as a guest

Name

Required, but never shown

StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53427193%2fhow-to-mockup-map-class-and-mock-return-map-size%23new-answer', 'question_page');
}
);

Post as a guest

Name

Required, but never shown

1 Answer
1

active

oldest

votes

1 Answer
1

active

oldest

votes

I suggest you mock the behavior of size() on existingItems specifically.
It would look something like this:

when(existingItems.size()).thenReturn(2);

answered Nov 22 '18 at 9:36

Gy. Timi

161

add a comment |

I suggest you mock the behavior of size() on existingItems specifically.
It would look something like this:

when(existingItems.size()).thenReturn(2);

answered Nov 22 '18 at 9:36

Gy. Timi

161

add a comment |

I suggest you mock the behavior of size() on existingItems specifically.
It would look something like this:

when(existingItems.size()).thenReturn(2);

answered Nov 22 '18 at 9:36

Gy. Timi

161

I suggest you mock the behavior of size() on existingItems specifically.
It would look something like this:

when(existingItems.size()).thenReturn(2);

answered Nov 22 '18 at 9:36

Gy. Timi

161

answered Nov 22 '18 at 9:36

Gy. Timi

161

answered Nov 22 '18 at 9:36

Gy. Timi

161

answered Nov 22 '18 at 9:36

Gy. Timi

161

add a comment |

draft saved

draft discarded

Thanks for contributing an answer to Stack Overflow!

Please be sure to answer the question. Provide details and share your research!

But avoid …

Asking for help, clarification, or responding to other answers.

Making statements based on opinion; back them up with references or personal experience.

To learn more, see our tips on writing great answers.

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Post as a guest

Name

Required, but never shown

Post as a guest

Name

Required, but never shown

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Post as a guest

Name

Required, but never shown

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Post as a guest

Name

Required, but never shown

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Post as a guest

Name

Required, but never shown

Name

Required, but never shown

Name

Required, but never shown

This page is only for reference, If you need detailed information, please check here

Search This Blog

Ufyukyu