Add test fixture for fixing translations

__snapshots__/test.js.snap

@@ -12,6 +12,22 @@ exports[`Test Fixtures decorators 1`] = `
 "
 `;
 
+exports[`Test Fixtures decorators 2`] = `Map {}`;
+
+exports[`Test Fixtures embedded-conditionals 1`] = `
+"[1/4] 🔍  Finding JS and HBS files...
+[2/4] 🔍  Searching for translations keys in JS and HBS files...
+[3/4] ⚙️   Checking for unused translations...
+[4/4] ⚙️   Checking for missing translations...
+
+ 👏  No unused translations were found!
+
+ 👏  No missing translations were found!
+"
+`;
+
+exports[`Test Fixtures embedded-conditionals 2`] = `Map {}`;
+
 exports[`Test Fixtures emblem 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
@@ -26,6 +42,8 @@ exports[`Test Fixtures emblem 1`] = `
 "
 `;
 
+exports[`Test Fixtures emblem 2`] = `Map {}`;
+
 exports[`Test Fixtures missing-translations 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
@@ -41,6 +59,8 @@ exports[`Test Fixtures missing-translations 1`] = `
 "
 `;
 
+exports[`Test Fixtures missing-translations 2`] = `Map {}`;
+
 exports[`Test Fixtures no-issues 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
@@ -53,34 +73,90 @@ exports[`Test Fixtures no-issues 1`] = `
 "
 `;
 
-exports[`Test Fixtures unused-translations 1`] = `
+exports[`Test Fixtures no-issues 2`] = `Map {}`;
+
+exports[`Test Fixtures remove-unused-translations 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
 [3/4] ⚙️   Checking for unused translations...
 [4/4] ⚙️   Checking for missing translations...
 
- ⚠️   Found 2 unused translations!
+ ⚠️   Found 3 unused translations!
 
    - foo (used in de.json and en.json)
    - extra (used in de.json)
+   - foo.bar.unnested (used in de.json and en.json)
 
  👏  No missing translations were found!
 "
 `;
 
-exports[`Test Fixtures yaml-translations 1`] = `
+exports[`Test Fixtures remove-unused-translations 2`] = `
+Map {
+  "translations/de.json" => "{
+  \\"hbs-translation\\": \\"Lenkstage\\",
+  \\"js-translation\\": \\"Affentheater\\"
+}",
+  "translations/en.json" => "{
+  \\"hbs-translation\\": \\"HBS!\\",
+  \\"js-translation\\": \\"JS!\\"
+}",
+}
+`;
+
+exports[`Test Fixtures remove-unused-translations-nested 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
 [3/4] ⚙️   Checking for unused translations...
 [4/4] ⚙️   Checking for missing translations...
 
- 👏  No unused translations were found!
+ ⚠️   Found 3 unused translations!
+
+   - foo (used in de.json and en.json)
+   - extra (used in de.json)
+   - parent.child.unusedKey (used in de.json and en.json)
 
  👏  No missing translations were found!
 "
 `;
 
-exports[`Test Fixtures embedded-conditionals 1`] = `
+exports[`Test Fixtures remove-unused-translations-nested 2`] = `
+Map {
+  "translations/de.json" => "{
+  \\"hbs-translation\\": \\"Lenkstage\\",
+  \\"js-translation\\": \\"Affentheater\\",
+  \\"parent\\": {
+    \\"secondChild\\": \\"usedKey\\"
+  }
+}",
+  "translations/en.json" => "{
+  \\"hbs-translation\\": \\"HBS!\\",
+  \\"js-translation\\": \\"JS!\\",
+  \\"parent\\": {
+    \\"secondChild\\": \\"usedKey\\"
+  }
+}",
+}
+`;
+
+exports[`Test Fixtures unused-translations 1`] = `
+"[1/4] 🔍  Finding JS and HBS files...
+[2/4] 🔍  Searching for translations keys in JS and HBS files...
+[3/4] ⚙️   Checking for unused translations...
+[4/4] ⚙️   Checking for missing translations...
+
+ ⚠️   Found 2 unused translations!
+
+   - foo (used in de.json and en.json)
+   - extra (used in de.json)
+
+ 👏  No missing translations were found!
+"
+`;
+
+exports[`Test Fixtures unused-translations 2`] = `Map {}`;
+
+exports[`Test Fixtures yaml-translations 1`] = `
 "[1/4] 🔍  Finding JS and HBS files...
 [2/4] 🔍  Searching for translations keys in JS and HBS files...
 [3/4] ⚙️   Checking for unused translations...
@@ -91,3 +167,5 @@ exports[`Test Fixtures embedded-conditionals 1`] = `
  👏  No missing translations were found!
 "
 `;
+
+exports[`Test Fixtures yaml-translations 2`] = `Map {}`;

fixtures/remove-unused-translations-nested/app/controllers/application.js

@@ -0,0 +1,7 @@
+import Controller from '@ember/controller';
+
+export default Controller.extend({
+  foo: computed('intl.locale', function() {
+    return this.intl.t('js-translation');
+  }),
+});

fixtures/remove-unused-translations-nested/app/templates/application.hbs

@@ -0,0 +1,2 @@
+{{t "hbs-translation"}}
+{{t "parent.secondChild"}}

fixtures/remove-unused-translations-nested/translations/de.json

@@ -0,0 +1,12 @@
+{
+  "hbs-translation": "Lenkstage",
+  "js-translation": "Affentheater",
+  "foo": "BARR!",
+  "extra": "drei",
+  "parent": {
+    "child": {
+      "unusedKey": "unused Key",
+    },
+    "secondChild": "usedKey"
+  }
+}

fixtures/remove-unused-translations-nested/translations/en.json

@@ -0,0 +1,11 @@
+{
+  "hbs-translation": "HBS!",
+  "js-translation": "JS!",
+  "foo": "bar",
+  "parent": {
+    "child": {
+      "unusedKey": "unused Key",
+    },
+    "secondChild": "usedKey"
+  }
+}

fixtures/remove-unused-translations/app/controllers/application.js

@@ -0,0 +1,7 @@
+import Controller from '@ember/controller';
+
+export default Controller.extend({
+  foo: computed('intl.locale', function() {
+    return this.intl.t('js-translation');
+  }),
+});

fixtures/remove-unused-translations/app/templates/application.hbs

@@ -0,0 +1 @@
+{{t "hbs-translation"}}

fixtures/remove-unused-translations/translations/de.json

@@ -0,0 +1,7 @@
+{
+  "hbs-translation": "Lenkstage",
+  "js-translation": "Affentheater",
+  "foo": "BARR!",
+  "extra": "drei",
+  "foo.bar.unnested": "unnested key"
+}

fixtures/remove-unused-translations/translations/en.json

@@ -0,0 +1,6 @@
+{
+  "hbs-translation": "HBS!",
+  "js-translation": "JS!",
+  "foo": "bar",
+  "foo.bar.unnested": "unnested key"
+}

test.js

@@ -5,24 +5,42 @@ let fixtures = fs.readdirSync(`${__dirname}/fixtures/`);
 
 describe('Test Fixtures', () => {
   let output;
+  let writtenFiles;
   let fixturesWithErrors = ['emblem', 'missing-translations', 'unused-translations'];
+  let fixturesWithFix = ['remove-unused-translations', 'remove-unused-translations-nested'];
 
   beforeEach(() => {
     output = '';
+    writtenFiles = new Map();
   });
 
   function log(str = '') {
     output += `${str}\n`;
   }
+  function writeToFile(filePath, updatedTranslations) {
+    let fileName = filePath.split('/').slice(-2).join('/');
+    writtenFiles.set(fileName, updatedTranslations);
+  }
 
   for (let fixture of fixtures) {
     test(fixture, async () => {
-      let returnValue = await run(`${__dirname}/fixtures/${fixture}`, { log, color: false });
+      if (fixturesWithFix.includes(fixture)) {
+        process.argv.push('--fix');
+      }
+
+      let returnValue = await run(`${__dirname}/fixtures/${fixture}`, {
+        log,
+        color: false,
+        writeToFile,
+      });
 
       let expectedReturnValue = fixturesWithErrors.includes(fixture) ? 1 : 0;
 
       expect(returnValue).toEqual(expectedReturnValue);
       expect(output).toMatchSnapshot();
+      expect(writtenFiles).toMatchSnapshot();
+
+      process.argv = process.argv.filter(arg => arg !== '--fix');
     });
   }
 });