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

[receiver/purefa] Ensuring test efficiency for the endpoints scraped on Mock server #23270

Closed
wants to merge 3 commits into from

Conversation

dgoscn
Copy link
Contributor

@dgoscn dgoscn commented Jun 11, 2023

Description:
It is crucial to validate that the httpserver library effectively scrapes all the necessary data from the endpoints during the Mock test. This validation ensures the accuracy and completeness of the purefareceiver.

Link to tracking Issue:
#14886

Testing:
The validation test must be made by the default configs on config.yaml that is present on README.md and using Debugging session from the VSCode.

Documentation:
Present on README

@dgoscn dgoscn requested a review from a team June 11, 2023 03:40
@dgoscn dgoscn requested a review from jpkrohling as a code owner June 11, 2023 03:40
@linux-foundation-easycla
Copy link

CLA Missing ID CLA Not Signed

  • ✅ login: dgoscn / name: Diego Amaral (51731f7fb9ba1bdb32cfb657a87e77ba833049a7)
  • ❌ The email address for the commit (c1989dbfe0611cfee735bab942ec4f624a49db15) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

@dgoscn dgoscn force-pushed the purefa_improv_scraper_test branch from c1989db to fdaf9f1 Compare June 11, 2023 04:22
@linux-foundation-easycla
Copy link

CLA Missing ID CLA Not Signed

@dgoscn dgoscn closed this Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants