angular.js/test, branch v1.2.12

fix(mocks): rename mock.animate to ngAnimateMock and ensure it contains all test helper code for ngAnimate

2014-02-06T06:22:14+00:00

Closes #5822
Closes #5917

fix(mocks): remove usage of $animate.flushNext in favour of queing

2014-02-06T06:21:41+00:00

The flushNext method of testing is difficult and highly coupled with the behavior
of ngAnimate's $animate workflow. It is much better instead to just queue all
$animate animation calls into a queue collection which is available on the $animate
service when mock.animate is included as a module within test code.

chore(testing): run end to end tests on firefox and safari as well as chrome

2014-02-05T23:39:59+00:00

Update the Travis and Jenkins configs to run protractor tests on Safari and Firefox as well,
and make the Travis tests run output XML and turn off color.

Fix tests which were failing in Firefox due to clear() not working as expected.

Fix tests which were failing in Safari due to SafariDriver not understanding the minus key,
and disable tests which SafariDriver has no support for.

fix(ngMock): return false from mock $interval.cancel() when no argument is supplied

2014-02-05T00:41:25+00:00

Closes #6103.
Closed #6099.

fix(jqLite): trim HTML string in jqLite constructor

2014-02-05T00:39:52+00:00

jQuery will construct DOM nodes containing leading whitespace. Prior to this change, jqLite would
throw a nosel minErr due to the first character of the string not being '<'. This change corrects
this behaviour by trimming the element string in jqLite constructor before testing for '<'.

Closes #6053

fix(ngResource): don't filter "$"-prefixed properties from ngResource requests/responses

2014-02-04T15:51:24+00:00

ngResource no longer filters properties prefixed with a single "$" character from requests or
responses, correcting a regression introduced in 1.2.6 (cb29632a) which caused shallowCopy and
shallowClearAndCopy to ignore properties prefixed with a single "$".

Closes #5666
Closes #6080
Closes #6033

fix(mocks): always call functions injected with `inject` with `this` set to the current spec

2014-02-04T06:25:30+00:00

Currently when a function is injected inside of a test we set the context to undefined which
is a bug.

Closes #6102

fix(ngResource): don't append number to '$' in url param value when encoding URI

2014-02-03T16:12:07+00:00

Previously, if a URL parameter value included a $, it would replace the dollar sign with a literal
'$1' for mysterious reasons. Using a function rather than a replacement string circumvents this
behaviour and produces a more expected result.

Closes #6003
Closes #6004

fix($q): make $q.reject support `finally` and `catch`

2014-01-31T19:38:06+00:00

Add support for the functions `finally` and `catch` to the
promise returned by `$q.reject`

Closes #6048
Closes #6076

fix($compile): retain CSS classes added in cloneAttachFn on asynchronous directives

2014-01-31T17:45:35+00:00

Previously, classes added to asynchronous directive elements during the clone
attach function would not persist after the node is merged with the template, prior
to linking. This change corrects this behaviour and brings it in line with synchronous
directives.

Closes #5439
Closes #5617