feature request: add a message to the options hash about ember-cli-acceptance-test-helpers HOT 7 CLOSED

@iezer Yeah, I think this would be great. Imo the most important thing is finding a good syntax for providing the message. Adding options: { message: '...' } seems ok, but do you have any ideas for something terser?

from ember-cli-acceptance-test-helpers.

@bantic We were thinking expectElement(findButton('Save Profile'), 'Save button displays correctly'); where the message is an optional parameter. CC @iezer

from ember-cli-acceptance-test-helpers.

@bantic @tundal45 @amyrlam

We can detect if the 3rd parameter is a string or an object. In the former case use that string as the message, and in the latter look for a msg or message property, so
expectElement(findButton('Save'), 1, 'Save Button');
and
expectElement(findButton('Save'), 1, { msg: 'Save Button' })
would both work.

In either case we should swap that message into the complete message from the helper so we would display Found 0 of 'Save Button' but expected 1

from ember-cli-acceptance-test-helpers.

something I have found useful in async tests, is the ability to pause and resume tests. At work we are using this, and its quite nice. This differs from a debugger, in that you can still use and interact with the paused application, and differs from the existing pauseTest helper, because it can be resumed.

example:

import pause from './helpers/pause';

test("fooo", async () => {

  await visit('/foo');

  await pause(); // will pause here, but can be resumed by typing `resume`
  await click('#bar');

  // some assertions
})

codez: https://gist.github.com/stefanpenner/0baadd08d0733f9e8ae0

from ember-cli-acceptance-test-helpers.

Submitted a PR. Please let me know if any feedback!

I think a good next step would be to change

expectElement(findButton('Save'), 1, { msg: 'Save Button' })

to just be

expectElement(findButton('Save'), 1, 'Save Button'); like assert.ok syntax

but not sure how to do that yet since options is { contains: 'foo', message: 'whatevs }

from ember-cli-acceptance-test-helpers.

@amyrlam Can I close this issue?

I've opened a separate issue #32 for @stefanpenner 's suggestion to add a pause helper.

from ember-cli-acceptance-test-helpers.

👍 and thanks for the help

from ember-cli-acceptance-test-helpers.