R4R: remove BlockHeight in context

[colin-axner]

closes #4578

• Targeted PR against correct branch (see CONTRIBUTING.md)

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

• Wrote tests

• Updated relevant documentation (docs/)

• Added a relevant changelog entry: clog add [section] [stanza] [message]

• rereviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[codecov]

Codecov Report

Merging #4580 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4580      +/-   ##
==========================================
+ Coverage   52.61%   52.65%   +0.04%     
==========================================
  Files         263      263              
  Lines       16464    16463       -1     
==========================================
+ Hits         8662     8669       +7     
+ Misses       7154     7146       -8     
  Partials      648      648

[alessio]

So far so good - I'd test the BlockHeight() return value after a WithBlockHeight() call

[alexanderbez]

ACK, thanks @colin-axner. Can we add a quick unit test in context_test.go?

[alexanderbez]

ACK