tree e2ac4aee520db815c4f623fc0633f0ae7780c449
parent f5f89beb70bb8b40be89971e253e0734d9dbf535
author Alice Kober-Sotzek <aliceks@google.com> 1508504390 +0200
committer David Pursehouse <dpursehouse@collab.net> 1508550843 +0000

Fix check for group index in GroupCacheImpl.ByIdLoader

The group index is added to the GroupIndexCollection at some point
during startup. When the constructor of a class is called via Guice,
we cannot be sure that the GroupIndexCollection already has the group
index. This situation becomes obvious when we run our tests: Without
this change, we never lookup the groups from the index in
GroupCacheImpl.ByIdLoader during test execution.

The fix is simple: We test for the existence of the group index right
before we try to access it.

Change-Id: Ic467bb1805d6058a7ae770e81e86ef59df81742a
(cherry picked from commit fccc9851760b93c7d6e3772435c045848988b6bf)