Skip to content

Fix/216 double bet overwrite#381

Open
Jayydy wants to merge 2 commits intoHahfyeex:Defaultfrom
Jayydy:fix/216-double-bet-overwrite
Open

Fix/216 double bet overwrite#381
Jayydy wants to merge 2 commits intoHahfyeex:Defaultfrom
Jayydy:fix/216-double-bet-overwrite

Conversation

@Jayydy
Copy link
Copy Markdown
Contributor

@Jayydy Jayydy commented Mar 27, 2026

Summary

Brief description of what this PR does.

setup() now returns 6-tuple including admin, uses mock_all_auths_allowing_non_root_auth()
initialize() now bootstraps all roles (FeeSetter, Resolver, Pauser) to admin so standalone tests work without manual role assignment
create_market no longer double-calls require_auth (removed redundant call since require_role already handles it)
All test helpers and standalone tests updated to use admin as creator and pass correct caller args
Double-bet test amounts updated to use LMSR-scale values (5_000_000 instead of 50)

Related Issue

Closes #309

Type of Change

  • [x ] Bug fix
  • New feature
  • Refactor
  • Documentation update
  • Other (describe):

Changes Made

Testing

Describe how you tested your changes.

Checklist

  • [ x] My code follows the project's style guidelines
  • [ x] I have performed a self-review of my code
  • [x ] I have commented complex logic where necessary
  • [x ] I have updated documentation if needed
  • [ x] My changes don't introduce new warnings or errors

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 27, 2026

@Jayydy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Hahfyeex
Copy link
Copy Markdown
Owner

@Jayydy resolve conflicts

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.

[BUG] #216 - Double Bet Allowed on Same Market

2 participants