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 more integration test cases #973

Open
mileszhang2016 opened this issue Jan 26, 2022 Discussed in #968 · 2 comments
Open

add more integration test cases #973

mileszhang2016 opened this issue Jan 26, 2022 Discussed in #968 · 2 comments

Comments

@mileszhang2016
Copy link
Member

Discussed in #968

Originally posted by kwanhur January 23, 2022
when add some feature into bfe_server, it seems to add test case uncomfortably, needs to mock a mass of struct case for BfeServer, any other friendly ways?

@kwanhur
Copy link
Contributor

kwanhur commented Feb 20, 2022

There MockServer locate at bfe_util/mock_server, but seems lack relate usage doc.

I found two places had used it, bfe_http2/server_test and bfe_websocket.

When modification on bfe_server, good for a trial with the MockServer.

@mileszhang2016
Copy link
Member Author

The requirement of "Add support for setting up context (e.g., bfe_config, bfe_server) to ease unit testing",
has been changed to "add more integration test cases"

@mileszhang2016 mileszhang2016 changed the title Add support for setting up context (e.g., bfe_config, bfe_server) to ease unit testing add more integration test cases Mar 7, 2022
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

No branches or pull requests

2 participants