Add an extra mocking test for what happens if running under windows w…

…here no signals are supported.

Revert this commit when we have actual CI tests running under windows.

Signed-off-by: Ying Li <[email protected]>

utils/configuration_test.go

@@ -536,7 +536,7 @@ func TestAdjustLogLevel(t *testing.T) {
 	}
 }
 
-func TestSetSignalTrap(t *testing.T) {
+func testSetSignalTrap(t *testing.T) {
 	var signalsPassedOn map[string]struct{}
 
 	signalHandler := func(s os.Signal) {
@@ -560,3 +560,16 @@ func TestSetSignalTrap(t *testing.T) {
 		require.NotNil(t, signalsPassedOn[s.String()])
 	}
 }
+
+// TODO: undo this extra indirection, needed for mocking notary.NotarySupportedSignals being empty, when we have
+// a windows CI system running
+func TestSetSignalTrap(t *testing.T) {
+	testSetSignalTrap(t)
+}
+
+func TestSetSignalTrapMockWindows(t *testing.T) {
+	old := notary.NotarySupportedSignals
+	notary.NotarySupportedSignals = nil
+	testSetSignalTrap(t)
+	notary.NotarySupportedSignals = old
+}