Closed Bug 862029 Opened 11 years ago Closed 11 years ago

[MakeAPI] Add tests for Grunt to run

Categories

(Webmaker Graveyard :: MakeAPI, defect, P1)

Product:
Webmaker Graveyard
Component:
MakeAPI
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: mjschranz, Assigned: mjschranz)

Details

(Whiteboard: u=dev c=makeAPI p=1 s=2013w17) 

In the repro restructuring I removed all the tests that existed because there wasn't much point in them anymore. However we will want testing to exist here in some fashion and we should add it back sometime this week.
Whiteboard: u=dev c=makeAPI p=1 s=2013w16
Priority: -- → P1
Whiteboard: u=dev c=makeAPI p=1 s=2013w16 → u=dev c=makeAPI p=1 s=2013w17
Going to be giving https://github.com/dscape/specify a try as Jon recommended against using Tap for tests like we do with Popcorn Maker.
Status: NEW → ASSIGNED
Talking with :humph we both agreed that right now these are not worth the effort and won't help advance what we need finished right now. We all want tests but in order to get them effectively done I'm basically forced to change a lot of our server code simply for the sake of testing.

Throw in the fact that the API itself is still changing and evolving it just doesn't seem that the gain is worth it right now. We plan on filing something for the future to be done in later weeks.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Yeah, let's file a new bug when things are churning so much in this API.  I want Matt's efforts elsewhere more at the moment.  +1
You need to log in before you can comment on or make changes to this bug.