Hockey-Scraper icon indicating copy to clipboard operation
Hockey-Scraper copied to clipboard

ESPN Api link broken - Failing integration tests

Open austinegri opened this issue 1 year ago • 4 comments

References

Integration tests are failing as the ESPN api link no longer returns game data:

  • http://www.espn.com/nhl/gamecast/data/masterFeed?lang=en&isAll=true&gameId=400885300
  • Failing tests
  • Link usage here

Tested with current 2023 gameId and found link was broken for this as well so ESPN must've changed the url.

Screenshot 2023-04-07 at 13-10-36 ESPN - Serving Sports Fans  Anytime  Anywhere

Acceptance Criteria

  • Proper link found and replaced to fetch ESPN game data

austinegri avatar Apr 07 '23 17:04 austinegri

Perhaps ESPN playbyplay tab could be the answer here i.e. https://www.espn.com/nhl/playbyplay/_/gameId/401459832

austinegri avatar Apr 07 '23 17:04 austinegri

Hey @austinegri!

Thanks for pointing this out. I actually noted this in one of my last commits (see b59c458e346eae604a8c581269b5ef5df74fb342). I don't think this should be a problem for most recent seasons as I believe the NHL has done a better job of recording the coordinates in the API game feed.

I'm currently very busy with work so I won't have a chance to take a closer look until the summer.

Regards, Harry

HarryShomer avatar Apr 11 '23 20:04 HarryShomer

Thanks for the reply @HarryShomer!

My bad I didn't see the recent commit. No worries I understand work is priority, this is a side project and figured it would be good to have a tracking issue.

Do you know around which season would start to have most of the data?

austinegri avatar Apr 12 '23 14:04 austinegri

Do you know around which season would start to have most of the data?

Tbh I'm not sure. The NHL has been known to go back and fix old games.

Your best bet is to just try different seasons and see what happens.

HarryShomer avatar Apr 14 '23 02:04 HarryShomer