[PR #3451] [MERGED] add e2e case for xtcp #4748

Closed
opened 2026-05-05 14:47:58 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fatedier/frp/pull/3451
Author: @fatedier
Created: 5/28/2023
Status: Merged
Merged: 5/28/2023
Merged by: @fatedier

Base: devHead: pwd


📝 Commits (1)

📊 Changes

2 files changed (+10 additions, -3 deletions)

View changed files

📝 hack/run-e2e.sh (+1 -1)
📝 test/e2e/basic/basic.go (+9 -2)

📄 Description

Summary

🤖 Generated by Copilot at f152b97

Added xtcp tests to the test/e2e/basic package and increased the test progress report interval in hack/run-e2e.sh. These changes improve the test coverage and readability of the end-to-end tests.

WHY

Walkthrough

🤖 Generated by Copilot at f152b97

  • Increase the test progress report interval to 30s to reduce log noise and size (link)
  • Import the time package to use timeouts and delays in some test cases (link)
    • Add "xtcp" to the proxy types slice in the STCP && SUDP describe block (link)
    • Add a case block for the xtcp proxy type in the BeforeEach block, setting the local port name and protocol to TCP (link)
    • Add a RequestModify function to the ExpectConnectServer function call in the It block, setting a timeout of 5 seconds for the request (link)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/fatedier/frp/pull/3451 **Author:** [@fatedier](https://github.com/fatedier) **Created:** 5/28/2023 **Status:** ✅ Merged **Merged:** 5/28/2023 **Merged by:** [@fatedier](https://github.com/fatedier) **Base:** `dev` ← **Head:** `pwd` --- ### 📝 Commits (1) - [`f152b97`](https://github.com/fatedier/frp/commit/f152b97f2e85735a26e1d0ebdf70799d8720af7e) add e2e case for xtcp ### 📊 Changes **2 files changed** (+10 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `hack/run-e2e.sh` (+1 -1) 📝 `test/e2e/basic/basic.go` (+9 -2) </details> ### 📄 Description ### Summary <!-- copilot:summary --> ### <samp>🤖 Generated by Copilot at f152b97</samp> Added `xtcp` tests to the `test/e2e/basic` package and increased the test progress report interval in `hack/run-e2e.sh`. These changes improve the test coverage and readability of the end-to-end tests. ### WHY <!-- author to complete --> ### Walkthrough <!-- copilot:walkthrough --> ### <samp>🤖 Generated by Copilot at f152b97</samp> * Increase the test progress report interval to 30s to reduce log noise and size ([link](https://github.com/fatedier/frp/pull/3451/files?diff=unified&w=0#diff-0913e7b61c925dbdbf31c36ad24131da81c64f8ec9c3477662eec4cca73dba07L20-R20)) * Import the `time` package to use timeouts and delays in some test cases ([link](https://github.com/fatedier/frp/pull/3451/files?diff=unified&w=0#diff-198c426913c63e51585ac1793ca15e8e195cf8a0a72ba93e9b12d2f6acd9fa44R7)) - Add `"xtcp"` to the proxy types slice in the `STCP && SUDP` describe block ([link](https://github.com/fatedier/frp/pull/3451/files?diff=unified&w=0#diff-198c426913c63e51585ac1793ca15e8e195cf8a0a72ba93e9b12d2f6acd9fa44L278-R280)) - Add a case block for the `xtcp` proxy type in the `BeforeEach` block, setting the local port name and protocol to TCP ([link](https://github.com/fatedier/frp/pull/3451/files?diff=unified&w=0#diff-198c426913c63e51585ac1793ca15e8e195cf8a0a72ba93e9b12d2f6acd9fa44R297-R299)) - Add a `RequestModify` function to the `ExpectConnectServer` function call in the `It` block, setting a timeout of 5 seconds for the request ([link](https://github.com/fatedier/frp/pull/3451/files?diff=unified&w=0#diff-198c426913c63e51585ac1793ca15e8e195cf8a0a72ba93e9b12d2f6acd9fa44R378-R380)) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 14:47:58 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/frp#4748
No description provided.