Skip to content

Commit

Permalink
ci(iast): enable iast_aggregated_leak_testing tests (#12005)
Browse files Browse the repository at this point in the history
Enable `iast_aggregated_leak_testing`

## Checklist
- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] Reviewer has checked that all the criteria below are met 
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)
  • Loading branch information
avara1986 authored Jan 22, 2025
1 parent 57c43a7 commit afad56b
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 7 deletions.
13 changes: 7 additions & 6 deletions hatch.toml
Original file line number Diff line number Diff line change
Expand Up @@ -444,8 +444,8 @@ fastapi = ["~=0.114.2"]

## ASM Appsec Aggregated Leak Testing

[envs.appsec_aggregated_leak_testing]
template = "appsec_aggregated_leak_testing"
[envs.iast_aggregated_leak_testing]
template = "iast_aggregated_leak_testing"
dependencies = [
"pytest",
"pytest-cov",
Expand All @@ -457,19 +457,20 @@ dependencies = [
"pydantic-settings",
]

[envs.appsec_aggregated_leak_testing.env-vars]
[envs.iast_aggregated_leak_testing.env-vars]
CMAKE_BUILD_PARALLEL_LEVEL = "12"
DD_IAST_ENABLED = "true"
_DD_IAST_PATCH_MODULES = "scripts.iast"

[envs.appsec_aggregated_leak_testing.scripts]
[envs.iast_aggregated_leak_testing.scripts]
test = [
"uname -a",
"pip freeze",
"python -m pytest tests/appsec/iast_aggregated_memcheck/test_aggregated_memleaks.py",
]

[[envs.appsec_aggregated_leak_testing.matrix]]
python = ["3.10", "3.11", "3.12", "3.13"]
[[envs.iast_aggregated_leak_testing.matrix]]
python = ["3.10", "3.11", "3.12"]



Expand Down
8 changes: 7 additions & 1 deletion tests/appsec/suitespec.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,12 +73,18 @@ suites:
- '@remoteconfig'
retry: 2
runner: hatch
iast_aggregated_leak_testing:
parallelism: 3
paths:
- '@appsec_iast'
- tests/appsec/iast_aggregated_memcheck/*
runner: hatch
timeout: 50m
appsec_iast_packages:
parallelism: 4
paths:
- '@appsec_iast'
- tests/appsec/iast_packages/*
retry: 2
runner: hatch
timeout: 50m
appsec_integrations_pygoat:
Expand Down

0 comments on commit afad56b

Please sign in to comment.