De-generify VarScan tumor/normal sample renaming
De-generify VarScan tumor/normal sample renaming

This patch makes the sample renaming needed for VarScan adjustment
exclusive for paired tumor/normal calling ("VarScan somatic"). Instead,
a dictionary is used to map NORMAL and TUMOR to the right sample names.

The rationale for the change is as follows:

- This function is only used in paired mode (no breakage of other calls);
- The change removes needless samtools calls for sample names (the
  helper function called during VarScan invokation already does the job);
- Reduces the risk of BAMs provided in a different order than [normal,
  tumor] which may cause incorrect results later on.

chapmanb/bcbio-nextgen,a113n/bcbio-nextgen,elkingtonmcb/bcbio-nextgen,verdurin/bcbio-nextgen,a113n/bcbio-nextgen,gifford-lab/bcbio-nextgen,mjafin/bcbio-nextgen,vladsaveliev/bcbio-nextgen,SciLifeLab/bcbio-nextgen,gifford-lab/bcbio-nextgen,hjanime/bcbio-nextgen,biocyberman/bcbio-nextgen,fw1121/bcbio-nextgen,elkingtonmcb/bcbio-nextgen,lbeltrame/bcbio-nextgen,vladsaveliev/bcbio-nextgen,elkingtonmcb/bcbio-nextgen,brainstorm/bcbio-nextgen,a113n/bcbio-nextgen,vladsaveliev/bcbio-nextgen,chapmanb/bcbio-nextgen,lpantano/bcbio-nextgen,brainstorm/bcbio-nextgen,lpantano/bcbio-nextgen,biocyberman/bcbio-nextgen,brainstorm/bcbio-nextgen,mjafin/bcbio-nextgen,chapmanb/bcbio-nextgen,SciLifeLab/bcbio-nextgen,guillermo-carrasco/bcbio-nextgen,verdurin/bcbio-nextgen,guillermo-carrasco/bcbio-nextgen,fw1121/bcbio-nextgen,biocyberman/bcbio-nextgen,Cyberbio-Lab/bcbio-nextgen,SciLifeLab/bcbio-nextgen,lpantano/bcbio-nextgen,guillermo-carrasco/bcbio-nextgen,lbeltrame/bcbio-nextgen,Cyberbio-Lab/bcbio-nextgen,hjanime/bcbio-nextgen,hjanime/bcbio-nextgen,gifford-lab/bcbio-nextgen,fw1121/bcbio-nextgen,mjafin/bcbio-nextgen,Cyberbio-Lab/bcbio-nextgen,lbeltrame/bcbio-nextgen,verdurin/bcbio-nextgen