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

Large User Testcase #180

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from
Draft

Large User Testcase #180

wants to merge 21 commits into from

Conversation

Atipriya
Copy link
Collaborator

closes #105

@mahaloz
Copy link
Member

mahaloz commented Aug 22, 2022

I want someone to pick this back up and refactor some of the code. I think this is close to done. The test case works right now but does not validate a magic sync worked with random data set for a variety of users.

TODO

  1. Generate random data for m users and then set them for each users
  2. Complete a magic_sync with some priority and validate that it actually works in the intended way
  3. Remove binaries from tests that we don't intend to use
  4. Maybe merge the decompile_angr into a single test file, since it's only one function. That or we should just make a new file called utils or common like angr-management tests does in their repo

@mahaloz mahaloz marked this pull request as draft August 22, 2022 23:16
@Atipriya
Copy link
Collaborator Author

I'll work on this.

@mahaloz
Copy link
Member

mahaloz commented Aug 26, 2022

I'll work on this.

You are the GOAT!

@Atipriya
Copy link
Collaborator Author

I'll get this done next week for sure!

@mahaloz
Copy link
Member

mahaloz commented Apr 29, 2023

@Atipriya that would be epic! But also there is probably a more pressing feature for BinSync that involves a certain research project you've done that you should ping me about next week :)

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.

Large User Testcase
5 participants