Skip to content

DCE/RPC - Add more testing#4921

Open
gpotter2 wants to merge 5 commits intosecdev:masterfrom
gpotter2:dcerpc-tests
Open

DCE/RPC - Add more testing#4921
gpotter2 wants to merge 5 commits intosecdev:masterfrom
gpotter2:dcerpc-tests

Conversation

@gpotter2
Copy link
Member

  • add more tests to DCE/RPC
  • fix NTLM server checking of username
  • minor smbclient fix
  • improve endpoint mapper resilliency

@codecov
Copy link

codecov bot commented Feb 15, 2026

Codecov Report

❌ Patch coverage is 5.55556% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.23%. Comparing base (ded1d73) to head (4ed1329).

Files with missing lines Patch % Lines
scapy/layers/msrpce/rpcclient.py 0.00% 8 Missing ⚠️
scapy/layers/ntlm.py 0.00% 5 Missing ⚠️
scapy/config.py 0.00% 4 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (ded1d73) and HEAD (4ed1329). Click for more details.

HEAD has 9 uploads less than BASE
Flag BASE (ded1d73) HEAD (4ed1329)
11 2
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4921       +/-   ##
===========================================
- Coverage   80.10%   48.23%   -31.87%     
===========================================
  Files         370      352       -18     
  Lines       91727    91402      -325     
===========================================
- Hits        73477    44089    -29388     
- Misses      18250    47313    +29063     
Files with missing lines Coverage Δ
scapy/layers/smbclient.py 22.07% <100.00%> (-52.32%) ⬇️
scapy/config.py 63.26% <0.00%> (-15.16%) ⬇️
scapy/layers/ntlm.py 24.93% <0.00%> (-56.35%) ⬇️
scapy/layers/msrpce/rpcclient.py 11.22% <0.00%> (-46.12%) ⬇️

... and 318 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant