Revert change enforcing color to be defined for text/box shadows

Author: jacobp100

src/index.test.js

@@ -489,6 +489,14 @@ it('transforms box-shadow without blur-radius', () =>
     shadowOpacity: 1,
   }))
 
+it('transforms box-shadow without color', () =>
+  runTest([['box-shadow', '10px 20px']], {
+    shadowOffset: { width: 10, height: 20 },
+    shadowRadius: 0,
+    shadowColor: 'black',
+    shadowOpacity: 1,
+  }))
+
 it('transforms box-shadow with rgb color', () =>
   runTest([['box-shadow', '10px 20px rgb(100, 100, 100)']], {
     shadowOffset: { width: 10, height: 20 },
@@ -524,24 +532,12 @@ it('transforms box-shadow with hsla color', () =>
 it('transforms box-shadow and throws if multiple colors are used', () => {
   expect(() =>
     transformCss([['box-shadow', '0 0 0 red yellow green blue']])
-  ).toThrow(
-    'Failed to parse declaration "boxShadow: 0 0 0 red yellow green blue"'
-  )
+  ).toThrow()
 })
 
 it('transforms box-shadow enforces offset to be present', () => {
-  expect(() => transformCss([['box-shadow', 'red']])).toThrow(
-    'Failed to parse declaration "boxShadow: red"'
-  )
-})
-
-it('transforms box-shadow and enforces offset-x and offset-y', () => {
-  expect(() => transformCss([['box-shadow', 'black']])).toThrow()
-  expect(() => transformCss([['box-shadow', '10px black']])).toThrow()
-})
-
-it('transforms box-shadow and enforces color', () => {
-  expect(() => transformCss([['text-decoration', '10px 20px 30px']])).toThrow()
+  expect(() => transformCss([['box-shadow', 'red']])).toThrow()
+  expect(() => transformCss([['box-shadow', '10px red']])).toThrow()
 })
 
 it('transforms text-decoration into text-decoration- properties', () =>

src/transforms/__tests__/textShadow.js

@@ -16,6 +16,14 @@ it('textShadow omitting blur', () => {
   })
 })
 
+it('textShadow omitting color', () => {
+  expect(transformCss([['text-shadow', '10px 20px black']])).toEqual({
+    textShadowOffset: { width: 10, height: 20 },
+    textShadowRadius: 0,
+    textShadowColor: 'black',
+  })
+})
+
 it('textShadow omitting blur, offset-y', () => {
   expect(transformCss([['text-shadow', '10px 20px red']])).toEqual({
     textShadowOffset: { width: 10, height: 20 },
@@ -28,7 +36,3 @@ it('textShadow enforces offset-x and offset-y', () => {
   expect(() => transformCss([['text-shadow', 'red']])).toThrow()
   expect(() => transformCss([['text-shadow', '10px red']])).toThrow()
 })
-
-it('textShadow enforces color', () => {
-  expect(() => transformCss([['text-shadow', '10px 20px']])).toThrow()
-})

src/transforms/util.js

@@ -80,7 +80,7 @@ module.exports.shadowOffsetFactory = () => tokenStream => {
 module.exports.parseShadow = tokenStream => {
   let offsetX
   let offsetY
-  let radius = 0
+  let radius
   let color
 
   if (tokenStream.matches(NONE)) {
@@ -116,11 +116,11 @@ module.exports.parseShadow = tokenStream => {
     didParseFirst = true
   }
 
-  if (offsetX === undefined || color === undefined) tokenStream.throw()
+  if (offsetX === undefined) tokenStream.throw()
 
   return {
     offset: { width: offsetX, height: offsetY },
-    radius,
-    color,
+    radius: radius !== undefined ? radius : 0,
+    color: color !== undefined ? color : 'black',
   }
 }