protein tests

[yosefdroth]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 836

• 67 of 68 (98.53%) changed or added relevant lines in 4 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.2%) to 91.521%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/scenes/BiochemicalEntityDetails/Rna/MetadataSection.js	8	9	88.89%

Files with Coverage Reduction	New Missed Lines	%
src/components/SearchForm/SearchForm.js	1	95.71%

Totals
Change from base Build 831:	0.2%
Covered Lines:	1125
Relevant Lines:	1186

---

💛 - Coveralls

[jonrkarr]

The branch is not ready for merging. The metabolite tests are failing, at least in part because the importing of the fixture has been deleted.

Todo

• Fix the failing tests
• Expand coverage. As necessary, use additional fixtures. Examples:
  • Rna/MetadataSection.js: formatMetadata, formatMetadataInner
  • Protein/MetadataSection.js: formatMetadataInner`
  • Reaction/MetadataSection.js: formatMetadataInner
• Remove commented out examples in tests such as below to remove potential sources of confusion

  //expect(formattedData[7].source).toEqual({
  //  source: "ymdb",
  //  id: "ymdb_id_xxx"
  //});
  

• Merge the branch with master and fix additional failures with the RNA and reaction tests