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.

Sign up for GitHub

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

Jump to bottom

Implement tests for helper functions - mapsutil #4

Closed
3 of 6 tasks
Mzack9999 opened this issue Oct 20, 2022 · 0 comments · Fixed by #17
Closed
3 of 6 tasks

Implement tests for helper functions - mapsutil #4

Mzack9999 opened this issue Oct 20, 2022 · 0 comments · Fixed by #17
Assignees
@LuitelSamikshya
Labels
Priority: Medium This issue may be useful, and needs some attention. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity.

Comments

@Mzack9999
Copy link
Member

Mzack9999 commented Oct 20, 2022 •
edited by LuitelSamikshya

Description

Implement tests for the following functions:

  • Flatten
  • Walk

Check if these functions are aligned with nuclei implementation, otherwise update them and implement tests:

  • HTTPToMap
  • DNSToMap
  • HTTPRequesToMap
  • HTTPResponseToMap
@Mzack9999 Mzack9999 added the Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity. label Oct 20, 2022
@Mzack9999 Mzack9999 transferred this issue from projectdiscovery/mapsutil Oct 23, 2022
@Mzack9999 Mzack9999 changed the title Implement tests for helper functions Implement tests for helper functions - mapsutil Oct 23, 2022
@Mzack9999 Mzack9999 transferred this issue from another repository Oct 31, 2022
@LuitelSamikshya LuitelSamikshya self-assigned this Nov 2, 2022
@ehsandeep ehsandeep added the Priority: Medium This issue may be useful, and needs some attention. label Nov 3, 2022
@LuitelSamikshya LuitelSamikshya linked a pull request Nov 9, 2022 that will close this issue
test cases addition mapsutil #17
Merged
@LuitelSamikshya LuitelSamikshya mentioned this issue Nov 9, 2022
Merged
@Mzack9999 Mzack9999 added the Status: Completed Nothing further to be done with this issue. Awaiting to be closed. label Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LuitelSamikshya LuitelSamikshya

Labels
Priority: Medium This issue may be useful, and needs some attention. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test cases addition mapsutil projectdiscovery/utils
3 participants
@ehsandeep @Mzack9999 @LuitelSamikshya