Removes unnecessary tests in snapshot_packager_service.rs

[brooksprumo]

Problem

I'm working to remove serde_snapshot::reserialize_bank_with_new_accounts_hash(), and instead only serialize the bank once.

One of the tests in snapshot_packager_service.rs is to package up a snapshot. The test calls snapshot_bank_utils::package_and_archive_snapshot(), which calls reserialize. An observation is that the test in SPS doesn't actually test any code in SPS; it's all snapshot code being tested. The next observation is that we have many tests that exercise package_and_archive_snapshot(), thus the test in SPS can be safely removed as redundant. This removes a caller of reserialize_bank_with_new_accounts_hash(), yay!

Summary of Changes

Remove the redundant tests in SPS.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.7%. Comparing base (ec9bd79) to head (326dd49).
Report is 7 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master    #1371     +/-   ##
=========================================
- Coverage    82.8%    82.7%   -0.1%     
=========================================
  Files         872      872             
  Lines      369235   369169     -66     
=========================================
- Hits       305811   305638    -173     
- Misses      63424    63531    +107     

[jeffwashington]

lgtm