fix non move multi boosts

[ikiris]

Pull request summary

Description:

Fixes boosting on dev branch when doing multiple boosts.

Fixed:

Non move boosts when doing multiple boosts.

• None

Testing checklist:

• [X ] Changes are backward-compatible OR version migration code is included
• [X ] Codebase compiles with current tsconfig configuration
• [X ] Tested changes on {choose PUBLIC/PRIVATE} server OR changes are trivial (e.g. typos)

[ahmedcharles]

I'd change this to have typescript help a little:

      const neededBoosts = zerg.getNeededBoosts();
      const neededBoostResources = _.keys(neededBoosts) as MineralBoostConstant[];

      const [moveBoosts, nonMoveBoosts] = _.partition(neededBoostResources,
                              resource => Abathur.isMoveBoost(<ResourceConstant>resource));

      for (const boost of [...moveBoosts, ...nonMoveBoosts]) { // try to get move boosts first if they're available
        const b: MineralBoostConstant = boost; // Have the types help ensure correctness.
        const boostLab = _.find(evolutionChamber.boostingLabs, lab => lab.mineralType === b);
        if (boostLab) {
          zerg.task = Tasks.getBoosted(boostLab, <ResourceConstant>boost);
          return;
        }
      }

Other than that, LGTM.