chore: Remove some todo items (#3772)

Author: just-boris

src/__a11y__/to-validate-a11y.ts

@@ -30,16 +30,6 @@ const htmlValidator = new HtmlValidate({
   },
 });
 
-// Polyfill for Array.prototype.flatMap
-function flatMap<Input, Output>(arr: ReadonlyArray<Input>, fn: (t: Input) => Output[]) {
-  return arr.reduce((acc: Output[], item: Input) => {
-    for (const flatItem of fn(item)) {
-      acc.push(flatItem);
-    }
-    return acc;
-  }, []);
-}
-
 function formatMessages(prefix: string, messages: Array<string>) {
   if (messages.length === 0) {
     return '';
@@ -73,8 +63,7 @@ async function toValidateA11y(this: jest.MatcherUtils, element: HTMLElement) {
     const htmlViolations = (htmlValidateResult.results[0]?.messages || []).map(
       message => `${message.message} [${message.ruleId}]`
     );
-    // TODO: remove polyfill with es2019 support
-    const flattenAxeViolations = flatMap(axeViolations, violation =>
+    const flattenAxeViolations = axeViolations.flatMap(violation =>
       violation.nodes.map(node => `[${violation.id}] ${node.failureSummary} Selector: ${node.target}`)
     );
 

src/app-layout/__tests__/main.test.tsx

@@ -49,8 +49,7 @@ describeEachAppLayout({ themes: ['classic', 'refresh-toolbar'], sizes: ['desktop
       },
     ],
   ] as const)('%s', (name, { propName, handlerName, findToggle, findOpenElement, findClose }) => {
-    // TODO: enable after fixing 'tools controlled property'
-    (theme === 'refresh-toolbar' && name === 'tools' ? test.skip : test)('property is controlled', () => {
+    test('property is controlled', () => {
       const onChange = jest.fn();
       const { wrapper, rerender } = renderComponent(<AppLayout {...{ [propName]: false, [handlerName]: onChange }} />);
       findToggle(wrapper).click();

src/app-layout/__tests__/state.test.tsx

@@ -28,8 +28,7 @@ describeEachAppLayout({ themes: ['refresh-toolbar'] }, () => {
     expect(refSpy).toHaveBeenCalledWith(null);
   });
 
-  // does not work without this fix: https://github.com/cloudscape-design/component-toolkit/pull/151
-  test.skip('content dom reference is preserved when changing switching between desktop and mobile', () => {
+  test('content dom reference is preserved when changing switching between desktop and mobile', () => {
     const refSpy = jest.fn();
     renderComponent(<AppLayout content={<div ref={refSpy}>content</div>} />);