Page MenuHomeElementl

[dagit] Repair duplicate keys in App.test
ClosedPublic

Authored by dish on Jul 6 2021, 6:23 PM.

Details

Summary

Try to fix App.test.

Test Plan

yarn test App.test

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish requested review of this revision.Jul 6 2021, 6:33 PM

This at least eliminates the duplicate key problem, which maybe gets us back under the timeout limit? It's weird to me that App.test does pass with this change, and it's the more correct thing to do here anyway, but I don't really know if it will repair master. ๐Ÿ™

This revision is now accepted and ready to land.Jul 6 2021, 7:42 PM
dish retitled this revision from [dagit] Fix App.test to [dagit] Repair duplicate keys in App.test.Jul 6 2021, 7:47 PM
This revision was automatically updated to reflect the committed changes.