respect custom ignores

Author: benmonro

src/__tests__/suggestions.js

@@ -10,14 +10,21 @@ afterAll(() => {
   configure({throwSuggestions: false})
 })
 
-test('dost not suggest for inline script, style', () => {
+test('does not suggest for inline script, style', () => {
   renderIntoDocument(
     `<script data-testid="script">alert('hello')</script><style data-testid="style">.hsuHs{margin:auto}.wFncld{margin-top:3px;color:#9AA0A6;height:20px;width:20px}</style>`,
   )
 
   expect(() => screen.getByTestId('script')).not.toThrow()
   expect(() => screen.getByTestId('style')).not.toThrow()
 })
+it('respects ignores', () => {
+  renderIntoDocument(`<my-thing>foo</my-thing>`)
+
+  expect(() =>
+    screen.queryByText('foo', {ignore: 'my-thing'}),
+  ).not.toThrowError()
+})
 
 test('does not suggest when using getByRole', () => {
   renderIntoDocument(`<button data-testid="foo">submit</button>`)

src/query-helpers.js

@@ -94,7 +94,7 @@ const wrapSingleQueryWithSuggestion = (query, queryAllByName, variant) => (
 ) => {
   const element = query(container, ...args)
   const [{suggest = getConfig().throwSuggestions} = {}] = args.slice(-1)
-  if (suggest) {
+  if (element && suggest) {
     const suggestion = getSuggestedQuery(element, variant)
     if (suggestion && !queryAllByName.endsWith(suggestion.queryName)) {
       throw getSuggestionError(suggestion.toString(), container)
@@ -111,7 +111,7 @@ const wrapAllByQueryWithSuggestion = (query, queryAllByName, variant) => (
   const els = query(container, ...args)
 
   const [{suggest = getConfig().throwSuggestions} = {}] = args.slice(-1)
-  if (suggest) {
+  if (els.length && suggest) {
     // get a unique list of all suggestion messages.  We are only going to make a suggestion if
     // all the suggestions are the same
     const uniqueSuggestionMessages = [