[PR #2884] [MERGED] Update route calculation tests #20780

Closed
opened 2026-08-05 05:07:41 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2884
Author: @mlsmaycon
Created: 11/13/2024
Status: Merged
Merged: 11/13/2024
Merged by: @mlsmaycon

Base: mainHead: update-route-calculation-tests


📝 Commits (1)

  • dc4e1ae Update route calculation tests

📊 Changes

1 file changed (+98 additions, -0 deletions)

View changed files

📝 client/internal/routemanager/client_test.go (+98 -0)

📄 Description

Describe your changes

  • Add two new test cases for p2p and relay routes with same latency
  • Add extra statuses generation

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 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/netbirdio/netbird/pull/2884 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 11/13/2024 **Status:** ✅ Merged **Merged:** 11/13/2024 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `update-route-calculation-tests` --- ### 📝 Commits (1) - [`dc4e1ae`](https://github.com/netbirdio/netbird/commit/dc4e1aec63bd3b35f7652dd7d21728b54355f1fc) Update route calculation tests ### 📊 Changes **1 file changed** (+98 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/routemanager/client_test.go` (+98 -0) </details> ### 📄 Description ## Describe your changes - Add two new test cases for p2p and relay routes with same latency - Add extra statuses generation ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 05:07:41 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#20780