-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Feat/comprehensive test suite resolved #677
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
Open
rohanrlobo
wants to merge
20
commits into
deepseek-ai:main
Choose a base branch
from
rohanrlobo:feat/comprehensive-test-suite-resolved
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feat/comprehensive test suite resolved #677
rohanrlobo
wants to merge
20
commits into
deepseek-ai:main
from
rohanrlobo:feat/comprehensive-test-suite-resolved
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit introduces a new Python application that automates sending personalized WhatsApp messages from an Excel file. The application includes the following features: - A file dialog to select the Excel file with contact information. - The ability to read contact data from an Excel file. - Automation of WhatsApp Web using Selenium to send messages. - Personalization of messages using placeholders. - A random delay between messages to reduce the risk of being banned. - A log file to track the status of each message. - Automated management of the ChromeDriver. - A detailed README with setup and usage instructions.
This commit packages the WhatsApp automation application into a tar.gz archive for easy distribution. It also includes instructions for users on how to publish the application to a new GitHub repository.
Create WhatsApp Automation Application
Package WhatsApp Automation Application
This commit fixes a bug in the message sending functionality. The previous implementation used URL encoding to pre-fill the message, which is unreliable and can fail if WhatsApp changes its URL structure. This commit changes the implementation to: - Use a more robust XPath selector to find the message box. - Use the `send_keys` method to type the message into the message box. - Clicks the send button. A new unit test has been added to verify this functionality.
Fix: Use send_keys to send messages
Adds the requirements.txt file to the whatsapp_automation directory to specify project dependencies.
Feat: Add requirements.txt file
This commit enhances the error logging functionality of the script. The `send_whatsapp_message` function now returns a tuple containing a boolean for success/failure and a status message. This allows for more detailed, human-readable error messages to be logged in `message_log.csv`. The main loop has been updated to handle this new return format and log the appropriate status message. The unit tests have been updated to reflect these changes, including a new test case to verify that failures are handled and logged correctly.
Feat: Add detailed error logging
This commit enhances the error logging functionality of the script and resolves merge conflicts. The `send_whatsapp_message` function now returns a tuple containing a boolean for success/failure and a status message. This allows for more detailed, human-readable error messages to be logged in `message_log.csv`. The main loop has been updated to handle this new return format and log the appropriate status message. The unit tests have been updated to reflect these changes, including a new test case to verify that failures are handled and logged correctly. This commit also resolves merge conflicts that arose from other changes being made to the codebase.
This commit enhances the error logging functionality of the script and resolves merge conflicts. The `send_whatsapp_message` function now returns a tuple containing a boolean for success/failure and a status message. This allows for more detailed, human-readable error messages to be logged in `message_log.csv`. The main loop has been updated to handle this new return format and log the appropriate status message. The unit tests have been updated to reflect these changes, including a new test case to verify that failures are handled and logged correctly. This commit also resolves merge conflicts that arose from other changes being made to the codebase.
…olved Feat: Add detailed error logging and resolve conflicts
This commit adds a comprehensive test suite to the project. The test suite now includes test cases for: - Successful message sending - Handling of timeouts when sending messages - Automatic phone number formatting - Reading valid Excel files - Handling of missing Excel files This expanded test suite provides greater confidence in the reliability and robustness of the application.
This commit adds a comprehensive test suite to the project and resolves all merge conflicts that arose during development. The test suite now includes test cases for: - Successful message sending - Handling of timeouts when sending messages - Automatic phone number formatting - Reading valid Excel files - Handling of missing Excel files All merge conflicts have been manually resolved by combining the incoming changes with the new test suite, and the final code has been fully tested to ensure correctness.
Feat: Add a comprehensive test suite
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.