Skip to content

Add die.py tests#1042

Merged
arkid15r merged 9 commits intoOWASP:masterfrom
pUrGe12:die-tests
Jun 12, 2025
Merged

Add die.py tests#1042
arkid15r merged 9 commits intoOWASP:masterfrom
pUrGe12:die-tests

Conversation

@pUrGe12
Copy link
Contributor

@pUrGe12 pUrGe12 commented Mar 11, 2025

Proposed change

Implementing test cases for die.py. Tested die_success and die_failure.

Type of change

  • New core framework functionality
  • Bugfix (non-breaking change which fixes an issue)
  • Code refactoring without any functionality changes
  • New or existing module/payload change
  • Localization improvement
  • Dependency upgrade
  • Documentation improvement

Checklist

  • I've followed the contributing guidelines
  • I've run make pre-commit, it didn't generate any changes
  • I've run make test, all tests passed locally

@arkid15r arkid15r enabled auto-merge June 12, 2025 00:06
@arkid15r arkid15r changed the title created tests for die.py Add die.py tests Jun 12, 2025
@arkid15r arkid15r added this pull request to the merge queue Jun 12, 2025
Merged via the queue into OWASP:master with commit e419d22 Jun 12, 2025
17 checks passed
pUrGe12 added a commit to pUrGe12/Nettacker that referenced this pull request Jun 12, 2025
* created tests for die.py

* updated

* migrate to pytest

* Update deps

* Revert poetry.lock

---------

Signed-off-by: Achintya Jai <153343775+pUrGe12@users.noreply.github.com>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
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