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.

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

Add more thorough testing #6

Open
dequbed opened this issue Sep 10, 2022 · 2 comments
Open

Add more thorough testing #6

dequbed opened this issue Sep 10, 2022 · 2 comments
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed

Comments

@dequbed
Copy link
Owner

dequbed commented Sep 10, 2022

As a reasonable approximation "more testing" = "more good" does hold. And rsasl already does some testing but it could definitely profit from more.

Especially as it's a rather abstract framework integration tests are relevant to make sure updates don't break stuff.

Especially integration tests that just test the client portion of a mechanism against its server portion should be a good place to start as they won't require knowledge of the inner workings of the mechanisms or rsasl itself.

@dequbed dequbed added help wanted Extra attention is needed good first issue Good for newcomers labels Sep 10, 2022
@duesee
Copy link
Contributor

duesee commented Nov 25, 2023

Would you be interested in a PR that adds fuzz-testing to rsasl? Fuzz-testing everything that is potentially exposed to untrusted data seems like a good start.

@dequbed
Copy link
Owner Author

dequbed commented Nov 28, 2023

Oh absolutely :)

I was looking into fuzz testing rsasl a while back, the main reason why I didn't go for it was that it currently doesn't expose any of the mechanisms directly so I felt like it'd mostly fuzz test the parsers. But that too would be a benefit so if you want to help with it, please do! ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants