Skip to content

Latest commit

 

History

History
72 lines (56 loc) · 1.88 KB

valid-expect-in-promise.md

File metadata and controls

72 lines (56 loc) · 1.88 KB

Ensure promises that have expectations in their chain are valid (valid-expect-in-promise)

Ensure promises that include expectations are returned or awaited.

Rule details

This rule flags any promises within the body of a test that include expectations that have either not been returned or awaited.

The following patterns is considered warning:

it('promises a person', () => {
  api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });
});

it('promises a counted person', () => {
  const promise = api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });

  promise.then(() => {
    expect(analytics.gottenPeopleCount).toBe(1);
  });
});

it('promises multiple people', () => {
  const firstPromise = api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });
  const secondPromise = api.getPersonByName('alice').then(person => {
    expect(person).toHaveProperty('name', 'Alice');
  });

  return Promise.any([firstPromise, secondPromise]);
});

The following pattern is not warning:

it('promises a person', async () => {
  await api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });
});

it('promises a counted person', () => {
  let promise = api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });

  promise = promise.then(() => {
    expect(analytics.gottenPeopleCount).toBe(1);
  });

  return promise;
});

it('promises multiple people', () => {
  const firstPromise = api.getPersonByName('bob').then(person => {
    expect(person).toHaveProperty('name', 'Bob');
  });
  const secondPromise = api.getPersonByName('alice').then(person => {
    expect(person).toHaveProperty('name', 'Alice');
  });

  return Promise.allSettled([firstPromise, secondPromise]);
});