If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[MakeAPI] Add tests for Grunt to run

RESOLVED WONTFIX

Status

Webmaker
MakeAPI
P1
normal
RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: mjschranz, Assigned: mjschranz)

Tracking

Details

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

(Assignee)

Description

5 years ago
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.

Updated

5 years ago
Whiteboard: u=dev c=makeAPI p=1 s=2013w16

Updated

5 years ago
Priority: -- → P1
Whiteboard: u=dev c=makeAPI p=1 s=2013w16 → u=dev c=makeAPI p=1 s=2013w17
(Assignee)

Comment 1

5 years ago
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
(Assignee)

Comment 2

5 years ago
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
Last Resolved: 5 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.