Add default local backstop.json file and improve actions tests

Author: juanjol

backstopjsFiles/local/backstop.json

@@ -0,0 +1,64 @@
+{
+  "id": "backstop_default",
+  "viewports": [
+    {
+      "label": "phone",
+      "width": 375,
+      "height": 480
+    },
+    {
+      "label": "tablet",
+      "width": 1024,
+      "height": 768
+    },
+    {
+      "label": "Desktop",
+      "width": 1920,
+      "height": 1080
+    }
+  ],
+  "onBeforeScript": "puppet/onBefore.js",
+  "onReadyScript": "puppet/onReady.js",
+  "scenarios": [
+    {
+      "label": "Homepage",
+      "url": "http://web/",
+      "selectorExpansion": true,
+      "readySelector": "body",
+      "misMatchThreshold" : 0.1
+    }
+  ],
+  "paths": {
+    "bitmaps_reference": "bitmaps_reference",
+    "bitmaps_test": "bitmaps_test",
+    "engine_scripts": "engine_scripts",
+    "html_report": "html_report",
+    "ci_report": "ci_report"
+  },
+  "report": [
+    "browser",
+    "CI"
+  ],
+  "ci": {
+    "format" :  "junit" ,
+    "testSuiteName" :  "backstopJS"
+  },
+  "engine": "puppeteer",
+  "engineOptions": {
+    "ignoreHTTPSErrors": true,
+    "slowMo": 500,
+    "args": [
+      "--no-sandbox",
+      "--disable-setuid-sandbox",
+      "--disable-gpu",
+      "--force-device-scale-factor=1",
+      "--disable-infobars=true",
+      "--hide-scrollbars"
+    ]
+
+  },
+  "asyncCaptureLimit": 10,
+  "asyncCompareLimit": 30,
+  "debug": false,
+  "debugWindow": false
+}

backstopjsFiles/local/engine_scripts/imageStub.jpg

@@ -0,0 +1,2 @@
+module.exports = async (page, scenario, vp) => {
+};

backstopjsFiles/local/engine_scripts/puppet/onBefore.js

@@ -0,0 +1,3 @@
+module.exports = async (page, scenario, vp) => {
+
+};

backstopjsFiles/local/engine_scripts/puppet/onReady.js

@@ -1,16 +1,14 @@
 name: backstopjs
 
-pre_install_actions:
-  - test -d ${DDEV_APPROOT}/tests/backstopjs/local || mkdir -p ${DDEV_APPROOT}/tests/backstopjs/local
-  - test -f "${DDEV_APPROOT}/tests/backstopjs/local/.gitignore" || printf "## ddev-generated\n**/bitmaps_test\n**/html_report\n" > ${DDEV_APPROOT}/tests/backstopjs/local/.gitignore
-  - grep -q "## ddev-generated" ${DDEV_APPROOT}/tests/backstopjs/local/.gitignore && printf "## ddev-generated\n**/bitmaps_test\n**/html_report\n" > ${DDEV_APPROOT}/tests/backstopjs/local/.gitignore
-
 project_files:
 - docker-compose.backstopjs.yaml
 - backstopjsBuild/
+- backstopjsFiles
 - commands/
 
 post_install_actions:
+ - test -d ../tests/backstopjs || mkdir -p ../tests/backstopjs
+ - test -d ../tests/backstopjs/local || mv backstopjsFiles/local ../tests/backstopjs/
  - echo "Install finished. Please restart ddev with 'ddev restart'"
  - echo "After that create your backstop config, e.g. run ddev backstop init."
  - echo "If you need to set a custom backstopjs version, add an .env file inside .ddev folder with variable BACKSTOPJS_VERSION set"

install.yaml

@@ -26,12 +26,15 @@ teardown() {
   ddev restart
 
   # backstop is installed and can show its version
-  ddev backstopjs version | grep 'Command "version" successfully executed'
+  ddev backstopjs local version | grep 'Command "version" successfully executed'
+
+  # Test should fail because there is no reference bitmaps
+  ddev backstopjs local test | grep 'Command "version" successfully executed' || true
+  # Create reference bitmaps
+  ddev backstopjs local reference
+  # Test should pass because there is a reference bitmaps
+  ddev backstopjs local test
 
-  # openReport and remote commands show an error message
-  set +o pipefail
-  ddev backstopjs openReport | grep -q 'This does not work for backstop in ddev'
-  ddev backstopjs remote | grep -q 'This does not work for backstop in ddev'
 }
 
 #@test "install from release" {