Updated readme, version, adding note about fingerprint name format

Author: gcscoder

README.md

@@ -126,6 +126,8 @@ ASSET_FINGERPRINT = '0556f9ab7cbd607d10ed'
 CUSTOM_ASSET_FINGERPRINT = '921dcffe35e5f2740ef5'
 ```
 
+The format for the fingerprint name must uppercase separating words with underscore i.e. "CUSTOM_ASSET_FINGERPRINT".  The name also needs to end with "_FINGERPRINT".
+
 
 ### Credit
 Concept adapted from @samullen - http://samuelmullen.com/articles/replacing-the-rails-asset-pipeline-with-webpack-and-yarn/

index.js

@@ -24,7 +24,7 @@ function AssetFingerprint(initializerDirectory, needsFingerprint = true, fingerp
 
   function _validateFingerprintName(fingerprintName) {
     if(!/^([A-Z]|_)*_FINGERPRINT$/.test(fingerprintName)) {
-      throw new Error('Please supply a fingerprint name that is all caps separating words by underscores (i.e. CUSTOM_ASSET_FINGERPRINT).');
+      throw new Error('Please supply a fingerprint name that is all caps separating words by underscores ending with "_FINGERPRINT" (i.e. CUSTOM_ASSET_FINGERPRINT).');
     }
   }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "asset-fingerprint-webpack-rails",
-  "version": "1.1.1",
+  "version": "1.2.1",
   "description": "A webpack plugin to fingerprint your JS for consumption by Rails",
   "main": "index.js",
   "scripts": {

spec/indexSpec.js

@@ -67,9 +67,16 @@ describe("AssetFingerprint", function() {
         fingerprint = new AssetFingerprint(dir, true, "TEST_FINGERPRINT");
         expect(fingerprint.fingerprintName).toEqual("TEST_FINGERPRINT");
       });
+
+      it("should not validate when file does not exist.", function () {
+        var dir = 'config/initializers';
+        fingerprint = new AssetFingerprint(dir, true, "TEST_FINGERPRINT");
+        expect(fingerprint.fingerprintName).toEqual("TEST_FINGERPRINT");
+      });
     });
 
     describe("fingerprint name", function() {
+      const formatErrorMessage = 'Please supply a fingerprint name that is all caps separating words by underscores ending with "_FINGERPRINT" (i.e. CUSTOM_ASSET_FINGERPRINT).';
       beforeEach(function() {
         spyOn(fs, "existsSync").and.returnValue(true);
         spyOn(fs, "readFileSync").and.returnValue("TEST_FINGERPRINT");
@@ -84,13 +91,13 @@ describe("AssetFingerprint", function() {
       it("should validate the file name characters.", function() {
         var dir = 'config/initializers';
         fingerprint = function() { new AssetFingerprint(dir, true, "asset_FINGERPRINT"); }
-        expect(fingerprint).toThrowError('Please supply a fingerprint name that is all caps separating words by underscores (i.e. CUSTOM_ASSET_FINGERPRINT).');
+        expect(fingerprint).toThrowError(formatErrorMessage);
       });
 
       it("should validate the file name format.", function() {
         var dir = 'config/initializers';
         fingerprint = function() { new AssetFingerprint(dir, true, "BAD_VALUE"); }
-        expect(fingerprint).toThrowError('Please supply a fingerprint name that is all caps separating words by underscores (i.e. CUSTOM_ASSET_FINGERPRINT).');
+        expect(fingerprint).toThrowError(formatErrorMessage);
       });
     });
 
@@ -107,19 +114,6 @@ describe("AssetFingerprint", function() {
       });
     });
 
-    describe("fingerprint name", function() {
-      beforeEach(function() {
-        spyOn(fs, "existsSync").and.returnValue(false);
-        spyOn(fs, "readFileSync").and.returnValue("");
-      });
-
-      it("should not validate when file does not exist.", function () {
-        var dir = 'config/initializers';
-        fingerprint = new AssetFingerprint(dir, true, "TEST_FINGERPRINT");
-        expect(fingerprint.fingerprintName).toEqual("TEST_FINGERPRINT");
-      });
-    });
-
     describe("apply", function() {
       beforeEach(function() {
         spyOn(fs, "writeFileSync");