-
Notifications
You must be signed in to change notification settings - Fork 101
sqli test case generator #1407
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
sqli test case generator #1407
Conversation
omursahin
commented
Dec 18, 2025
| if (ind.individual is GraphQLIndividual) { | ||
| ind.evaluatedMainActions().forEachIndexed { index, a -> | ||
| handleSingleCall(a, index, ind.fitness, lines, testCaseName, testSuitePath, baseUrlOfSut) | ||
| handleSingleCall(a, index, ind.fitness, lines, testCaseName, testSuitePath, baseUrlOfSut, false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need 1 E2E under core-tests/e2e-tests/spring/spring-rest-bb to make sure the generation of tests works fine for all test output formats
| if(username.contains("pg_sleep")) { | ||
| //simulating delay | ||
| CoveredTargets.cover("sqli") | ||
| Thread.sleep(8000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need to add comment to specify why we do this... ie cannot currently run real database due to Docker issues on OS in GitHub action. but, can leave TODO stating we could update/replace it one day if can safely run Docker on GA
| package com.foo.rest.examples.bb.sqli | ||
|
|
||
|
|
||
| data class UserDto( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these classes are not used... can leave a TODO comment stating might be needed if one day can run BB tests with Docker on GitHub Actions