fix: handle case when using configure only with defaultHidden option

Author: EBAM006

src/config.ts

@@ -26,6 +26,20 @@ const defaultConfig: Config = {
 let config = { ...defaultConfig };
 
 export function configure(options: Partial<Config>) {
+  if (
+    options.defaultHidden !== undefined &&
+    options.defaultIncludeHiddenElements === undefined
+  ) {
+    options.defaultIncludeHiddenElements = options.defaultHidden;
+  }
+
+  if (
+    options.defaultIncludeHiddenElements !== undefined &&
+    options.defaultHidden === undefined
+  ) {
+    options.defaultHidden = options.defaultIncludeHiddenElements;
+  }
+
   config = {
     ...config,
     ...options,

src/helpers/__tests__/includeHiddenElements.test.tsx

@@ -3,7 +3,7 @@ import React from 'react';
 
 import render from '../../render';
 import { screen } from '../../screen';
-import { configure } from '../../config';
+import { configure, getConfig } from '../../config';
 
 test('includeHiddenElements query option takes priority over hidden option and global config', () => {
   configure({ defaultHidden: true, defaultIncludeHiddenElements: true });
@@ -24,3 +24,19 @@ test('global config defaultIncludeElements option takes priority over defaultHid
   render(<View testID="view" style={{ display: 'none' }} />);
   expect(screen.getByTestId('view')).toBeTruthy();
 });
+
+test('defaultHidden takes priority when it was set last', () => {
+  // also simulates the case when defaultIncludeHiddenElements is true by default in the config
+  configure({ defaultIncludeHiddenElements: true });
+  configure({ defaultHidden: false });
+  render(<View testID="view" style={{ display: 'none' }} />);
+  expect(screen.queryByTestId('view')).toBeFalsy();
+});
+
+test('defaultIncludeHiddenElements takes priority when it was set last', () => {
+  // also simulates the case when defaultHidden is true by default in the config
+  configure({ defaultHidden: true });
+  configure({ defaultIncludeHiddenElements: false });
+  render(<View testID="view" style={{ display: 'none' }} />);
+  expect(screen.queryByTestId('view')).toBeFalsy();
+});

src/helpers/findAll.ts

@@ -17,8 +17,7 @@ export function findAll(
   const hidden =
     options?.includeHiddenElements ??
     options?.hidden ??
-    getConfig()?.defaultIncludeHiddenElements ??
-    getConfig()?.defaultHidden;
+    getConfig()?.defaultIncludeHiddenElements;
 
   if (hidden) {
     return results;