Skip to content

Bump Bandit version and fix incomplete test assertions#894

Merged
solnic merged 2 commits into
getsentry:masterfrom
martosaur:am-minor-test-fix
May 27, 2025
Merged

Bump Bandit version and fix incomplete test assertions#894
solnic merged 2 commits into
getsentry:masterfrom
martosaur:am-minor-test-fix

Conversation

@martosaur
Copy link
Copy Markdown
Contributor

@martosaur martosaur commented May 23, 2025

Just a bunch of very minor fixes I ran into while investigating something:

  • Bumping Bandit also bumped Plug, which deprecated Plug.Test.
  • Couple of tests related to duplicated reporting only asserted one event instead of 2

#skip-changelog

@solnic solnic self-requested a review May 26, 2025 09:45
Copy link
Copy Markdown
Collaborator

@solnic solnic left a comment

Choose a reason for hiding this comment

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

Thank you 😄

@martosaur martosaur force-pushed the am-minor-test-fix branch from 58f555e to 0ae29f6 Compare May 26, 2025 17:00
@martosaur
Copy link
Copy Markdown
Contributor Author

rebased 🫡

@solnic solnic merged commit f85b956 into getsentry:master May 27, 2025
6 checks passed
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