fixing some tests

gravitational · Dec 5, 2024 · 1b1feae · 1b1feae
1 parent 0d932f2
commit 1b1feae
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 15 deletions.
diff --git a/lib/services/role_test.go b/lib/services/role_test.go
@@ -267,7 +267,6 @@ func TestRoleParse(t *testing.T) {
 					Options: types.RoleOptions{
 						CertificateFormat: constants.CertificateFormatStandard,
 						MaxSessionTTL:     types.NewDuration(apidefaults.MaxCertDuration),
-						PortForwarding:    types.NewBoolOption(true),
 						RecordSession: &types.RecordSession{
 							Default: constants.SessionRecordingModeBestEffort,
 							Desktop: types.NewBoolOption(true),
@@ -321,7 +320,6 @@ func TestRoleParse(t *testing.T) {
 					Options: types.RoleOptions{
 						CertificateFormat: constants.CertificateFormatStandard,
 						MaxSessionTTL:     types.NewDuration(apidefaults.MaxCertDuration),
-						PortForwarding:    types.NewBoolOption(true),
 						RecordSession: &types.RecordSession{
 							Default: constants.SessionRecordingModeBestEffort,
 							Desktop: types.NewBoolOption(true),

diff --git a/lib/srv/forward/sshserver_test.go b/lib/srv/forward/sshserver_test.go
@@ -213,19 +213,6 @@ func TestDirectTCPIP(t *testing.T) {
 			// which return errors on accept.
 			expectRejected: true,
 		},
-		{
-			name: "port forwarding denied",
-			login: func() string {
-				u, err := user.Current()
-				require.NoError(t, err)
-				return u.Username
-			}(),
-			accessChecker:  &fakePortForwardChecker{mode: services.SSHPortForwardModeOff},
-			expectAccepted: false,
-			// expectRejected is set to true because we are using mock channel
-			// which return errors on accept.
-			expectRejected: true,
-		},
 	}
 
 	for _, tt := range cases {