Skip to content

Commit 23edf06

Browse files
tyhicksmheon
authored andcommitted
golang: Add filterAttrLog getter/setters test
Update TestFilterAttributeGettersAndSetters() to set the log filter flag and then verify its value. The log filter flag is not tested when libseccomp is not new enough to support API level operations. Signed-off-by: Tyler Hicks <[email protected]> Signed-off-by: Matthew Heon <[email protected]>
1 parent 4b17538 commit 23edf06

File tree

1 file changed

+32
-0
lines changed

1 file changed

+32
-0
lines changed

seccomp_test.go

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -432,6 +432,38 @@ func TestFilterAttributeGettersAndSetters(t *testing.T) {
432432
t.Errorf("No new privileges bit was not set correctly")
433433
}
434434

435+
if ApiLevelIsSupported() {
436+
api, err := GetApi()
437+
if err != nil {
438+
t.Errorf("Error getting API level: %s", err)
439+
} else if api < 3 {
440+
err = SetApi(3)
441+
if err != nil {
442+
t.Errorf("Error setting API level: %s", err)
443+
}
444+
}
445+
}
446+
447+
err = filter.SetLogBit(true)
448+
if err != nil {
449+
if !ApiLevelIsSupported() {
450+
t.Logf("Ignoring failure: %s\n", err)
451+
} else {
452+
t.Errorf("Error setting log bit")
453+
}
454+
}
455+
456+
log, err := filter.GetLogBit()
457+
if err != nil {
458+
if !ApiLevelIsSupported() {
459+
t.Logf("Ignoring failure: %s\n", err)
460+
} else {
461+
t.Errorf("Error getting log bit")
462+
}
463+
} else if log != true {
464+
t.Errorf("Log bit was not set correctly")
465+
}
466+
435467
err = filter.SetBadArchAction(ActInvalid)
436468
if err == nil {
437469
t.Errorf("Setting bad arch action to an invalid action should error")

0 commit comments

Comments
 (0)