Fix: Creating a service file with a custom name falls back to the pac…

…kage.name when package.json has "files" (#108)

* Refactor service file name to accept a name rather than package obj

* fix: Creating custom service when file defined falls back to package.name

lib/files.js

@@ -3,11 +3,11 @@
 const path = require('path');
 
 module.exports = {
-  serviceFileName: function (pkg) {
-    return `${pkg.name}.service`;
+  serviceFileName: function (name) {
+    return `${name}.service`;
   },
   serviceFile: function (root, pkg) {
-    return path.resolve(root, this.serviceFileName(pkg));
+    return path.resolve(root, this.serviceFileName(pkg.name));
   },
   specsDirectory: function (root) {
     return path.resolve(root, 'SPECS');

lib/generate.js

@@ -40,9 +40,9 @@ function relativeToRoot(root, files) {
   });
 }
 
-function getArchiveWhitelist(pkg) {
+function getArchiveWhitelist(pkg, customName) {
   return {
-    service: files.serviceFileName(pkg),
+    service: files.serviceFileName(customName || pkg.name),
     main: pkg.main,
     files: pkg.files
   };
@@ -59,7 +59,7 @@ module.exports = async (root, pkg, release, customName) => {
 
   const serviceFile = generateServiceFile(root, customPackage);
   const specFile = generateSpecFile(specsDirectory, customPackage, release);
-  const archiveWhitelist = getArchiveWhitelist(pkg);
+  const archiveWhitelist = getArchiveWhitelist(pkg, customName);
 
   await archiver.compress(root, sourcesArchive, archiveWhitelist);
 

test/generate.js

@@ -112,7 +112,7 @@ describe('generate', () => {
       {
         files: undefined,
         main: 'index.js',
-        service: 'my-cool-api.service'
+        service: 'penguin.service'
       }
     );
   });
@@ -135,6 +135,24 @@ describe('generate', () => {
     );
   });
 
+  it('creates the service file with a custom name if specified and "files" defined', async () => {
+    await generate('/path/to/project', pkgWithWhitelist, null, 'penguin');
+    sandbox.assert.calledWith(
+      archiver.compress,
+      '/path/to/project',
+      '/path/to/project/SOURCES/penguin.tar.gz',
+      {
+        main: 'server.js',
+        files: [
+          'lib',
+          'routes',
+          'index.js'
+        ],
+        service: 'penguin.service'
+      }
+    );
+  });
+
   it('creates the service file with a custom name if specified', async () => {
     await generate('/path/to/project', pkg, 1, 'penguin');
     sandbox.assert.calledWith(