https://codereview.appspot.com/7005044/diff/1/app/modules-debug.js#newcode29
app/modules-debug.js:29: fullpath:
'juju-ui/assets/javascripts/gallery-timer-debug.js'
On 2012/12/21 10:57:51, frankban wrote:
> These three paths need to be absolute (add a slash at the beginning of
each
> one). Otherwise, in test-debug runs, they are searched inside the
/test/ path,
> and a bunch of tests fail.
Thank you, Francesco and Nicola!
https:/ /codereview. appspot. com/7005044/ diff/1/ app/modules- debug.js debug.js (right):
File app/modules-
https:/ /codereview. appspot. com/7005044/ diff/1/ app/modules- debug.js# newcode29 debug.js: 29: fullpath: assets/ javascripts/ gallery- timer-debug. js'
app/modules-
'juju-ui/
On 2012/12/21 10:57:51, frankban wrote:
> These three paths need to be absolute (add a slash at the beginning of
each
> one). Otherwise, in test-debug runs, they are searched inside the
/test/ path,
> and a bunch of tests fail.
Thank you! Fixed.
https:/ /codereview. appspot. com/7005044/ diff/1/ app/views/ utils.js
File app/views/utils.js (right):
https:/ /codereview. appspot. com/7005044/ diff/1/ app/views/ utils.js# newcode121 utils.js: 121: debug: noop
app/views/
On 2012/12/21 10:57:51, frankban wrote:
> Yay! No more errors running test-prod!
:-)
https:/ /codereview. appspot. com/7005044/