search

bastion-dev / Bastion

Java test library for HTTP APIs
http://bastion.rocks
GNU General Public License v3.0
13 stars 8 forks source link

Remove the BastionRunner before our first official release #61

Closed KPull closed 7 years ago

KPull commented 7 years ago

BastionRunner, our JUnit Test Runner has turned out to be quite the letdown. A JUnit Runner needs to prepare a test complete Description to show in the IDE's GUI before the tests start to run. We wanted to show each Bastion call as it happened in the IDE's GUI but it appears we cannot do it.

Also, poor documentation means that tests running using BastionRunner are causing bugs such as #49. I believe we should just get rid of BastionRunner for version 1.0 of Bastion since it's not worth the effort of maintaining for now.

What does everyone else think?

KPull commented 7 years ago

By the way, I think, in this case, we can get rid of all the descriptive texts we have in Bastion since they won't really be used for anything now. This includes Bastion.request(name, request) and HttpRequest.name().

KPull commented 7 years ago

Merged in #87.