Skip to content

fix: replace IMDb with TMDB and fix one test assertion#2393

Open
honzajavorek wants to merge 5 commits intomasterfrom
honzajavorek/fix-ex-tests
Open

fix: replace IMDb with TMDB and fix one test assertion#2393
honzajavorek wants to merge 5 commits intomasterfrom
honzajavorek/fix-ex-tests

Conversation

@honzajavorek
Copy link
Copy Markdown
Collaborator

@honzajavorek honzajavorek commented Apr 2, 2026

On 1st April the tests for Academy exercises have failed:

  • IMDb now employs anti-scraping measures, so I replaced it with TMDB
  • I fixed some assertions so they're not checking data, just structure. The exercise works fine, but the assertion assumed too much about the output and was fragile.

I verified everything passes now.

@honzajavorek honzajavorek requested a review from TC-MO as a code owner April 2, 2026 17:50
@honzajavorek honzajavorek changed the title Replace IMDb with TMDB and fix one test assertion fix: replace IMDb with TMDB and fix one test assertion Apr 2, 2026
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit fb3640b and is ready at https://pr-2393.preview.docs.apify.com!

@honzajavorek honzajavorek requested a review from B4nan April 2, 2026 17:57
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit 339fe6d and is ready at https://pr-2393.preview.docs.apify.com!

@B4nan
Copy link
Copy Markdown
Member

B4nan commented Apr 2, 2026

I will surely forget about this, so please poke me after the holidays:)

@honzajavorek
Copy link
Copy Markdown
Collaborator Author

@B4nan sure thing! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants