CB-12517 : pkgjson displayname should equal config's name and pkgjson.name should...
authorAudrey So <audreyso@apache.org>
Wed, 8 Mar 2017 00:07:13 +0000 (16:07 -0800)
committerSteve Gill <stevengill97@gmail.com>
Wed, 29 Mar 2017 17:15:27 +0000 (10:15 -0700)
 This closes #3

index.js
spec/create.spec.js

index e25f5cf..cdc6d4e 100644 (file)
--- a/index.js
+++ b/index.js
@@ -287,9 +287,19 @@ module.exports = function(dir, optionalId, optionalName, cfg, extEvents) {
         // Update package.json name and version fields
         if (fs.existsSync(pkgjsonPath)) {
             var pkgjson = require(pkgjsonPath);
+
+            // Pkjson.displayName should equal config's name.
             if (cfg.name) {
-                pkgjson.name = cfg.name.toLowerCase();
+                pkgjson.displayName = cfg.name;
+            }
+            // Pkjson.name should equal config's id.
+            if(cfg.id) {
+                pkgjson.name = cfg.id.toLowerCase();
+            } else if(!cfg.id) {
+                // Use default name.
+                pkgjson.name = 'helloworld';
             }
+
             pkgjson.version = '1.0.0';
             fs.writeFileSync(pkgjsonPath, JSON.stringify(pkgjson, null, 4), 'utf8');
         }
@@ -473,4 +483,4 @@ function writeToConfigJson(project_root, opts, autoPersist) {
     if (!fs.existsSync(copyDst) && fs.existsSync(copySrc)) {
         shell.cp(copySrc, projectDir);
     }
- }
+ }
\ No newline at end of file
index 4cc445e..ddbc64b 100644 (file)
@@ -175,7 +175,8 @@ describe('create end-to-end', function() {
 
         // Check that we got package.json (the correct one)
         var pkjson = require(path.join(project, 'package.json'));
-        expect(pkjson.name).toEqual(appName.toLowerCase());
+        // Pkjson.displayName should equal config's name.
+        expect(pkjson.displayName).toEqual(appName);
         expect(pkjson.valid).toEqual('true');
 
         // Check that we got the right config.xml
@@ -418,7 +419,8 @@ describe('create end-to-end', function() {
 
                     // Check that we got package.json (the correct one) and it was changed
                     var pkjson = require(path.join(project, 'package.json'));
-                    expect(pkjson.name).toEqual(appName.toLowerCase());
+                    // Pkjson.name should equal config's id.
+                    expect(pkjson.name).toEqual(appId.toLowerCase());
                     expect(pkjson.valid).toEqual('true');
                 }
                 var config = {