fix(e2e): Use current directory so e2e test won't complain about directory not exists

[tinayuangao]

R: @jelbourn

[devversion]

Should we add this to the .gitignore?

[jelbourn]

Yeah, the directory should be marked as ignored.

[crisbeto]

Also shouldn't we disable screenshots when developing locally? I feel like they would stack up a lot.

[jelbourn]

@tinayuangao that's another good point- @devversion @crisbeto is there an easy way for the screenshot utility to know if it's local? I'd assume there's an environment variable we can check.

[tinayuangao]

Use process.env['TRAVIS'] to check whether it's local. Please take another look. Thanks!

[devversion]

Yeah checking the TRAVIS variable is the right way. LGTM

[devversion]

Just one question, how can we take a look at those screenshots on the CI?

It's probably for the screenshot-diffs?

[tinayuangao]

Just one question, how can we take a look at those screenshots on the CI?

Currently there's no way to take a look. We will upload the images, compare them and generate a report later.

[devversion]

@tinayuangao Cool. I have some nice ideas about displaying those reports since we also have data about the payload size etc.

Edit: Removing the LGTM label as @jelbourn is the intended reviewer 😄

[jelbourn]

LGTM

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}