Skip to content
This repository has been archived by the owner on Aug 31, 2022. It is now read-only.

file_list_modified callback changed in karma 0.13 #25

Merged
merged 2 commits into from
Sep 11, 2015
Merged

file_list_modified callback changed in karma 0.13 #25

merged 2 commits into from
Sep 11, 2015

Conversation

hikirsch
Copy link
Contributor

in karma 0.13, they removed the filesPromise and replaced it with the real files object. this pull request fixes that.

@hikirsch
Copy link
Contributor Author

Fixes: #23

@zzo
Copy link
Contributor

zzo commented Sep 9, 2015

what's up with the travis failures? something about firefox not being available?

@hikirsch
Copy link
Contributor Author

i don't know. there's no mention of firefox anywhere in the project. is there a way to test travis without having to push to the repo?

@zzo
Copy link
Contributor

zzo commented Sep 11, 2015

yes there is a little 'reload' icon in the upper right corner or the results which I just clicked to re-run it.
Looks like the issue is after running 'grunt test' the .travis.yml file then runs:

./node_modules/karma/bin/karma start test-app/karma.conf.js --browsers Firefox --single-run --reporters dots

but the karma-firefox-launcher plugin is never installed so karma barfs.
Maybe just adding

npm install karma-firefox-launcher

to .travis.yml in before_script block will fix it? Do you mind adding that to your PR and giving it another shot?

thanks!!
Mark

PS And yes I have no idea how this ever worked :)

@hikirsch
Copy link
Contributor Author

looks like that fixed it.

zzo added a commit that referenced this pull request Sep 11, 2015
file_list_modified callback changed in karma 0.13
@zzo zzo merged commit d0827b3 into karma-runner:master Sep 11, 2015
@zzo
Copy link
Contributor

zzo commented Sep 11, 2015

great thanks!!
karma-closure 0.1.1 on npm
Mark

On Fri, Sep 11, 2015 at 1:46 PM, Adam S. Kirschner <notifications@github.com

wrote:

looks like that fixed it.


Reply to this email directly or view it on GitHub
#25 (comment)
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants