redruin1 / factorio-draftsman

A complete, well-tested, and up-to-date module to manipulate Factorio blueprint strings. Compatible with mods.
MIT License
94 stars 17 forks source link

loading blueprint from dict did not convert connection ids from numbers to associations #29

Closed elswindle closed 2 years ago

elswindle commented 2 years ago

moved this conversion from out of load_from_string to setup. This will let loading from dict also convert connections/schedules to Associations. Test passed, ran black.

Additionally, IOTypeMixin was not looking for "type", not "io_type", inside of the Entity dict for underground belts. This caused them to always be inputs and never outputs. This fix broke test, and I don't really understand why. The fix is correct though.