I would prefer we mock iccManager instead of this trick. Every test can have its own instance of iccManager configured to respond with the proper contacts. I would extend shared/js/test/unit/mocks/mock_iccmanager accordingly. 

On the other hand please do not use setTimeout to call the onsuccess callbacks.