[EuiInMemoryTable] Update state.prevProps.sortName & sortDirection

[chandlerprall]

Summary

Fixes #6032 by updating EuiInMemoryTable's getDerivedStateFromProps to update the sort details in prevProps as necessary.

To be honest, I'm not sure how to write a meaningful test for this. @sebelga can you extract your use case to a simple test scenario? (just describing it should be fine, I can put the test together)

Checklist

- [ ] Checked in both light and dark modes
- [ ] Checked in mobile
- [ ] Checked in Chrome, Safari, Edge, and Firefox
- [ ] Props have proper autodocs and playground toggles
- [ ] Added documentation
- [ ] Checked Code Sandbox works for any docs examples

• Added or updated jest and cypress tests
• Checked for breaking changes and labeled appropriately
  - [ ] Checked for accessibility including keyboard-only and screenreader modes
  - [ ] Updated the Figma library counterpart
• A changelog entry exists and is marked appropriately

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_6228/

[sebelga]

if (
      sortName !== prevState.prevProps.sortName ||
      sortDirection !== prevState.prevProps.sortDirection
    ) {
      updatedPrevState = {
        ...updatedPrevState,
        sortName,
        sortDirection,
      };
    }

In the above code prevState.prevProps never gets updated, meaning that sortName !== prevState.prevProps.sortName is never true for the initial column we sort on.

The issue can be reproduced by control the sort

• Control the sort state in the consumer component ( passing tableSort={tableSort} to the table)

// Consumer state
const [tableSort] = useState({
    field: 'updatedAt'; // initial column we sort on
    direction: 'desc';
});

• Change the sort order by clicking the column header. It won't let for the udpatedAt field
• Change the sort order for any other column (e.g. title): no problem, the sortName is different

[chandlerprall]

Perfect, thanks @sebelga! I've pushed up a test for this too

[cee-chen]

LGTM! one very minor nit

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_6228/

[sebelga]

@chandlerprall Will we have this fix for 8.5 stack release?

[chandlerprall]

@chandlerprall Will we have this fix for 8.5 stack release?

Yes, we're including this in a backport release for 8.5