Update tests

modules/ansible/collector_test.go

@@ -2,13 +2,19 @@ package ansible
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find ansible in the test environment")
+		return
+	}
+
 	buf := &bytes.Buffer{}
 	c := collection.New(buf)
 

modules/corosync/collector_test.go

@@ -2,13 +2,19 @@ package corosync
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find corosync in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/elastic/collector_test.go

@@ -11,6 +11,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find elastic in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/foreman/collector.go

@@ -2,7 +2,6 @@ package foreman
 
 import (
 	"github.com/NETWAYS/support-collector/internal/util"
-	"os"
 	"path/filepath"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
@@ -27,11 +26,6 @@ var detailedFiles = []string{
 	"/var/log/foreman-proxy",
 }
 
-func detect() bool {
-	_, err := os.Stat("/etc/foreman")
-	return err == nil
-}
-
 var obfuscaters = []*obfuscate.Obfuscator{
 	obfuscate.NewFile(`(?i)(?:password)\s*:\s*(.*)`, "yml"),
 	obfuscate.NewFile(`(?i)(?:ENCRYPTION_KEY)\s*=\s*(.*)`, "rb"),

modules/foreman/collector_test.go

@@ -11,13 +11,13 @@ import (
 )
 
 func TestCollect(t *testing.T) {
-	c := collection.New(&bytes.Buffer{})
-
-	if !detect() {
+	if !util.ModuleExists(relevantPaths) {
 		t.Skip("could not find foreman in the test environment")
 		return
 	}
 
+	c := collection.New(&bytes.Buffer{})
+
 	Collect(c)
 
 	err := c.Close()

modules/grafana/collector_test.go

@@ -2,13 +2,19 @@ package grafana
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find grafana in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/graphite/collector_test.go

@@ -2,13 +2,19 @@ package graphite
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find graphite in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/graylog/collector_test.go

@@ -11,6 +11,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find graylog in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/icinga2/collector_test.go

@@ -12,6 +12,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find icinga2 in the test environment")
+		return
+	}
+
 	file, err := os.ReadFile("testdata/icinga-version.txt")
 	if err != nil {
 		t.Skip("cant read version file")
@@ -22,7 +27,7 @@ func TestCollect(t *testing.T) {
 		t.Skip("cant detect icinga2 version")
 	}
 
-	if !detectIcinga() {
+	if !util.ModuleExists(relevantPaths) {
 		t.Skip("could not find icinga2 in the test environment")
 		return
 	}

modules/icingadb/collector_test.go

@@ -2,13 +2,19 @@ package icingadb
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find icingadb in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/icingadirector/collector_test.go

@@ -2,13 +2,19 @@ package icingadirector
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists([]string{InstallationPath}) {
+		t.Skip("could not find icingadirector in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/icingaweb2/collector_test.go

@@ -11,7 +11,7 @@ import (
 )
 
 func TestCollect(t *testing.T) {
-	if !Detect() {
+	if !util.ModuleExists(relevantPaths) {
 		t.Skip("could not find icingaweb2 in the test environment")
 		return
 	}

modules/influxdb/collector_test.go

@@ -2,13 +2,19 @@ package influxdb
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find influxdb in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/keepalived/collector_test.go

@@ -11,6 +11,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find keepalived in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/mongodb/collector_test.go

@@ -11,6 +11,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find mongodb in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/mysql/collector_test.go

@@ -9,6 +9,7 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/postgresql/collector_test.go

@@ -2,13 +2,19 @@ package postgresql
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find postgresql in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/prometheus/collector_test.go

@@ -11,6 +11,11 @@ import (
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find prometheus in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/puppet/collector_test.go

@@ -2,13 +2,19 @@ package puppet
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find puppet in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)

modules/webservers/collector_test.go

@@ -2,13 +2,19 @@ package webservers
 
 import (
 	"bytes"
+	"github.com/NETWAYS/support-collector/internal/util"
 	"testing"
 
 	"github.com/NETWAYS/support-collector/internal/collection"
 	"github.com/stretchr/testify/assert"
 )
 
 func TestCollect(t *testing.T) {
+	if !util.ModuleExists(relevantPaths) {
+		t.Skip("could not find webservers in the test environment")
+		return
+	}
+
 	c := collection.New(&bytes.Buffer{})
 
 	Collect(c)