Refactor profiles usage to projects

Author: Anuj

src/services/config.js

@@ -20,7 +20,7 @@ class ConfigData {
     this.edge = undefined;
     this.email = {};
     this.prompts = {};
-    this.profiles = [];
+    this.projects = [];
     this.activeProfile = null;
   }
 
@@ -66,7 +66,7 @@ class ConfigData {
       if (profileId) {
         // Clean the profile ID.
         profileId = this.sanitize(profileId);
-        profile = this.profiles.find((p) => p.id === profileId);
+        profile = this.projects.find((p) => p.id === profileId);
       } else {
         profile = this.getActiveProfile();
       }
@@ -90,19 +90,19 @@ class ConfigData {
 
   getActiveProfile() {
     let profile;
-    if (this.profiles.length > 0) {
+    if (this.projects.length > 0) {
       if (this.activeProfile) {
-        profile = this.profiles.find((p) => p.id === this.activeProfile);
+        profile = this.projects.find((p) => p.id === this.activeProfile);
       }
       if (!profile) {
-        profile = this.profiles[0];
+        profile = this.projects[0];
       }
     }
     return profile;
   }
 
   removeProfile(profileToRemove) {
-    this.profiles = this.profiles.filter((profile) => {
+    this.projects = this.projects.filter((profile) => {
       return profile.id !== profileToRemove.id;
     });
     if (profileToRemove.id === this.activeProfile) {
@@ -121,7 +121,7 @@ class ConfigData {
       existing.accountSid = accountSid;
       existing.region = region;
     } else {
-      this.profiles.push(new ConfigDataProfile(id, accountSid, region));
+      this.projects.push(new ConfigDataProfile(id, accountSid, region));
     }
   }
 
@@ -147,8 +147,8 @@ class ConfigData {
     this.email = configObj.email || {};
     this.prompts = configObj.prompts || {};
     // Note the historical 'projects' naming.
-    configObj.profiles = configObj.projects || [];
-    configObj.profiles.forEach((profile) => this.addProfile(profile.id, profile.accountSid, profile.region));
+    configObj.projects = configObj.projects || [];
+    configObj.projects.forEach((project) => this.addProfile(project.id, project.accountSid, project.region));
     this.setActiveProfile(configObj.activeProject);
   }
 
@@ -181,7 +181,7 @@ class Config {
       email: configData.email,
       prompts: configData.prompts,
       // Note the historical 'projects' naming.
-      projects: configData.profiles,
+      projects: configData.projects,
       activeProject: configData.activeProfile,
     };
 

test/services/config.test.js

@@ -14,19 +14,19 @@ describe('services', () => {
         const configData = new ConfigData();
         configData.addProfile('newProfile', constants.FAKE_ACCOUNT_SID, 'dev');
 
-        expect(configData.profiles[0].id).to.equal('newProfile');
-        expect(configData.profiles[0].accountSid).to.equal(constants.FAKE_ACCOUNT_SID);
-        expect(configData.profiles[0].region).to.equal('dev');
+        expect(configData.projects[0].id).to.equal('newProfile');
+        expect(configData.projects[0].accountSid).to.equal(constants.FAKE_ACCOUNT_SID);
+        expect(configData.projects[0].region).to.equal('dev');
       });
 
       test.it('should update an existing profile', () => {
         const configData = new ConfigData();
         configData.addProfile('activeProfile', constants.FAKE_ACCOUNT_SID, 'dev');
         configData.addProfile('activeProfile', 'new-account-sid');
 
-        expect(configData.profiles[0].id).to.equal('activeProfile');
-        expect(configData.profiles[0].accountSid).to.equal('new-account-sid');
-        expect(configData.profiles[0].region).to.be.undefined;
+        expect(configData.projects[0].id).to.equal('activeProfile');
+        expect(configData.projects[0].accountSid).to.equal('new-account-sid');
+        expect(configData.projects[0].region).to.be.undefined;
       });
     });
 
@@ -161,10 +161,10 @@ describe('services', () => {
           id: 'DOES_NOT_EXIST',
           accountSid: 'fake_SID',
         };
-        const originalLength = configData.profiles.length;
+        const originalLength = configData.projects.length;
         configData.removeProfile(fakeProfile);
 
-        expect(configData.profiles.length).to.equal(originalLength);
+        expect(configData.projects.length).to.equal(originalLength);
       });
 
       test.it('removes profile', () => {
@@ -175,8 +175,8 @@ describe('services', () => {
         const profile = configData.getProfileById('secondProfile');
         configData.removeProfile(profile);
 
-        expect(configData.profiles[1].id).to.equal('thirdProfile');
-        expect(configData.profiles[1].accountSid).to.equal('newest_account_SID');
+        expect(configData.projects[1].id).to.equal('thirdProfile');
+        expect(configData.projects[1].accountSid).to.equal('newest_account_SID');
       });
 
       test.it('removes active profile', () => {
@@ -187,8 +187,8 @@ describe('services', () => {
         const profile = configData.setActiveProfile('firstProfile');
         configData.removeProfile(profile);
 
-        expect(configData.profiles[1].id).to.equal('thirdProfile');
-        expect(configData.profiles[1].accountSid).to.equal('newest_account_SID');
+        expect(configData.projects[1].id).to.equal('thirdProfile');
+        expect(configData.projects[1].accountSid).to.equal('newest_account_SID');
         expect(configData.activeProfile).to.equal(null);
       });
     });