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 unit-tests #1

Closed
postmodern opened this issue Nov 23, 2013 · 4 comments · Fixed by #145
Closed

Add unit-tests #1

postmodern opened this issue Nov 23, 2013 · 4 comments · Fixed by #145

Comments

@postmodern
Copy link
Contributor

All code must have tests.

@ghost ghost assigned awruef Nov 23, 2013
@dguido dguido assigned jkolek and unassigned awruef Mar 9, 2017
@jkolek
Copy link
Contributor

jkolek commented Mar 26, 2017

Is there an unit testing framework that you would prefer?

@dguido
Copy link
Member

dguido commented Mar 27, 2017

Let's come back to this issue later. We will probably want to setup an archive of PE files and host them on S3, then run pe-parse on them.

@woodruffw
Copy link
Member

Revisiting this: I put together a zero-config googletest harness for uthenticode that we could reuse here.

We also have now have pegoat with daily auto-builds for a suite of PE binaries.

@woodruffw
Copy link
Member

We'll close this with #145, since it'll add an initial test suite. Coverage refinements can be handled by subsequent issues and PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants