diff --git a/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.js b/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.js
index f0d5b343c..b8c8285e1 100644
--- a/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.js
+++ b/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.js
@@ -51,7 +51,7 @@ const ReadingRoomAccess = (props) => {
if (!isPending) {
setSortedRecordsDetails(prev => ({
...prev.sortedRecordsDetails,
- data: orderBy(userRRAPermissions, ['access'])
+ data: orderBy(userRRAPermissions, [rraColumns.ACCESS])
}));
}
}, [userRRAPermissions, isPending]);
@@ -61,7 +61,7 @@ const ReadingRoomAccess = (props) => {
const filteredRRs = userRRAPermissions.filter(r => r.readingRoomName.includes(name));
setSortedRecordsDetails(prev => ({
...prev.sortedRecordsDetails,
- data: orderBy(filteredRRs, ['access'])
+ data: orderBy(filteredRRs, [rraColumns.ACCESS])
}));
};
@@ -125,7 +125,7 @@ const ReadingRoomAccess = (props) => {
onChange={filterReadingRoomsByName}
onClear={() => setSortedRecordsDetails(prev => ({
...prev.sortedRecordsDetails,
- data: orderBy(userRRAPermissions, ['access'])
+ data: orderBy(userRRAPermissions, [rraColumns.ACCESS])
}))}
placeholder={intl.formatMessage({ id:'ui-users.readingRoom.filter' })}
/>
diff --git a/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.test.js b/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.test.js
index f089d0f28..97d403b0d 100644
--- a/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.test.js
+++ b/src/components/UserDetailSections/ReadingRoomAccess/ReadingRoomAccess.test.js
@@ -92,6 +92,16 @@ describe('ReadingRoomAccess', () => {
await waitFor(() => expect(numOfRows).toBe(1));
});
+ it('should render all records when search string is cleared', async () => {
+ render();
+ const inputEl = screen.getByPlaceholderText('ui-users.readingRoom.filter');
+ fireEvent.change(inputEl, { target: { value: 'room 1' } });
+ const numOfRows = document.querySelectorAll('[class^="mclRowFormatterContainer"]').length;
+ await waitFor(() => expect(numOfRows).toBe(1));
+ fireEvent.change(inputEl, { target: { value: '' } });
+ expect(document.querySelectorAll('[class^="mclRowFormatterContainer"]').length).toBe(2);
+ });
+
it('should render updated date', () => {
render();
expect(screen.getByText('ui-users.readingRoom.metaSection.lastUpdatedDetails')).toBeDefined();