CI: Ensure algod_expect_test.go tests run by avoiding double partitioning

[michaeldiamant]

Summary

The PR fixes a partitioning configuration error in test/e2e-go/cli/algod/expect/algod_expect_test.go. The parent test runner should not partition itself because its subtests perform partitioning.

• In a forthcoming PR, I'm tweaking CircleCI parallelism for E2E expect tests. During experimentation, I realized the configuration is broken when the build failed: https://app.circleci.com/pipelines/github/algorand/go-algorand/8782/workflows/3136b236-4a2a-4f63-a5df-f014b57880c7/jobs/158744?invite=true#step-103-22. Reproduced below for convenience:

============= UNINTENTIONALLY SKIPPED ============
1 tests were skipped unintentionally
github.com/algorand/go-algorand/test/e2e-go/cli/algod/expect TestAlgodWithExpect/algodTelemetryLocationTest.exp -- skipped unintentionally. (due to partitiontest.PartitionTest() being called twice?)
1 tests were skipped unintentionally.
==================================================

• As an additional proof point, you can see the forthcoming PR's branch pass after making this PR's change: https://app.circleci.com/pipelines/github/algorand/go-algorand/8787/workflows/a566ab26-42ad-48de-940d-08d504ce1c15.
• I confirmed all other parent E2E expect test runners do not partition tests.

Test Plan

Run CI.

[codecov]

Codecov Report

Merging #4421 (4bea609) into master (f156f69) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4421      +/-   ##
==========================================
- Coverage   55.18%   55.14%   -0.05%     
==========================================
  Files         397      397              
  Lines       50073    50073              
==========================================
- Hits        27634    27612      -22     
- Misses      20150    20166      +16     
- Partials     2289     2295       +6     

Impacted Files	Coverage Δ
ledger/roundlru.go	90.56% <0.00%> (-5.67%)	⬇️
ledger/blockqueue.go	85.63% <0.00%> (-2.88%)	⬇️
crypto/merkletrie/trie.go	66.42% <0.00%> (-2.19%)	⬇️
crypto/merkletrie/node.go	91.62% <0.00%> (-1.87%)	⬇️
catchup/service.go	68.64% <0.00%> (-0.75%)	⬇️
ledger/acctupdates.go	69.92% <0.00%> (-0.61%)	⬇️
network/wsPeer.go	67.67% <0.00%> (-0.55%)	⬇️
network/wsNetwork.go	64.89% <0.00%> (+0.19%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[cce]

LGTM — another way to mark this would be

//nolint:partitiontest // Causes double partition, so commented out on purpose