@jan-molak The main issue with what's proposed here is this assertion:

photo.map(value=>expect(value.toString('base64')).to.equal(photo.base64EncodedValue));

base64EncodedValue is a member variable that includes any extra whitespace.value.toString('base64') produces the "normalized" version that includes no whitespace, failing.

Hello@greghopkins! Nice catch. Hmm, I think I'd be leaning more towards sanitising the input rather than relaxing the constraint.

This would require us to change:

• WebdriverIO implementation

  serenity-js/packages/webdriverio/src/screenplay/models/WebdriverIOPage.ts

  Line 150 in06ae5d8

  returnawaitthis.browser.takeScreenshot();

• Protractor implementation

  serenity-js/packages/protractor/src/screenplay/models/ProtractorPage.ts

  Line 213 in06ae5d8

  returnpromised(this.browser.takeScreenshot());

• and leavePlaywright alone since it uses a different protocol (I don't think you can use Playwright with BrowserStack, can you?)

To avoid code duplication we could introduce a protected method in the basePage class likePage.base64Rfc2045ToBase64(...) to sanitise the result ofthis.browser.takeScreenshot(), or add such converter tocore/io instead.

What do you think?