keySet with predicate should respect map store loadAllKeys

[ahmetmircik]

This test is failing:

   @Test
    public void testKeysWithPredicateShouldLoadMapStore() throws InterruptedException {
        TestEventBasedMapStore testMapStore = new TestEventBasedMapStore();

        Map mapForStore = new HashMap();
        mapForStore.put("key1", 17);
        mapForStore.put("key2", 37);
        mapForStore.put("key3", 47);
        testMapStore.getStore().putAll(mapForStore);

        Config config = newConfig(testMapStore, 0);
        HazelcastInstance instance = createHazelcastInstance(config);
        IMap map = instance.getMap("default");

        Set expected = map.keySet(Predicates.greaterThan("value", 1));
        assertEquals(3, expected.size());
        assertTrue(expected.contains("key1"));
        assertTrue(expected.contains("key2"));
        assertTrue(expected.contains("key3"));
    }

[pveentjer]

Does map.values(predicate) and map.entrySet(predicate) suffer from the same bug?

[ahmetmircik]

No they should work as expected.

[pveentjer]

Do we have tests for it?

[pveentjer]

I was cleaning up the methods so that foo() will make use of foo(TruePredicate.INSTANCE) if there is an overloaded version available.

[ahmetmircik]

Sorry they should be fixed too. And fix of this PR should fix them too.