mandiant / mandiant/capa

Add tests (and refactor) API extraction and symbol generation

Open
#1,899 23 comments 0 reactions 1 assignee Claimed by @v1bh475u View on GitHub
enhancement
Dominant language
Python
Stars
6.2k
Forks
726
Avg merge
11d 11h
Merged PRs (30d)
7

Description

> the routine you're updating is starting to look a little complicated, so maybe soon it's time to attempt to refactor, but without tests, i wouldn't be comfortable. even one trivial test would encode the special case/scenario that you encountered.

_Originally posted by @williballenthin in https://github.com/mandiant/capa/issues/1897#issuecomment-1848265238_

We should have more tests to encode all the various API patterns we support.

Refactoring of the existing routines and maybe introducing separate functions for .NET vs. native vs. others may make sense as part of that.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.