smith-chem-wisc / MetaMorpheus

Proteomics search software with integrated calibration, PTM discovery, bottom-up, top-down and LFQ capabilities
MIT License
90 stars 46 forks source link

warnings from old unit tests have crept into appveyor. #2220

Closed trishorts closed 1 year ago

trishorts commented 1 year ago

Some warnings in appveyor are expected from unit tests and some are not. Some old unit tests were throwing unexpected warnings. I fixed these old tests so we're getting the right number again.

codecov[bot] commented 1 year ago

Codecov Report

Merging #2220 (ff72e36) into master (8dabd2d) will increase coverage by 0.04%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2220      +/-   ##
==========================================
+ Coverage   91.08%   91.13%   +0.04%     
==========================================
  Files         133      133              
  Lines       20022    20021       -1     
  Branches     2778     2778              
==========================================
+ Hits        18238    18246       +8     
+ Misses       1288     1280       -8     
+ Partials      496      495       -1     
Impacted Files Coverage Δ
GuiFunctions/MetaDraw/MetaDrawLogic.cs 95.13% <100.00%> (+0.68%) :arrow_up:
TaskLayer/SearchTask/PostSearchAnalysisTask.cs 92.55% <0.00%> (+0.32%) :arrow_up: