send state identifiery when getting miner worker address

[laser]

Makes incremental progress towards #148.

[codecov-io]

Codecov Report

Merging #153 into master will decrease coverage by 0.31%.
The diff coverage is 61.12%.

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
- Coverage   82.07%   81.76%   -0.30%     
==========================================
  Files          33       33              
  Lines        1561     1573      +12     
==========================================
+ Hits         1281     1286       +5     
- Misses        200      204       +4     
- Partials       80       83       +3     

Impacted Files	Coverage Δ
storagemarket/types.go	100.00% <ø> (ø)
retrievalmarket/impl/provider.go	70.67% <50.00%> (-1.05%)	⬇️
...oragemarket/impl/providerstates/provider_states.go	95.78% <50.00%> (-1.34%)	⬇️
storagemarket/impl/storedask/storedask.go	79.73% <66.67%> (-1.69%)	⬇️
storagemarket/impl/providerutils/providerutils.go	88.24% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b4d091...82d76dd. Read the comment docs.

[hannahhoward]

LGTM but two comments I'm hoping you can answer before you hit that merge button...

• does an address lookup really change based on the TipSet key?
• just curious -- why the method rename?

[laser]

does an address lookup really change based on the TipSet key?

Yep, see here. A storage miner actor's worker address may be X at block height H and X' at H'. Passing the tipset key allows disambiguation when chain queries from the provider span multiple chain heads.

just curious -- why the method rename?

I changed the method name to match what's in go-storage-miner. I could change what's in go-storage-miner to match what's in go-fil-markets, I suppose - but I'm already making edits to this codebase. What do you prefer?

[laser]

Oops. I linked to the wrong part of specs-actors. See this link for more info.

[hannahhoward]

all good! LGTM