JavaScriptExpert
diff --git a/‎core-build/gulp-core-build-sass/src/CSSModules.ts‎
Lines changed: 8 additions & 0 deletions b/‎core-build/gulp-core-build-sass/src/CSSModules.ts‎
Lines changed: 8 additions & 0 deletions
diff --git a/‎core-build/gulp-core-build-sass/src/SassTask.ts‎
Lines changed: 37 additions & 37 deletions b/‎core-build/gulp-core-build-sass/src/SassTask.ts‎
Lines changed: 37 additions & 37 deletions
@@ -5,7 +5,15 @@ import * as cssModules from 'postcss-modules';
 import * as crypto from 'crypto';
 
 export interface ICSSModules {
+  /**
+   * Return a configured postcss plugin that will map class names to a
+   * consistently generated scoped name.
+   */
   getPlugin: () => postcss.AcceptedPlugin;
+
+  /**
+   * Return the CSS class map that is stored after postcss-modules runs.
+   */
   getCssJSON: () => Object;
 }
 
 
@@ -212,56 +212,22 @@ export class SassTask extends GulpTask<ISassTaskConfig> {
 
       const scssTsOutputPath: string = `${filePath}.ts`;
       const classMap: Object = cssModules.getCssJSON();
-      let exportClassNames: string = '';
+      const stylesExportString: string = this._getStylesExportString(classMap);
       const content: string | undefined = result.styles;
 
-      if (classMap) {
-        const classKeys: string[] = Object.keys(classMap);
-        const styleLines: string[] = [];
-        classKeys.forEach((key: string) => {
-          const value: string = classMap[key];
-          if (key.indexOf('-') !== -1) {
-            const message: string = `The local CSS class '${key}' is not ` +
-                                    `camelCase and will not be type-safe.`;
-            if (this.taskConfig.warnOnCssInvalidPropertyName) {
-              this.logWarning(message);
-            } else {
-              this.logVerbose(message);
-            }
-          }
-          styleLines.push(`  ${key}: '${value}'`);
-        });
-
-        let exportString: string = 'export default styles;';
-
-        if (this.taskConfig.moduleExportName === '') {
-          exportString = 'export = styles;';
-        } else if (!!this.taskConfig.moduleExportName) {
-          // exportString = `export const ${this.taskConfig.moduleExportName} = styles;`;
-        }
-
-        exportClassNames = [
-          'const styles = {',
-          styleLines.join(`,${EOL}`),
-          '};',
-          '',
-          exportString
-        ].join(EOL);
-      }
-
       let lines: string[] = [];
       lines.push(this.taskConfig.preamble || '');
 
       if (cssOutputPathAbsolute) {
         lines = lines.concat([
           `require(${JSON.stringify(`./${path.basename(cssOutputPathAbsolute)}`)});`,
-          exportClassNames
+          stylesExportString
         ]);
       } else if (!!content) {
         lines = lines.concat([
           'import { loadStyles } from \'@microsoft/load-themed-styles\';',
           '',
-          exportClassNames,
+          stylesExportString,
           '',
           `loadStyles(${JSON.stringify(splitStyles(content))});`
         ]);
@@ -308,4 +274,38 @@ export class SassTask extends GulpTask<ISassTaskConfig> {
 
     return url;
   }
+
+  private _getStylesExportString(classMap: Object): string {
+    const classKeys: string[] = Object.keys(classMap);
+    const styleLines: string[] = [];
+    classKeys.forEach((key: string) => {
+      const value: string = classMap[key];
+      if (key.indexOf('-') !== -1) {
+        const message: string = `The local CSS class '${key}' is not ` +
+                                `camelCase and will not be type-safe.`;
+        if (this.taskConfig.warnOnCssInvalidPropertyName) {
+          this.logWarning(message);
+        } else {
+          this.logVerbose(message);
+        }
+      }
+      styleLines.push(`  ${key}: '${value}'`);
+    });
+
+    let exportString: string = 'export default styles;';
+
+    if (this.taskConfig.moduleExportName === '') {
+      exportString = 'export = styles;';
+    } else if (!!this.taskConfig.moduleExportName) {
+      // exportString = `export const ${this.taskConfig.moduleExportName} = styles;`;
+    }
+
+    return [
+      'const styles = {',
+      styleLines.join(`,${EOL}`),
+      '};',
+      '',
+      exportString
+    ].join(EOL);
+  }
 }