Remove core_security_mr_widget_downloads feature flag [RUN ALL RSPEC] [RUN AS-IF-FOSS]
What does this MR do?
Remove core_security_mr_widget_downloads flag
This removes all the relevant code from the component, and significantly rewrites the component specs, since they weren't written in a way that made it easy to simply remove contexts/tests.
The removed code was added in !48769 (merged), and the feature flag was enabled by default in !49572 (merged).
Addresses #292266 (closed).
Draft
Currently in draft, since this depends on !51926 (merged).
Screenshots (strongly suggested)
Before
This table represents the possible states the CE Security MR Widget could be in based on two feature flags, as is the case in master. Since this MR removes the core_security_mr_widget_downloads feature flag, the cases where this is disabled have been removed entirely.
| Feature flag statuses | coreSecurityMrWidgetCounts |
coreSecurityMrWidgetCounts |
|---|---|---|
coreSecurityMrWidgetDownloads |
i.e., no visual change from current |
i.e., no visual change from what was implemented in !47656 (merged). To recreate this screenshot, follow the testing instructions in that MR. |
coreSecurityMrWidgetDownloads |
 |
 |
After
coreSecurityMrWidgetCounts |
coreSecurityMrWidgetCounts |
|---|---|
| (i.e., identical to the bottom left image from the `Before` table above) |
(i.e., identical to the bottom right image from the `Before` table above) |
Does this MR meet the acceptance criteria?
Conformity
- [-] Changelog entry
- [-] Documentation (if required)
- [-] Code review guidelines
- [-] Merge request performance guidelines
-
Style guides - [-] Database guides
-
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. - [-] Tested in all supported browsers
- [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done