[PR #1872] [MERGED] move "embedded derp" settings into With options for integration tests #2361

Closed
opened 2025-12-29 03:20:55 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/1872
Author: @kradalby
Created: 4/16/2024
Status: Merged
Merged: 4/16/2024
Merged by: @kradalby

Base: mainHead: integration-with-derp


📝 Commits (1)

  • f209bbd move "embedded derp" settings into With options for integration tests

📊 Changes

3 files changed (+22 additions, -30 deletions)

View changed files

📝 integration/embedded_derp_test.go (+1 -15)
📝 integration/general_test.go (+1 -15)
📝 integration/hsic/hsic.go (+20 -0)

📄 Description

QoL and Dry improvement.

Over time I want to try to use this option more to see if I can make the integration tests able to run offline, would be nice when working on flight.

But I'm sure docker and other things that always wants to fetch things will put a stopper to that anyways...


🔄 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/juanfont/headscale/pull/1872 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 4/16/2024 **Status:** ✅ Merged **Merged:** 4/16/2024 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `integration-with-derp` --- ### 📝 Commits (1) - [`f209bbd`](https://github.com/juanfont/headscale/commit/f209bbdbf5e65ae6c98e573b1345eaccf323b2c6) move "embedded derp" settings into With options for integration tests ### 📊 Changes **3 files changed** (+22 additions, -30 deletions) <details> <summary>View changed files</summary> 📝 `integration/embedded_derp_test.go` (+1 -15) 📝 `integration/general_test.go` (+1 -15) 📝 `integration/hsic/hsic.go` (+20 -0) </details> ### 📄 Description QoL and Dry improvement. Over time I want to try to use this option more to see if I can make the integration tests able to run offline, would be nice when working on flight. But I'm sure docker and other things that always wants to fetch things will put a stopper to that anyways... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 03:20:55 +01:00
adam closed this issue 2025-12-29 03:20:55 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2361