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

Fail test: show file import error #30

Closed
wants to merge 6 commits into from
Closed

Conversation

Sam152
Copy link
Collaborator

@Sam152 Sam152 commented Dec 16, 2014

This test shows a fail based on the bug reported in #29.

Fail

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

Seeing this error is also blocked by: https://www.drupal.org/node/2392979

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

Hm, it isn't getting to that point yet either, as the file in the export is relative to my dev environment. Need to update relative to something the CI server has access to during the test run.

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

Hard coded file path to sites/default/files didn't work because $this->public_files_directory is random and not sites/default/files during tests.

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

@larowlan, finally have travis outputting the bug I was running into.

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

Looked a bit deeper into this with the help of a colleague. Filed this core bug: https://www.drupal.org/node/2394063

@Sam152
Copy link
Collaborator Author

Sam152 commented Dec 16, 2014

Also fixed by: https://www.drupal.org/node/1927648

@larowlan larowlan closed this Feb 7, 2016
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