Add comment and clean up code (#4876)

Signed-off-by: Thomas Quandt <[email protected]>

Author: thquad

src/frontend/packages/core/src/shared/components/list/list-types/endpoint/endpoint-list.helpers.ts

@@ -172,6 +172,8 @@ export class EndpointListHelper {
         label: 'Unregister',
         description: 'Remove the endpoint',
         createVisible: (row$: Observable<EndpointModel>) => {
+          // I think if we end up using these often there should be specific create,
+          // edit, delete style permissions in the stratos permissions checker
           return combineLatest([
             this.sessionService.userEndpointsNotDisabled(),
             this.currentUserPermissionsService.can(StratosCurrentUserPermissions.EDIT_ADMIN_ENDPOINT),

src/jetstream/cnsi_test.go

@@ -570,12 +570,6 @@ func TestRegisterWithUserEndpointsEnabled(t *testing.T) {
 
 			})
 		})
-
-		Convey("with UserEndpointsEnabled=disabled", func() {
-			Convey("register existing user endpoint", func() {
-				// todo user endpoints should automatically overwritten?
-			})
-		})
 	})
 }