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

flow: Unit tests needed. #82

Open
ygoldfeld opened this issue Mar 10, 2024 · 0 comments
Open

flow: Unit tests needed. #82

ygoldfeld opened this issue Mar 10, 2024 · 0 comments
Labels
bug Something isn't working from-akamai-pre-open Issue origin is Akamai, before opening source test Unit and functional tests; demo/example programs

Comments

@ygoldfeld
Copy link
Contributor

Filed by @ygoldfeld pre-open-source:

The good news is Flow has been widely tested with tons of integration testing, elsewhere. It has proven itself solid in practice.

The bad news is it lacks systematic unit testing which reduces confidence and increases risk of regressions. As an open-source project this is a deficit. Flow is manageably-sized, and it is not a hugely difficult task to resolve this. We have a pretty large GoogleTest-based unit test suite in sister/dependent project, Flow-IPC.

Since net_flow is a bit of an outlier as essentially a demo/niche module, it could be excluded from this work at first (separate ticket to handle it later/separately - TODO).

@ygoldfeld ygoldfeld added bug Something isn't working from-akamai-pre-open Issue origin is Akamai, before opening source labels Mar 10, 2024
@ygoldfeld ygoldfeld added the test Unit and functional tests; demo/example programs label Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working from-akamai-pre-open Issue origin is Akamai, before opening source test Unit and functional tests; demo/example programs
Projects
None yet
Development

No branches or pull requests

1 participant