simple-ups icon indicating copy to clipboard operation
simple-ups copied to clipboard

Introducing phpspec for test coverage.

Open Richtermeister opened this issue 11 years ago • 4 comments

This is a baby-step to introduce test coverage. phpSpec is an up-and-coming testing methodology with rapid growth of adoption over phpUnit.

If this approach is acceptable I would be happy to add more such tests, including realistic fixtures based on real UPS api responses.

Richtermeister avatar Jun 25 '14 06:06 Richtermeister

Ok, cool. So if I switch this to the "real" xml from the UPS api, and wrap the fixture loading into a re-usable class, this direction is acceptable? Have a great weekend! :smiley:

Richtermeister avatar Jun 28 '14 15:06 Richtermeister

Yeah, I'd say go for it.

bkuhl avatar Jun 28 '14 19:06 bkuhl

@Richtermeister I'm considering some overhauling on this package to make it more testable and add a Laravel service provider and some other things. Do you have any more phpspec work that hasn't been pushed up for this?

bkuhl avatar Dec 07 '15 15:12 bkuhl

Hey Ben, I currently don't.

Richtermeister avatar Dec 08 '15 18:12 Richtermeister