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

Different reference names in different environments #291

Closed
gotbahn opened this issue Sep 25, 2016 · 3 comments
Closed

Different reference names in different environments #291

gotbahn opened this issue Sep 25, 2016 · 3 comments

Comments

@gotbahn
Copy link

gotbahn commented Sep 25, 2016

Have a trouble with running tests in CI (previous week or so all was fine):

Bitmap file generation completed.
COMMAND | Executing core for `report`
COMMAND | Command `report` ended with an error
COMMAND | Reference image not found: /home/travis/build/godban/godban.github.io/tests/visual/backstop_data/bitmaps_reference/01876053680_Index_0_logo_0_phone.png

Happening because of prefix in reference name 01876053680_Index_0_logo_0_phone.png which is different from my local & committed one 2005537259_Index_0_logo_0_phone.png

It is looks like prefix is environment depending. Why it was changed? Previously all was fine without prefixes.

@garris
Copy link
Owner

garris commented Sep 25, 2016

@godban Thanks for posting. The hash was added because we anticipate users with many config files may run into name conflicts with matching scenario names from different configs. Therefore BackstopJS attempts to create a unique ID for each un-named config. This does present an issue when sharing files.

To avoid this conflict you can simply name your scenarios by using the ID property. Please see the docs here... https://github.com/garris/BackstopJS/tree/version_2_0#required-config-properties

Please let me know if this solves your issue -- I don't want this new feature to overly complicate the app.

Thanks!

@gotbahn
Copy link
Author

gotbahn commented Sep 26, 2016

@garris thx for quick response

Setting id resolves this issue.

@gotbahn gotbahn closed this as completed Sep 26, 2016
@garris
Copy link
Owner

garris commented Sep 26, 2016

Glad it worked out. Cheers!

maoberlehner added a commit to avalanchesass/avalanche that referenced this issue Apr 23, 2017
This fixes problems with running tests in different environments
garris/BackstopJS#291
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

No branches or pull requests

2 participants