[PR #28] [MERGED] test: add basic signal IT tests #2497

Closed
opened 2025-11-20 07:11:14 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/28
Author: @braginini
Created: 6/14/2021
Status: Merged
Merged: 6/19/2021
Merged by: @braginini

Base: mainHead: test-signal-grpc


📝 Commits (10+)

  • 83fe84d test: add basic signal IT tests
  • 7449848 Add build to the test
  • 7df6cde fix a typo and rename the embedded dll
  • d5c4f6c fix matrix var to use os
  • 714c4c3 use darwin
  • 6b5010f Fix Signal doc styling
  • 2784f6a Merge pull request #30 from andpar83/signal-doc
  • ca83e8c fix typo in directory name
  • 51337fb Merge pull request #31 from stv0g/fix-typo
  • 8b31088 Merge pull request #29 from wiretrustee/fix-mac-build

📊 Changes

14 files changed (+339 additions, -26 deletions)

View changed files

📝 .github/workflows/golang-test.yml (+30 -1)
📝 cmd/signal.go (+2 -2)
📝 go.mod (+3 -1)
📝 go.sum (+52 -3)
📝 iface/iface_darwin.go (+1 -1)
📝 infrastructure_files/docker-compose.yml (+0 -0)
📝 infrastructure_files/turnserver.conf (+0 -0)
📝 resources_windows_amd64.syso (+0 -0)
📝 signal/README.md (+12 -7)
📝 signal/client.go (+12 -3)
📝 signal/proto/constants.go (+1 -0)
📝 signal/signal.go (+14 -8)
signal/signal_suite_test.go (+13 -0)
signal/signal_test.go (+199 -0)

📄 Description

No description provided


🔄 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/28 **Author:** [@braginini](https://github.com/braginini) **Created:** 6/14/2021 **Status:** ✅ Merged **Merged:** 6/19/2021 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `test-signal-grpc` --- ### 📝 Commits (10+) - [`83fe84d`](https://github.com/netbirdio/netbird/commit/83fe84d11a90d91b03d63d40afef77c18ee91df4) test: add basic signal IT tests - [`7449848`](https://github.com/netbirdio/netbird/commit/744984861bc59bc2e60b8efc0041c94dc7b27cbd) Add build to the test - [`7df6cde`](https://github.com/netbirdio/netbird/commit/7df6cde968db67e31c573249282cc9a0ca59713c) fix a typo and rename the embedded dll - [`d5c4f6c`](https://github.com/netbirdio/netbird/commit/d5c4f6cb40016698f4e0af36604bfdd8b2819135) fix matrix var to use os - [`714c4c3`](https://github.com/netbirdio/netbird/commit/714c4c3c44957a5d4563ae7d7d1ebc0afeba969b) use darwin - [`6b5010f`](https://github.com/netbirdio/netbird/commit/6b5010f7d53ca6efc4232cb5196364a1163d9f96) Fix Signal doc styling - [`2784f6a`](https://github.com/netbirdio/netbird/commit/2784f6a09834e892f049ab37b782c0d8d6fac6a0) Merge pull request #30 from andpar83/signal-doc - [`ca83e8c`](https://github.com/netbirdio/netbird/commit/ca83e8c4a019ee82ed00052882f11d71f26048ac) fix typo in directory name - [`51337fb`](https://github.com/netbirdio/netbird/commit/51337fbf6593e668b64950c1b45d00063385e42f) Merge pull request #31 from stv0g/fix-typo - [`8b31088`](https://github.com/netbirdio/netbird/commit/8b3108896853335cc40150457302446ef5e59b0c) Merge pull request #29 from wiretrustee/fix-mac-build ### 📊 Changes **14 files changed** (+339 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golang-test.yml` (+30 -1) 📝 `cmd/signal.go` (+2 -2) 📝 `go.mod` (+3 -1) 📝 `go.sum` (+52 -3) 📝 `iface/iface_darwin.go` (+1 -1) 📝 `infrastructure_files/docker-compose.yml` (+0 -0) 📝 `infrastructure_files/turnserver.conf` (+0 -0) 📝 `resources_windows_amd64.syso` (+0 -0) 📝 `signal/README.md` (+12 -7) 📝 `signal/client.go` (+12 -3) 📝 `signal/proto/constants.go` (+1 -0) 📝 `signal/signal.go` (+14 -8) ➕ `signal/signal_suite_test.go` (+13 -0) ➕ `signal/signal_test.go` (+199 -0) </details> ### 📄 Description _No description provided_ --- <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 2025-11-20 07:11:14 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/netbird#2497