Skip to content

fix: adjust tests of scrapy user data#284

Merged
janbuchar merged 1 commit into
masterfrom
fix-scrapy-user-data-tests
Oct 2, 2024
Merged

fix: adjust tests of scrapy user data#284
janbuchar merged 1 commit into
masterfrom
fix-scrapy-user-data-tests

Conversation

@janbuchar

@janbuchar janbuchar commented Oct 2, 2024

Copy link
Copy Markdown
Contributor

@janbuchar janbuchar added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Oct 2, 2024
@janbuchar
janbuchar requested a review from vdusek October 2, 2024 14:22
@github-actions github-actions Bot added this to the 99th sprint - Tooling team milestone Oct 2, 2024
@github-actions github-actions Bot added the tested Temporary label used only programatically for some analytics. label Oct 2, 2024

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Pull Request Tookit has failed!

If issue is not linked to the pull request then estimate the pull request!

@janbuchar janbuchar changed the title Fix: tests of scrapy user data fix: tests of scrapy user data Oct 2, 2024
@janbuchar janbuchar changed the title fix: tests of scrapy user data fix: adjust tests of scrapy user data Oct 2, 2024
assert apify_request.url == 'https://example.com'

user_data = apify_request.user_data
assert isinstance(user_data, dict)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get it why are we removing this? thanks

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UserData won't pass the check

@janbuchar
janbuchar merged commit 26ffb15 into master Oct 2, 2024
@janbuchar
janbuchar deleted the fix-scrapy-user-data-tests branch October 2, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants