Skip to content

Add ip.py tests#1075

Merged
arkid15r merged 6 commits intoOWASP:masterfrom
pUrGe12:test-ip
Jun 11, 2025
Merged

Add ip.py tests#1075
arkid15r merged 6 commits intoOWASP:masterfrom
pUrGe12:test-ip

Conversation

@pUrGe12
Copy link
Contributor

@pUrGe12 pUrGe12 commented Jun 4, 2025

Proposed change

Test file for ip.py

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 changed the title tests for ip.py Add ip.py tests Jun 11, 2025
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

// @pUrGe12 please use assert ... and assert not ... for true/false cases.

@arkid15r arkid15r enabled auto-merge June 11, 2025 00:30
@arkid15r arkid15r added this pull request to the merge queue Jun 11, 2025
Merged via the queue into OWASP:master with commit 6244176 Jun 11, 2025
17 checks passed
pUrGe12 added a commit to pUrGe12/Nettacker that referenced this pull request Jun 12, 2025
* tests for ip.py

* migrate to pytest

* Update asserts

* Update tests

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
@pUrGe12 pUrGe12 deleted the test-ip branch July 12, 2025 12:15
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