use a simpler hashing algorithm when a file system is available

Author: quantizor

src/visitors/displayNameAndId.js

@@ -76,7 +76,15 @@ const getFileHash = (state) => {
   const moduleName = moduleRoot && JSON.parse(fs.readFileSync(path.join(moduleRoot, 'package.json'))).name
   const code = file.code
 
-  const fileHash = hash([moduleName, filePath, code].join(''))
+  const stuffToHash = [moduleName]
+
+  if (filePath) {
+    stuffToHash.push(filePath)
+  } else {
+    stuffToHash.push(code)
+  }
+
+  const fileHash = hash(stuffToHash.join(''))
   file.set(FILE_HASH, fileHash)
   return fileHash
 }

test/__snapshots__/index.test.js.snap

@@ -24,49 +24,49 @@ const WrappedComponent = styled(Inner).withConfig({
 
 exports[`fixtures should add identifier 1`] = `
 "const Test = styled.div.withConfig({
-  componentId: \"f15zqz-0\"
+  componentId: \"iiazu4-0\"
 })\`width:100%;\`;
 const Test2 = true ? styled.div.withConfig({
-  componentId: \"f15zqz-1\"
+  componentId: \"iiazu4-1\"
 })\`\` : styled.div.withConfig({
-  componentId: \"f15zqz-2\"
+  componentId: \"iiazu4-2\"
 })\`\`;
 const styles = { One: styled.div.withConfig({
-    componentId: \"f15zqz-3\"
+    componentId: \"iiazu4-3\"
   })\`\` };
 let Component;
 Component = styled.div.withConfig({
-  componentId: \"f15zqz-4\"
+  componentId: \"iiazu4-4\"
 })\`\`;
 const WrappedComponent = styled(Inner).withConfig({
-  componentId: \"f15zqz-5\"
+  componentId: \"iiazu4-5\"
 })\`\`;"
 `;
 
 exports[`fixtures should add identifier and display name 1`] = `
 "const Test = styled.div.withConfig({
   displayName: \"Test\",
-  componentId: \"s1j54k93-0\"
+  componentId: \"s1vkcy8r-0\"
 })\`width:100%;\`;
 const Test2 = true ? styled.div.withConfig({
   displayName: \"Test2\",
-  componentId: \"s1j54k93-1\"
+  componentId: \"s1vkcy8r-1\"
 })\`\` : styled.div.withConfig({
   displayName: \"Test2\",
-  componentId: \"s1j54k93-2\"
+  componentId: \"s1vkcy8r-2\"
 })\`\`;
 const styles = { One: styled.div.withConfig({
     displayName: \"One\",
-    componentId: \"s1j54k93-3\"
+    componentId: \"s1vkcy8r-3\"
   })\`\` };
 let Component;
 Component = styled.div.withConfig({
   displayName: \"Component\",
-  componentId: \"s1j54k93-4\"
+  componentId: \"s1vkcy8r-4\"
 })\`\`;
 const WrappedComponent = styled(Inner).withConfig({
   displayName: \"WrappedComponent\",
-  componentId: \"s1j54k93-5\"
+  componentId: \"s1vkcy8r-5\"
 })\`\`;"
 `;
 
@@ -198,27 +198,27 @@ exports[`fixtures should track the imported variable 1`] = `
 
 const Test = s.div.withConfig({
   displayName: \"Test\",
-  componentId: \"syqrfc-0\"
+  componentId: \"s13i891w-0\"
 })\`width:100%;\`;
 const Test2 = true ? s.div.withConfig({
   displayName: \"Test2\",
-  componentId: \"syqrfc-1\"
+  componentId: \"s13i891w-1\"
 })\`\` : s.div.withConfig({
   displayName: \"Test2\",
-  componentId: \"syqrfc-2\"
+  componentId: \"s13i891w-2\"
 })\`\`;
 const styles = { One: s.div.withConfig({
     displayName: \"One\",
-    componentId: \"syqrfc-3\"
+    componentId: \"s13i891w-3\"
   })\`\` };
 let Component;
 Component = s.div.withConfig({
   displayName: \"Component\",
-  componentId: \"syqrfc-4\"
+  componentId: \"s13i891w-4\"
 })\`\`;
 const WrappedComponent = s(Inner).withConfig({
   displayName: \"WrappedComponent\",
-  componentId: \"syqrfc-5\"
+  componentId: \"s13i891w-5\"
 })\`\`;"
 `;
 
@@ -227,17 +227,17 @@ exports[`fixtures should transpile require default 1`] = `
 
 const TestNormal = styled.div.withConfig({
   displayName: \"before__TestNormal\",
-  componentId: \"y0c69d-0\"
+  componentId: \"iqyhdm-0\"
 })([[\"{width:100%;}\"]]);
 
 const Test = styled_default.default.div.withConfig({
   displayName: \"before__Test\",
-  componentId: \"y0c69d-1\"
+  componentId: \"iqyhdm-1\"
 })([[\"{width:100%;}\"]]);
 
 const TestCallExpression = styled_default.default(Test).withConfig({
   displayName: \"before__TestCallExpression\",
-  componentId: \"y0c69d-2\"
+  componentId: \"iqyhdm-2\"
 })([[\"{height:20px;}\"]]);"
 `;
 
@@ -288,26 +288,26 @@ exports[`fixtures should use file name 1`] = `
 
 const Test = styled.div.withConfig({
   displayName: \"before__Test\",
-  componentId: \"s1x8c0g0-0\"
+  componentId: \"s1tyeg9p-0\"
 })\`color:red;\`;
 const before = styled.div.withConfig({
   displayName: \"before\",
-  componentId: \"s1x8c0g0-1\"
+  componentId: \"s1tyeg9p-1\"
 })\`color:blue;\`;
 styled.div.withConfig({
   displayName: \"before\",
-  componentId: \"s1x8c0g0-2\"
+  componentId: \"s1tyeg9p-2\"
 })\`\`;
 export default styled.button.withConfig({
   displayName: \"before\",
-  componentId: \"s1x8c0g0-3\"
+  componentId: \"s1tyeg9p-3\"
 })\`\`;"
 `;
 
 exports[`fixtures should work with hoisted default as import 1`] = `
 "const Test = s.div.withConfig({
   displayName: \'before__Test\',
-  componentId: \'s2wuo7o-0\'
+  componentId: \'s1vk20b3-0\'
 })\`width:100%;\`;
 import { default as s, css } from \'styled-components\';"
 `;