merge upstream

Author: SjorsO

.eslintignore

@@ -1,4 +1,4 @@
 /lib
 /docs
 /__tests__/fixtures/cli-utils.js
-defaultConfig.stub.js
+/stubs/*

.eslintrc

@@ -3,11 +3,8 @@
     "jest": true
   },
   "parserOptions": {
-    "ecmaVersion": 6,
-    "sourceType": "module",
-    "ecmaFeatures": {
-      "experimentalObjectRestSpread": true
-    }
+    "ecmaVersion": 2018,
+    "sourceType": "module"
   },
   "extends": ["eslint-config-postcss", "prettier"],
   "plugins": ["prettier"],

.gitignore

@@ -1,4 +1,6 @@
 /node_modules
 /lib
 /example
+index.html
+package-lock.json
 yarn-error.log

.npmignore

@@ -1,4 +1,5 @@
 /__tests__/
 /jest/
 /src/
+index.html
 yarn-error.log

README.md

@@ -24,7 +24,7 @@ For help, discussion about best practices, or any other conversation that would
 
 For casual chit-chat with others using the framework:
 
-[Join the Tailwind CSS Slack Community](https://join.slack.com/t/tailwindcss/shared_invite/enQtNDQ1MDYyNDA0NzA3LTAzOGEzYTRmMjE2OWUwMGViMGM2NGM5OWVmN2UzZjlmNzQ0ZTA2NGUwODYyOWMzNzM0M2MzMmE1NGYyNjk5NTI)
+[Join the Tailwind CSS Discord Server](https://discord.gg/7NF8GNe)
 
 ## Contributing
 

__tests__/applyAtRule.test.js

@@ -1,12 +1,18 @@
 import postcss from 'postcss'
 import substituteClassApplyAtRules from '../src/lib/substituteClassApplyAtRules'
 import processPlugins from '../src/util/processPlugins'
+import resolveConfig from '../src/util/resolveConfig'
 import corePlugins from '../src/corePlugins'
-import defaultConfig from '../defaultConfig.stub.js'
+import defaultConfig from '../stubs/defaultConfig.stub.js'
 
-const { utilities: defaultUtilities } = processPlugins(corePlugins(defaultConfig), defaultConfig)
+const resolvedDefaultConfig = resolveConfig([defaultConfig])
 
-function run(input, config = defaultConfig, utilities = defaultUtilities) {
+const { utilities: defaultUtilities } = processPlugins(
+  corePlugins(resolvedDefaultConfig),
+  resolvedDefaultConfig
+)
+
+function run(input, config = resolvedDefaultConfig, utilities = defaultUtilities) {
   return postcss([substituteClassApplyAtRules(config, utilities)]).process(input, {
     from: undefined,
   })
@@ -200,10 +206,12 @@ test('you can apply utility classes without using the given prefix', () => {
     .foo { margin-top: 1rem; margin-bottom: 1rem; }
   `
 
-  const config = {
-    ...defaultConfig,
-    prefix: 'tw-',
-  }
+  const config = resolveConfig([
+    {
+      ...defaultConfig,
+      prefix: 'tw-',
+    },
+  ])
 
   return run(input, config, processPlugins(corePlugins(config), config).utilities).then(result => {
     expect(result.css).toEqual(expected)
@@ -220,12 +228,14 @@ test('you can apply utility classes without using the given prefix when using a
     .foo { margin-top: 1rem; margin-bottom: 1rem; }
   `
 
-  const config = {
-    ...defaultConfig,
-    prefix: () => {
-      return 'tw-'
+  const config = resolveConfig([
+    {
+      ...defaultConfig,
+      prefix: () => {
+        return 'tw-'
+      },
     },
-  }
+  ])
 
   return run(input, config, processPlugins(corePlugins(config), config).utilities).then(result => {
     expect(result.css).toEqual(expected)

__tests__/cli.compile.test.js

@@ -0,0 +1,19 @@
+import path from 'path'
+
+import autoprefixer from 'autoprefixer'
+
+import tailwind from '../src'
+import compile from '../src/cli/compile'
+
+describe('cli compile', () => {
+  const inputFile = path.resolve(__dirname, 'fixtures/tailwind-input.css')
+  const outputFile = 'output.css'
+  const plugins = [tailwind(), autoprefixer]
+
+  it('compiles CSS file', () => {
+    return compile({ inputFile, outputFile, plugins }).then(result => {
+      expect(result.css).toContain('.example')
+      expect(result.css).toContain('-ms-input-placeholder')
+    })
+  })
+})

__tests__/cli.test.js

@@ -1,36 +1,43 @@
 import path from 'path'
 
 import cli from '../src/cli/main'
-import * as constants from '../src/cli/constants'
+import * as constants from '../src/constants'
 import * as utils from '../src/cli/utils'
+import runInTempDirectory from '../jest/runInTempDirectory'
 
 describe('cli', () => {
   const inputCssPath = path.resolve(__dirname, 'fixtures/tailwind-input.css')
   const customConfigPath = path.resolve(__dirname, 'fixtures/custom-config.js')
+  const defaultConfigFixture = utils.readFile(constants.defaultConfigStubFile)
+  const simpleConfigFixture = utils.readFile(constants.simpleConfigStubFile)
 
   beforeEach(() => {
     console.log = jest.fn()
     process.stdout.write = jest.fn()
-    utils.writeFile = jest.fn()
   })
 
   describe('init', () => {
     it('creates a Tailwind config file', () => {
-      return cli(['init']).then(() => {
-        expect(utils.writeFile.mock.calls[0][0]).toEqual(constants.defaultConfigFile)
+      return runInTempDirectory(() => {
+        return cli(['init']).then(() => {
+          expect(utils.readFile(constants.defaultConfigFile)).toEqual(simpleConfigFixture)
+        })
       })
     })
 
-    it('creates a Tailwind config file in a custom location', () => {
-      return cli(['init', 'custom.js']).then(() => {
-        expect(utils.writeFile.mock.calls[0][0]).toEqual('custom.js')
+    it('creates a full Tailwind config file', () => {
+      return runInTempDirectory(() => {
+        return cli(['init', '--full']).then(() => {
+          expect(utils.readFile(constants.defaultConfigFile)).toEqual(defaultConfigFixture)
+        })
       })
     })
 
-    it('creates a Tailwind config file without comments', () => {
-      return cli(['init', '--no-comments']).then(() => {
-        expect(utils.writeFile.mock.calls[0][1]).not.toContain('/**')
-        expect(utils.writeFile.mock.calls[0][1]).toContain('//')
+    it('creates a Tailwind config file in a custom location', () => {
+      return runInTempDirectory(() => {
+        return cli(['init', 'custom.js']).then(() => {
+          expect(utils.exists('custom.js')).toEqual(true)
+        })
       })
     })
   })
@@ -49,9 +56,10 @@ describe('cli', () => {
     })
 
     it('creates compiled CSS file', () => {
-      return cli(['build', inputCssPath, '--output', 'output.css']).then(() => {
-        expect(utils.writeFile.mock.calls[0][0]).toEqual('output.css')
-        expect(utils.writeFile.mock.calls[0][1]).toContain('.example')
+      return runInTempDirectory(() => {
+        return cli(['build', inputCssPath, '--output', 'output.css']).then(() => {
+          expect(utils.readFile('output.css')).toContain('.example')
+        })
       })
     })
 

__tests__/cli.utils.test.js

@@ -1,9 +1,6 @@
-import path from 'path'
 import * as utils from '../src/cli/utils'
 
 describe('cli utils', () => {
-  const fixture = utils.readFile(path.resolve(__dirname, 'fixtures/cli-utils.js'))
-
   describe('parseCliParams', () => {
     it('parses CLI parameters', () => {
       const result = utils.parseCliParams(['a', 'b', '-c', 'd'])
@@ -60,52 +57,17 @@ describe('cli utils', () => {
     })
   })
 
-  describe('stripBlockComments', () => {
-    it('does not strip code', () => {
-      const result = utils.stripBlockComments(fixture)
-
-      expect(result).toEqual(expect.stringContaining('__code_no_comment__'))
-      expect(result).toEqual(expect.stringContaining('__code_comment_line__'))
-      expect(result).toEqual(expect.stringContaining('__code_comment_block__'))
-      expect(result).toEqual(expect.stringContaining('__code_comment_line_important__'))
-      expect(result).toEqual(expect.stringContaining('__code_comment_block_important__'))
-    })
-
-    it('strips block comments', () => {
-      const result = utils.stripBlockComments(fixture)
-
-      expect(result).not.toEqual(expect.stringContaining('__comment_block__'))
-      expect(result).not.toEqual(expect.stringContaining('__comment_block_multiline__'))
-      expect(result).not.toEqual(expect.stringContaining('__comment_block_code__'))
-    })
-
-    it('strips docblock comments', () => {
-      const result = utils.stripBlockComments(fixture)
-
-      expect(result).not.toEqual(expect.stringContaining('__comment_docblock__'))
-    })
-
-    it('does not strip line comments', () => {
-      const result = utils.stripBlockComments(fixture)
-
-      expect(result).toEqual(expect.stringContaining('__comment_line__'))
-      expect(result).toEqual(expect.stringContaining('__comment_line_important__'))
-      expect(result).toEqual(expect.stringContaining('__comment_line_code__'))
-      expect(result).toEqual(expect.stringContaining('__comment_line_important_code__'))
-    })
-
-    it('does not strip important block comments', () => {
-      const result = utils.stripBlockComments(fixture)
+  describe('getSimplePath', () => {
+    it('strips leading ./', () => {
+      const result = utils.getSimplePath('./test')
 
-      expect(result).toEqual(expect.stringContaining('__comment_block_important__'))
-      expect(result).toEqual(expect.stringContaining('__comment_block_multiline_important__'))
-      expect(result).toEqual(expect.stringContaining('__comment_block_important_code__'))
+      expect(result).toEqual('test')
     })
 
-    it('does not strip important docblock comments', () => {
-      const result = utils.stripBlockComments(fixture)
+    it('returns unchanged path if it does not begin with ./', () => {
+      const result = utils.getSimplePath('../test')
 
-      expect(result).toEqual(expect.stringContaining('__comment_docblock_important__'))
+      expect(result).toEqual('../test')
     })
   })
 })