Add test coverage to ensure navigation isn't rendered for users that don't have access

Created on 29 April 2024, 7 months ago
Updated 7 May 2024, 7 months ago

Problem/Motivation

Follow up from πŸ› Navigation render array is completely generated, even if the navigation bar is not rendered Fixed from @catch

It would be possible to write performance tests - i.e. that any extra queries/cache gets don't happen when the module is enabled for anonymous users, but I think that can be moved to a stable-blocking follow-up, rather than this which ought to block beta and is a simple change in itself. There's not really a way to test it other than performance testing because no user-facing bug.

Steps to reproduce

N/A

Proposed resolution

Add test coverage for this feature

Remaining tasks

Do it

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component
NavigationΒ  β†’

Last updated about 5 hours ago

No maintainer
Created by

πŸ‡ΊπŸ‡ΈUnited States smustgrave

Live updates comments and jobs are added and updated live.
  • Performance

    It affects performance. It is often combined with the Needs profiling tag.

Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024