Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

feat(prover): use httptest.Server to simplify the prover web server test code #389

Merged
merged 12 commits into from
Sep 28, 2023

Conversation

alexshliu
Copy link
Contributor

No description provided.

@alexshliu alexshliu changed the title feat: use httptest.Server to simplify the prover web server test code feat(prover): use httptest.Server to simplify the prover web server test code Sep 14, 2023
@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #389 (d54676e) into main (bdca930) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
- Coverage   50.44%   50.42%   -0.02%     
==========================================
  Files          39       39              
  Lines        5000     4998       -2     
==========================================
- Hits         2522     2520       -2     
  Misses       2223     2223              
  Partials      255      255              
Files Coverage Δ
prover/server/server.go 57.14% <ø> (-1.69%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@alexshliu alexshliu marked this pull request as ready for review September 14, 2023 13:05
@cyberhorsey
Copy link
Contributor

can you resolve the conflicts @alexshliu

@alexshliu alexshliu force-pushed the feat/fix-prover-httpserver-test branch from 5038b0f to c5e4fa2 Compare September 27, 2023 02:38
@alexshliu
Copy link
Contributor Author

can you resolve the conflicts @alexshliu

fixed

@davidtaikocha davidtaikocha merged commit 84eedae into main Sep 28, 2023
4 checks passed
@davidtaikocha davidtaikocha deleted the feat/fix-prover-httpserver-test branch September 28, 2023 07:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants