test(internal/procfs): print error instead of warning

Author: huhong-web

internal/procfs/proc_test.go

@@ -106,9 +106,6 @@ func TestNewProc_Integration(t *testing.T) {
 	}
 
 	proc, err := NewProc(1)
-	if err != nil {
-		t.Skipf("PID 1 not available: %v", err)
-	}
 	rightProcCheck(t, proc, err)
 
 	proc, err = NewProc(999999999)
@@ -121,8 +118,5 @@ func TestSelf_Integration(t *testing.T) {
 	}
 
 	proc, err := Self()
-	if err != nil {
-		t.Skipf("self process not available: %v", err)
-	}
 	rightProcCheck(t, proc, err)
 }

internal/procfs/sysfs/fs_test.go

@@ -133,8 +133,6 @@ func TestNewDefaultFS_Integration(t *testing.T) {
 	}
 
 	fs, err := NewDefaultFS()
-	if err != nil {
-		t.Skipf("/sys not available: %v", err)
-	}
+	assert.NoError(t, err)
 	assert.NotNil(t, fs)
 }

internal/procfs/sysfs/net_class_test.go

@@ -204,9 +204,7 @@ func TestDefaultNetClassDevices_Integration(t *testing.T) {
 	}
 
 	devices, err := DefaultNetClassDevices()
-	if err != nil {
-		t.Skipf("/sys/class/net not available: %v", err)
-	}
+	assert.NoError(t, err)
 
 	assert.NotEmpty(t, devices, "should have at least one network device")
 	t.Logf("Found %d devices: %v", len(devices), devices)
@@ -218,9 +216,7 @@ func TestDefaultNetClass_Integration(t *testing.T) {
 	}
 
 	netClass, err := DefaultNetClass()
-	if err != nil {
-		t.Skipf("/sys/class/net not available: %v", err)
-	}
+	assert.NoError(t, err)
 
 	assert.NotNil(t, netClass, "should have a valid NetClass object")
 	t.Logf("Got NetClass: %+v", netClass)

internal/procfs/xfs/fs_test.go

@@ -112,8 +112,5 @@ func TestNewDefaultFS_Integration(t *testing.T) {
 		t.Skip("Set TEST_INTEGRATION=true to run")
 	}
 	fs, err := NewDefaultFS()
-	if err != nil {
-		t.Skipf("/proc or /sys unavailable: %v", err)
-	}
 	rightFSCheck(t, fs, err)
 }