protractor-jasmine2-screenshot-reporter - npm Package Compare versions

Comparing version 0.0.4 to 0.0.5

index.js

 var fs     = require('fs'),
     mkdirp = require('mkdirp'),
-    _      = require('lodash');
+    _      = require('lodash'),
+    path   = require('path'),
+    hat    = require('hat');
 
@@ -47,2 +49,6 @@ function Jasmine2ScreenShotReporter(opts) {
 
+    var pathBuilder = function(spec, suites, capabilities) {
+      return hat();
+    };
+
     // TODO: more options
@@ -54,2 +60,3 @@ opts          = opts || {};
     opts.captureOnlyFailedSpecs = opts.captureOnlyFailedSpecs || false;
+    opts.pathBuilder = opts.pathBuilder || pathBuilder;
 
@@ -119,5 +126,11 @@
             browser.getCapabilities().then(function (capabilities) {
-                // TODO: capabilities.browserName
-                spec.filename = spec.status + '-' + spec.fullName + '.png';
-                writeScreenshot(png, spec.filename);
+                spec.filename = opts.pathBuilder(spec, suites, capabilities) + '.png';
+                var fullpath = path.join(opts.dest, spec.filename);
+
+                mkdirp(path.dirname(fullpath), function(err) {
+                    if(err) {
+                        throw new Error('Could not create directory for ' + fullpath);
+                    }
+                    writeScreenshot(png, spec.filename);
+                });
             });
@@ -124,0 +137,0 @@ });

package.json

 {
   "name": "protractor-jasmine2-screenshot-reporter",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "Use the screenshot reporter to capture screenshots after each executed Protractor test case.",
@@ -11,4 +11,6 @@ "main": "index.js",
   "dependencies": {
+    "hat": "0.0.3",
     "lodash": "^3.0.0",
-    "mkdirp": "^0.5.0"
+    "mkdirp": "^0.5.0",
+    "path": "^0.11.14"
   },
@@ -15,0 +17,0 @@ "keywords": [

README.md

@@ -69,1 +69,14 @@ ## Protractor screenshot reporter for Jasmine2
 Default is <code>false</code>
+
+### Path Builder (optional)
+
+Function used to build custom paths for screenshots. For example:
+
+<pre><code>jasmine.getEnv().addReporter(new ScreenShotReporter({
+   pathBuilder: function(currentSpec, suites, browserCapabilities) {
+      // will return chrome/your-spec-name.png
+      return browserCapabilities.browserName + '/' + spec.fullName);
+   }
+}));</code></pre>
+
+By default, the path builder will generate a random ID for each spec.

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

9230

increased by11.41%

Lines of code

138

increased by8.66%

Number of lines in readme file

82

increased by18.84%

Worsened metrics

Dependency count

4

increased by100%

Dependency changes

• + Addedhat@0.0.3

• + Addedpath@^0.11.14

• + Addedhat@0.0.3(transitive)

• + Addedpath@0.11.14(transitive)