Additional tidy up of unused

public/app/features/alerting/unified/RuleEditor.tsx

@@ -65,7 +65,7 @@ const RuleEditor = () => {
     }
 
     if (identifier) {
-      return <ExistingRuleEditor key={id} identifier={identifier} id={id} />;
+      return <ExistingRuleEditor key={id} identifier={identifier} />;
     }
 
     if (copyFromIdentifier) {

public/app/features/alerting/unified/Settings.test.tsx

@@ -7,7 +7,7 @@ import SettingsPage from './Settings';
 import DataSourcesResponse from './components/settings/__mocks__/api/datasources.json';
 import { setupGrafanaManagedServer, withExternalOnlySetting } from './components/settings/__mocks__/server';
 import { setupMswServer } from './mockApi';
-import { grantUserRole } from './mocks';
+import { grantAllUserRoles } from './mocks';
 
 jest.mock('@grafana/runtime', () => ({
   ...jest.requireActual('@grafana/runtime'),
@@ -39,7 +39,7 @@ const ui = {
 
 describe('Alerting settings', () => {
   beforeEach(() => {
-    grantUserRole('ServerAdmin');
+    grantAllUserRoles();
     setupGrafanaManagedServer(server);
   });
 

public/app/features/alerting/unified/components/rules/Filter/RulesFilter.v2.tsx

@@ -28,6 +28,7 @@ type RulesFilterProps = {
 
 type ActiveTab = 'custom' | 'saved';
 
+// eslint-disable-next-line no-unused-vars
 export default function RulesFilter({ onClear = () => {} }: RulesFilterProps) {
   const styles = useStyles2(getStyles);
   const [activeTab, setActiveTab] = useState<ActiveTab>('custom');
@@ -155,6 +156,7 @@ type TableColumns = {
 };
 
 const SavedSearches = () => {
+  // eslint-disable-next-line no-unused-vars
   const applySearch = useCallback((name: string) => {}, []);
 
   return (

public/app/features/alerting/unified/components/settings/ConfigurationDrawer.tsx

@@ -68,9 +68,7 @@ export function useEditConfigurationDrawer() {
             onReset={resetAlertmanagerSettings}
           />
         )}
-        {activeTab === 'versions' && dataSourceName && (
-          <AlertmanagerConfigurationVersionManager alertmanagerName={dataSourceName} />
-        )}
+        {activeTab === 'versions' && dataSourceName && <AlertmanagerConfigurationVersionManager />}
       </Drawer>
     );
   }, [open, dataSourceName, handleDismiss, activeTab, updateAlertmanagerSettings, resetAlertmanagerSettings]);

public/app/features/alerting/unified/mocks.ts

@@ -709,7 +709,7 @@ export const grantUserPermissions = (permissions: AccessControlAction[]) => {
     .mockImplementation((action) => permissions.includes(action as AccessControlAction));
 };
 
-export const grantUserRole = () => {
+export const grantAllUserRoles = () => {
   jest.spyOn(contextSrv, 'hasRole').mockReturnValue(true);
 };