Skip to content

feat: Add tests for RobotsNode and update test setup #362

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 9, 2024

Conversation

tejhande
Copy link
Contributor

@tejhande tejhande commented Jun 9, 2024

  • Added pytest fixture to set up the RobotsNode with the initial state.
  • Implemented test_robots_node to test the execution of RobotsNode.
  • Used unittest.mock.patch to mock the execute method, ensuring faster and more reliable tests without actual network calls.
  • Added assertions to verify the correctness of the result and ensure the execute method is called once with the correct arguments.

- Added pytest fixture to set up the RobotsNode with the initial state.
- Implemented test_robots_node to test the execution of RobotsNode.
- Used unittest.mock.patch to mock the execute method, ensuring faster and more reliable tests without actual network calls.
- Added assertions to verify the correctness of the result and ensure the execute method is called once with the correct arguments.
@VinciGit00
Copy link
Collaborator

hi, thank you. can you add also a test for search_link_node_test.py?

@VinciGit00 VinciGit00 merged commit e688480 into ScrapeGraphAI:main Jun 9, 2024
2 checks passed
Copy link

github-actions bot commented Jun 9, 2024

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants