Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Film support to ForceVue #55

Closed
wants to merge 4 commits into from
Closed

Adding Film support to ForceVue #55

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 24, 2016

These changes are intended to close issue #9 .

One note: I was unable to get Selenium and Nightwatch to play nicely together and therefore could not verify that the e2e tests pass. That's why I created the first commit -- I wanted to move my changes to a Linux machine and see if I could get the e2e tests to run there (I couldn't). However, the unit tests pass and I have tested the code, and it works fine as far as I can tell. Obviously I would prefer to verify the e2e tests before committing the code, but I've hit a wall on my end so I'm submitting what I have. I look forward to seeing the results of the e2e tests and making any necessary changes to get them to pass.

@ghost
Copy link
Author

ghost commented Oct 24, 2016

All of my tests seem to be passing at this point; these other errors look to me like persistent issues rather than problems with my commit. I was interested to see the e2e output on CircleCI for the Firefox tests -- that's more or less exactly the error I was getting as well.

@alexkramer
Copy link
Owner

You are correct @dimerization . There appears to be an issue with Firefox on the Circle CI build server. I will investigate that.

@alexkramer alexkramer closed this Mar 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants