indiehd / web-api

GNU Affero General Public License v3.0
6 stars 4 forks source link

Rework OrderControllerTest class #185

Open cbj4074 opened 4 years ago

cbj4074 commented 4 years ago

This class's setUp() method calls $this->markTestSkipped('The Order implementation is being reworked');, added as a temporary holdover during the Velkart package integration process.

This call needs to removed, and the relevant tests updated/reworked, as appropriate.

cbj4074 commented 3 years ago

After further scrutiny, my instinct is that all of this functionality (and accompanying tests) should be moved into the Velkart package. I'll post another update soon, once I've worked through it. (I'm currently adding a Velkart section to the documentation, which will elucidate the details.)

mblarsen commented 3 years ago

Sounds reasonable :+1: