Include test requirements only if we are building that node

[isuruf]

Fixes #1331

[codecov]

Codecov Report

Merging #1332 (6930f03) into master (f0214de) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1332      +/-   ##
==========================================
- Coverage   57.65%   57.62%   -0.03%     
==========================================
  Files          66       66              
  Lines        6570     6573       +3     
==========================================
  Hits         3788     3788              
- Misses       2782     2785       +3     

Impacted Files	Coverage Δ
conda_forge_tick/migrators/migration_yaml.py	55.79% <0.00%> (-0.73%)	⬇️

---

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 f0214de...6930f03. Read the comment docs.

[beckermr]

Why are we also keeping the build/host here too? I think we can get rid of those as well for nodes we do not build, right?

[beckermr]

We can remove those in another PR if we want.

[beckermr]

Thanks @isuruf for the clever fix!

[isuruf]

Why are we also keeping the build/host here too? I think we can get rid of those as well for nodes we do not build, right?

Depends on if the code here handles run_exports correctly or not.

[h-vetinari]

Very cool, thanks!

[beckermr]

I think we handle run exports correctly now when building the graph nodes. Or at least we are doing it better than before.

I'll make a PR to remove them and we can see what happens.