Skip to content
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

Add tests #42

Merged
merged 3 commits into from
Apr 17, 2024
Merged

Add tests #42

merged 3 commits into from
Apr 17, 2024

Conversation

codingjoe
Copy link
Member

@codingjoe codingjoe commented Apr 17, 2024

Test Fail GIF by P!NK

@codingjoe codingjoe self-assigned this Apr 17, 2024
@codingjoe codingjoe requested a review from amureki April 17, 2024 14:00
@codingjoe codingjoe force-pushed the tests branch 2 times, most recently from 0989eeb to 3c26e73 Compare April 17, 2024 14:03
Copy link

codecov bot commented Apr 17, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

sam/slack.py Show resolved Hide resolved
sam/utils.py Show resolved Hide resolved
Copy link
Member

@amureki amureki left a comment

Choose a reason for hiding this comment

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

Fine by me

@codingjoe codingjoe merged commit b5267a4 into main Apr 17, 2024
5 checks passed
@codingjoe codingjoe deleted the tests branch April 17, 2024 14:42
@amureki amureki mentioned this pull request Apr 18, 2024
Copy link

sentry-io bot commented Apr 18, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: AsyncMessages.create() got an unexpected keyword argument 'file_ids' sam.slack in handle_message View Issue
  • ‼️ Failed to run listener function (error: Error code: 400 - {'error': {'message': "Can't add messag... sam.slack in handle_message View Issue
  • ‼️ HTTPError: 403 Client Error: Forbidden for url: https://www.apollotechnical.com/statistics-on-work-life-bala... sam.tools in fetch_website View Issue
  • ‼️ BadRequestError: Error code: 400 - {'error': {'message': 'Thread thread_2HTOGTmuCghX3RhEJSw87HeX already has an ac... sam.bot in run View Issue
  • ‼️ SSLError: HTTPSConnectionPool(host='www.laciteduvin.com', port=443): Max retries exceeded with url: /en/inf... sam.tools in fetch_website View Issue

Did you find this useful? React with a 👍 or 👎

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.

2 participants