Skip to content

Commit

Permalink
comment why expect.assertions call
Browse files Browse the repository at this point in the history
  • Loading branch information
chandlerprall committed May 1, 2018
1 parent 2362caf commit d7bd0a9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/components/context_menu/context_menu_panel.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,7 @@ describe('EuiContextMenuPanel', () => {
describe('updating items and content', () => {
describe('updates to items', () => {
it(`should not re-render if any items's watchedItemProps did not change`, () => {
expect.assertions(2);
expect.assertions(2); // make sure the assertion in the `setProps` callback is executed

// by not passing `watchedItemProps` no changes to items should cause a re-render
const component = mount(
Expand Down Expand Up @@ -396,7 +396,7 @@ describe('EuiContextMenuPanel', () => {
});

it(`should re-render if any items's watchedItemProps did change`, () => {
expect.assertions(2);
expect.assertions(2); // make sure the assertion in the `setProps` callback is executed

// by referencing the `data-counter` property in `watchedItemProps`
// changes to the items should be picked up and re-rendered
Expand Down Expand Up @@ -426,7 +426,7 @@ describe('EuiContextMenuPanel', () => {
});

it(`should re-render at all times when children exists`, () => {
expect.assertions(2);
expect.assertions(2); // make sure the assertion in the `setProps` callback is executed

const component = mount(
<EuiContextMenuPanel>
Expand Down

0 comments on commit d7bd0a9

Please sign in to comment.