TestFilterAttributeGettersAndSetters: fixup

Commit 23edf06 moved the SetBadArchAction(ActInvalid) check
to the end of TestFilterAttributeGettersAndSetters. A few subsequent
commits added more code before it, until finally commit 541420d
added a conditional t.Skipf before it.

As a result, this check is not performed with latest libseccomp.

Move it back to where it belongs (before the API/version checks).

Signed-off-by: Kir Kolyshkin <[email protected]>
Acked-by: Tom Hromatka <[email protected]>
Signed-off-by: Paul Moore <[email protected]>

Author: kolyshkin

seccomp_test.go

@@ -427,6 +427,11 @@ func TestFilterAttributeGettersAndSetters(t *testing.T) {
 		t.Errorf("Bad arch action was not set correctly!")
 	}
 
+	err = filter.SetBadArchAction(ActInvalid)
+	if err == nil {
+		t.Errorf("Setting bad arch action to an invalid action should error")
+	}
+
 	err = filter.SetNoNewPrivsBit(false)
 	if err != nil {
 		t.Errorf("Error setting no new privileges bit")
@@ -502,11 +507,6 @@ func TestFilterAttributeGettersAndSetters(t *testing.T) {
 	} else if ssb != true {
 		t.Errorf("SSB bit was not set correctly")
 	}
-
-	err = filter.SetBadArchAction(ActInvalid)
-	if err == nil {
-		t.Errorf("Setting bad arch action to an invalid action should error")
-	}
 }
 
 func TestMergeFilters(t *testing.T) {