Ticket #603 (assigned defect)

Opened 2 years ago

Last modified 7 months ago

yelp's test script is broken

Reported by: mario Owned by: ddp
Priority: major Milestone:
Component: extensions Version: 4.7.x
Keywords: yelp Cc:

Change History

Changed 22 months ago by ddp

  • status changed from new to assigned

Yes and no. The issue is that the Yelp API requires a developer key. For the obvious reason, I don't want to check mine in, so I wrote the test script such that if you go and get one and put it in a magically named file in the right place all will work. Not particularly useful for regression testing I realize, but better than not checking in the test script at all, methinks. Maybe I should go see if I can get another API key out of Yelp.

Changed 19 months ago by felix

  • milestone 4.8.0 deleted

Changed 7 months ago by ddp

I asked Yelp for an additional API key. If they give me one (and assuming it's a v1 API YWSID), I'll check it into the test script. Otherwise I'm going to close this ticket because nothing's really broken. A while back I added additional comments about what's going on in the test script and in the Wiki entry for the egg. Everyone using this egg needs their own private YWSID key from Yelp. See http://www.yelp.com/developers for more information.

Note: See TracTickets for help on using tickets.