tree 816b423c30ca002bce2a316ae0ddf4ed1e1fcdd8
parent 68f122f166922fbb440591cf58bd9ef3735bc4a9
author Han Seoul-Oh <laughinghan@gmail.com> 1486089669 -0800
committer Han Seoul-Oh <laughinghan@gmail.com> 1486089669 -0800

CI: Reorg screenshots.js for clarity

- rename list of browser info objects
- make `pinned` optional boolean key
- log sessionID
- log browser version, not just name and platform
- use info on the actual browser we're using as reported by Sauce, not
  our config info
- check for specifically Firefox and IE for single-screenshot case,
  rather than checking that it's not one of the ones that require
  multiple screenshots (wish this was feature-based, i.e. it would
  actually measure the returned screenshot and see if it's less than the
  viewport height)
- factor out capturing browser logs to a subsequent .then() after the
  if/else and the loop, rather than duplicating it (also, don't write
  logs at all if no logs returned (Edge/IE))
- loop starts out by taking screenshot and then scrolls, rather than
  starting by scrolling to 0