Skip to content

Commit

Permalink
Merge pull request #61 from awaigand/main
Browse files Browse the repository at this point in the history
fix: electron requires numbers as parameters
  • Loading branch information
PippoRaimondi authored Dec 14, 2021
2 parents 40fda1a + a2aa929 commit 36f137c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/plugin.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,9 +98,9 @@ const getCompareSnapshotsPlugin = on => {
}
if (browser.name === 'electron') {
// eslint-disable-next-line no-param-reassign
launchOptions.preferences.width = width
launchOptions.preferences.width = Number.parseInt(width, 10)
// eslint-disable-next-line no-param-reassign
launchOptions.preferences.height = height
launchOptions.preferences.height = Number.parseInt(height, 10)
}
if (browser.name === 'firefox') {
launchOptions.args.push(`--width=${width}`)
Expand Down

0 comments on commit 36f137c

Please sign in to comment.