feat(repository-name): return the name of the created repository as p…

…art of vcs results

package.json

@@ -90,6 +90,7 @@
     "@form8ion/repository-settings": "^1.1.3",
     "@octokit/rest": "^20.0.0",
     "@travi/cli-messages": "^1.1.1",
+    "deepmerge": "^4.3.1",
     "lodash.zip": "^4.2.0",
     "octokit-auth-netrc": "^3.1.1"
   }

src/repository/scaffolder.js

@@ -70,10 +70,10 @@ export default async function ({name, owner, visibility, octokit}) {
 
   const {data: {login: authenticatedUser}} = await octokit.users.getAuthenticated();
 
-  if (owner === authenticatedUser) return {vcs: await createForUser(octokit, owner, name, visibility)};
+  if (owner === authenticatedUser) return {vcs: {...await createForUser(octokit, owner, name, visibility), name}};
 
   if (await authenticatedUserIsMemberOfRequestedOrganization(owner, octokit)) {
-    return {vcs: await createForOrganization(octokit, owner, name, visibility)};
+    return {vcs: {...await createForOrganization(octokit, owner, name, visibility), name}};
   }
 
   throw new Error(`User ${authenticatedUser} does not have access to create a repository in the ${owner} account`);

src/repository/scaffolder.test.js

@@ -40,7 +40,7 @@ describe('creation', () => {
       });
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Public', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
     });
 
     it('should not create the repository when it already exists', async () => {
@@ -50,7 +50,7 @@ describe('creation', () => {
       when(get).calledWith({owner: account, repo: name}).mockResolvedValue(repoDetailsResponse);
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Public', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
       expect(createForAuthenticatedUser).not.toHaveBeenCalled();
     });
 
@@ -64,7 +64,7 @@ describe('creation', () => {
       });
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Private', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
     });
 
     it('should rethrow other errors', async () => {
@@ -105,7 +105,7 @@ describe('creation', () => {
       });
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Public', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
     });
 
     it('should not create the repository when it already exists', async () => {
@@ -115,7 +115,7 @@ describe('creation', () => {
       when(get).calledWith({owner: account, repo: name}).mockResolvedValue(repoDetailsResponse);
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Public', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
       expect(createInOrg).not.toHaveBeenCalled();
     });
 
@@ -129,7 +129,7 @@ describe('creation', () => {
       });
 
       expect(await scaffoldRepository({name, owner: account, visibility: 'Private', octokit: client}))
-        .toEqual({vcs: {sshUrl, htmlUrl}});
+        .toEqual({vcs: {sshUrl, htmlUrl, name}});
     });
 
     it('should rethrow other errors', async () => {

test/integration/features/step_definitions/repository-steps.js

@@ -126,6 +126,7 @@ Given('the project is not versioned on GitHub', async function () {
 Then('repository details are returned', async function () {
   assert.equal(this.result.vcs.sshUrl, sshUrl);
   assert.equal(this.result.vcs.htmlUrl, htmlUrl);
+  assert.equal(this.result.vcs.name, this.projectName);
 });
 
 Then('no repository details are returned', async function () {