FIX: Add echo entity to MPM/MTS

[effigies]

By my reading of anat.yaml, this needs fixing. Also leading to an error in https://github.com/effigies/BIDS-examples/tree/qmri/qmri_mpm.

[tsalo]

LGTM! Not sure why the test is failing though.

[effigies]

Yeah, I restarted it and it still fails. @rwblair any ideas? It's possible that some dependency just made a release and broke everything. Will know that when you update #1183.

[effigies]

Just searching, we might need to look into something like jestjs/jest#8384 (comment) or jestjs/jest#9881 (comment).

[nellh]

Yeah, this looks like it's just inconsistent performance on Circle. A higher timeout can be passed as the third argument to the it() call - the two node spawn tests that reference the valid_headers dir should just have the timeout raised. It's probably slow just because it ends up accessing more of node_modules in the new node process than the other tests.

[codecov]

Codecov Report

Merging #1184 (441a88c) into master (96a7104) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1184   +/-   ##
=======================================
  Coverage   79.05%   79.05%           
=======================================
  Files          78       78           
  Lines        2617     2617           
  Branches      598      598           
=======================================
  Hits         2069     2069           
  Misses        407      407           
  Partials      141      141           

---

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 96a7104...441a88c. Read the comment docs.