* Remove timestamp from non-watch test runs
* Fix mini reporter tests
* Pass options object to reporter constructor with watching boolean
* Refactor reporter options
* Move timestamp lines to watcher test
The current issue template uses HTML comment tags to remove template text from the rendered output. I have gone through the last dozen issues filed, and it appears everyone deletes that anyways. By removing the comment tags, people can click "Preview" and get a pretty print of the template markdown.
Reformat lists and links. Tries to get links as close to the left edge as possible, so they are easy to see, and copy-paste from the raw markdown editor. Puts multiple links on their own line for the same reason.
Adds a request that they explicitly tell us what their AVA and `npm` versions are.
Adds a section for their AVA config and CLI args.
Adds a section for linking to thier project if public.
Requests they create a minimal reproduction and post it to GitHub.
Closes#787
* refactor watcher tests
* move some methods from watcher to AvaFiles
* add some tests for ava-files
* add more test coverage
* fix linter error after merge
* PR Feedback
* use cross-platform split
* Basic macro support.
This adds basic macro support as discussed in #695.
It does not completely implement the spec outlined there, specifically:
- The macro can only specify the title using a function in `macroFn.title`. We discussed allowing `macroFn.title` to also be a string, and allowing some form of template language for extracting a title. However, using ES2015 string templates is already pretty easy, so we may just skip this.
- We discussed allowing groups of tests to be created using arrays.
Both the above proposals are found in [this comment](https://github.com/sindresorhus/ava/issues/695#issuecomment-205929738). They both enhance the implementation found in this commit, and would not break the contract. So I don't think there is anything preventing us from shipping this now.
* fix readme indentation
* spread arguments
* Allow arrays of macros
* pass providedTitle as first argument to title function.
* improve docs
If a non-function, non-promise, non-observable argument (i.e. a string) is passed to `t.throws`, the assertion passes. This obviously isn't what we want.
* change failing test hint
* mark .failing as code in readme
* group and reword failing tests
The tests are better when grouped together. I tried to give them more consistent titles too.
* remove spurious trailing space in t.end error message
Fixes#474
* add after.always, afterEach.always hook
* update readme about always modifier
* add test for verifying throw
* add tests for verifying always hook are added
* update readme for better "always" explaination
* fix typo in test/test-collection.js
* better always checking and implement
* modify error message for only
* fix build error
* add always to default in test
* fix profiler script
* add some profiling tests
* fix some lint errors
* disable eslint error for profile.js
* fix for Node.js < 4
eventEmitter.listenerCount was not a thing in days of yore.
* Exit process async for Node 0.10 which emits uncaught exceptions async.
* use process.execPath ... because windows
* fix error-code descrepencies on Windows