Commitf51fae7

committed

linting

1 parent8967a42 commitf51fae7Copy full SHA for f51fae7

File tree

-2

lines changed

-2

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -906,6 +906,7 @@ func (o orgGroupAssert) Assert(t *testing.T, orgID uuid.UUID, db database.Store,`
`906`	`906`	`}`
`907`	`907`	`}`
`908`	`908`
	`909`	`+//nolint:revive`
`909`	`910`	`func (oorgRoleAssert)Assert(t*testing.T,orgID uuid.UUID,db database.Store,notMemberbool,user database.User) {`
`910`	`911`	`t.Helper()`
`911`	`912`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,8 @@ func (s AGPLIDPSync) SyncRoles(ctx context.Context, db database.Store, user data`
`70`	`70`
`71`	`71`	`// sync roles per organization`
`72`	`72`	`orgMemberships,err:=tx.OrganizationMembers(ctx, database.OrganizationMembersParams{`
`73`		`-UserID:user.ID,`
	`73`	`+OrganizationID:uuid.Nil,`
	`74`	`+UserID:user.ID,`
`74`	`75`	`})`
`75`	`76`	`iferr!=nil {`
`76`	`77`	`returnxerrors.Errorf("get organizations by user id: %w",err)`
`@@ -241,7 +242,7 @@ func (s AGPLIDPSync) syncSiteWideRoles(ctx context.Context, tx database.Store, u`
`241`	`242`	`returnnil`
`242`	`243`	`}`
`243`	`244`
`244`		`-func (sAGPLIDPSync)RolesFromClaim(fieldstring,claims jwt.MapClaims) ([]string,error) {`
	`245`	`+func (AGPLIDPSync)RolesFromClaim(fieldstring,claims jwt.MapClaims) ([]string,error) {`
`245`	`246`	`rolesRow,ok:=claims[field]`
`246`	`247`	`if!ok {`
`247`	`248`	`// If no claim is provided than we can assume the user is just`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -301,6 +301,8 @@ func TestRoleSyncTable(t *testing.T) {`
`301`	`301`	`// per organization. This limits the number of db calls to O(1) if there`
`302`	`302`	`// are no changes. Which is the usual case, as user's roles do not change often.`
`303`	`303`	`funcTestNoopNoDiff(t*testing.T) {`
	`304`	`+t.Parallel()`
	`305`	`+`
`304`	`306`	`ctx:=context.Background()`
`305`	`307`	`ctrl:=gomock.NewController(t)`
`306`	`308`	`mDB:=dbmock.NewMockStore(ctrl)`

Comments

(0)