chore(build): fix version placeholder matching

gkalpak · gkalpak · commit 337b9a9418e6 · 2016-08-12T11:19:22.000+03:00
During the `build` task, the version placeholders will be replaced with the actual values using a RegExp, which expects the placeholders to be surrounded by double quotes. By replacing the quotes from double to single in #15011, the RegExp was not able to match the placeholders. (For reference, the RegExps that match and replace the version placeholders are in [lib/grunt/utils.js][1].) [1]: https://github.com/angular/angular.js/blob/859348c7f61ff5f93b9f81eb7f46842bd018d8e3/lib/grunt/utils.js#L125-L130
diff --git a/src/AngularPublic.js b/src/AngularPublic.js
@@ -114,11 +114,15 @@
  * - `codeName` – `{string}` – Code name of the release, such as "jiggling-armfat".
  */
 var version = {
-  full: '"NG_VERSION_FULL"',    // all of these placeholder strings will be replaced by grunt's
-  major: 'NG_VERSION_MAJOR',    // package task
-  minor: 'NG_VERSION_MINOR',
-  dot: 'NG_VERSION_DOT',
+  // These placeholder strings will be replaced by grunt's `build` task.
+  // They need to be surrounded by double-quotes.
+  /* eslint-disable quotes */
+  full: '"NG_VERSION_FULL"',
+  major: "NG_VERSION_MAJOR",
+  minor: "NG_VERSION_MINOR",
+  dot: "NG_VERSION_DOT",
   codeName: '"NG_VERSION_CODENAME"'
+  /* eslint-enable quotes */
 };
 
 
diff --git a/test/e2e/fixtures/version/index.html b/test/e2e/fixtures/version/index.html
@@ -0,0 +1,6 @@
+<!DOCTYPE html>
+<html ng-app>
+  <body>
+    <script src="angular.js"></script>
+  </body>
+</html>
diff --git a/test/e2e/tests/version.spec.js b/test/e2e/tests/version.spec.js
@@ -0,0 +1,51 @@
+'use strict';
+
+describe('angular.version', function() {
+  var version;
+
+  beforeEach(function() {
+    loadFixture('version');
+    version = browser.driver.executeScript('return angular.version');
+  });
+
+
+  it('should expose the current version as object', function() {
+    expect(version).toEqual(jasmine.any(Object));
+  });
+
+  it('should contain property `full` (string)', function() {
+    expect(version.then(get('full'))).toEqual(jasmine.any(String));
+  });
+
+  it('should contain property `major` (number)', function() {
+    expect(version.then(get('major'))).toEqual(jasmine.any(Number));
+  });
+
+  it('should contain property `minor` (number)', function() {
+    expect(version.then(get('minor'))).toEqual(jasmine.any(Number));
+  });
+
+  it('should contain property `dot` (number)', function() {
+    expect(version.then(get('dot'))).toEqual(jasmine.any(Number));
+  });
+
+  it('should contain property `codeName` (string)', function() {
+    expect(version.then(get('codeName'))).toEqual(jasmine.any(String));
+  });
+
+  it('should have `full` === `"major.minor.dot"`', function() {
+    expect(version.then(validate)).toBe(true);
+
+    function validate(ver) {
+      return ver.full.indexOf([ver.major, ver.minor, ver.dot].join('.')) === 0;
+    }
+  });
+
+
+  // Helpers
+  function get(prop) {
+    return function getter(obj) {
+      return obj[prop];
+    };
+  }
+});
