Remove hardcoded transform from float to cssFloat in objects (#172)

The value should only be transformed if the raw value was also
"cssFloat". Otherwise, we should allow the user to use the "float"
keyword in the cases it is actually valid, which we can't easily detect.

Misuse of the keyword can be caught by Javascript code analysis tools.

Fixes: stylelint/stylelint#4490

Author: rbong

object-stringifier.js

@@ -26,10 +26,6 @@ class ObjectStringifier extends Stringifier {
 	decl(node, semicolon) {
 		let prop = this.rawValue(node, 'prop');
 
-		if (prop === 'float') {
-			prop = 'cssFloat';
-		}
-
 		let string = prop;
 
 		const isObjectShorthand = node.raws.node && node.raws.node.shorthand;

test/css-in-js.js

@@ -128,7 +128,7 @@ describe('CSS in JS', () => {
 		`);
 	});
 
-	it('float', () => {
+	it('cssFloat', () => {
 		const code = `
 			import glm from 'glamorous';
 			const Component1 = glm.a({
@@ -148,21 +148,24 @@ describe('CSS in JS', () => {
 				cssFloat: "left",
 			});
 		`);
+	});
 
-		root.first.first.nodes = [
-			postcss.decl({
-				prop: 'float',
-				value: 'right',
-				raws: {
-					before: root.first.first.first.raws.before,
-				},
-			}),
-		];
+	it('float', () => {
+		const code = `
+			const component = styled(Btn)({
+				float: "left",
+			});
+		`;
+
+		const root = syntax.parse(code, {
+			from: '/fixtures/styled-float.jsx',
+		});
+
+		expect(root.first.first.first).toHaveProperty('prop', 'float');
 
 		expect(root.toString()).toBe(`
-			import glm from 'glamorous';
-			const Component1 = glm.a({
-				cssFloat: "right",
+			const component = styled(Btn)({
+				float: "left",
 			});
 		`);
 	});