search

matsim-org / matsim-libs

Multi-Agent Transport Simulation
www.matsim.org
477 stars 445 forks source link

fix: personMoney event reference lookup missing in EventsReader #3258

Closed mfrawley-moia closed 4 months ago

mfrawley-moia commented 5 months ago

We (Moia) discovered that the reference attribute for personMoney events was always null when parsed by the EventsReaderv1 class. I added a hopefully-appropriate unit test for it in EventsReadersTest, and adjusted the events.xml fixture accordingly.

Feedback is more than welcome! cc @nkuehnel @fzwick

nkuehnel commented 5 months ago

Thanks Mark, looks good to me : )