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

test(*): remove http_server #11009

Merged
merged 1 commit into from
Jun 30, 2023
Merged

test(*): remove http_server #11009

merged 1 commit into from
Jun 30, 2023

Conversation

StarlightIbuki
Copy link
Contributor

@StarlightIbuki StarlightIbuki commented Jun 5, 2023

Summary

Replacing uses of helpers.http_server with the new implementation of HTTP mocking, which is more stable.

Checklist

  • The Pull Request has tests

Issue reference

Fix KAG-1148

bungle
bungle previously approved these changes Jun 5, 2023
@bungle bungle dismissed their stale review June 5, 2023 20:08

I'll check it later again, seems like tests are failing.

@StarlightIbuki
Copy link
Contributor Author

StarlightIbuki commented Jun 7, 2023

@bungle The test is failing because the migration tests are using an older version of helpers which do not include the http_mock. Trying to solve it in a separate PR.

hanshuebner
hanshuebner previously approved these changes Jun 8, 2023
@hanshuebner hanshuebner dismissed their stale review June 8, 2023 15:54

Still failing in upgrade test

@StarlightIbuki StarlightIbuki force-pushed the test/depre-http-server branch 2 times, most recently from 6a1b7a3 to d011180 Compare June 9, 2023 06:55
@StarlightIbuki StarlightIbuki force-pushed the test/depre-http-server branch from 3dfc240 to 1b22bed Compare June 29, 2023 08:30
@StarlightIbuki
Copy link
Contributor Author

I got confused. The http-log does not send any request, and I've tried to recover the test to use a copied http_server, it still fails.

@StarlightIbuki StarlightIbuki force-pushed the test/depre-http-server branch from bffad6d to c8bf339 Compare June 29, 2023 09:40
Fix KAG-1148

Co-authored-by: Hans Hübner <hans.huebner@gmail.com>
@StarlightIbuki StarlightIbuki force-pushed the test/depre-http-server branch from c8bf339 to b022475 Compare June 29, 2023 09:51
@hanshuebner hanshuebner merged commit 660cc98 into master Jun 30, 2023
@hanshuebner hanshuebner deleted the test/depre-http-server branch June 30, 2023 04:21
chobits added a commit that referenced this pull request Nov 29, 2023
This commit backports HTTP mocking to release/3.2.x, including the
following PRs from master branch: #10885, #11009, #11182 and #11331.
windmgc pushed a commit that referenced this pull request Nov 29, 2023
This commit backports HTTP mocking to release/3.2.x, including the
following PRs from master branch: #10885, #11009, #11182 and #11331.
chobits added a commit that referenced this pull request Nov 30, 2023
This commit backports HTTP mocking to release/3.2.x, including the
following PRs from master branch: #10885, #11009, #11182 and #11331.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants