fix: generate properties at runtime

Author: cdoublev

.eslintignore

@@ -1,4 +1,2 @@
 node_modules
-lib/implementedProperties.js
-lib/properties.js
 jest.config.js

.eslintrc.js

@@ -33,12 +33,6 @@ module.exports = {
     strict: ['warn', 'global'],
   },
   overrides: [
-    {
-      files: ['lib/implementedProperties.js', 'lib/properties.js'],
-      rules: {
-        'prettier/prettier': 'off',
-      },
-    },
     {
       files: 'scripts/**/*',
       rules: {

.gitignore

@@ -1,5 +1,3 @@
 node_modules
 npm-debug.log
-lib/implementedProperties.js
-lib/properties.js
 coverage

lib/CSSStyleDeclaration.js

@@ -6,8 +6,9 @@
 var CSSOM = require('cssom');
 var allProperties = require('./allProperties');
 var allExtraProperties = require('./allExtraProperties');
-var implementedProperties = require('./implementedProperties');
-var { dashedToCamelCase } = require('./parsers');
+const { camelToDashed, dashedToCamelCase } = require('./parsers');
+const fs = require('fs');
+const path = require('path');
 var getBasicPropertyDescriptor = require('./utils/getBasicPropertyDescriptor');
 
 /**
@@ -239,7 +240,18 @@ Object.defineProperties(CSSStyleDeclaration.prototype, {
   },
 });
 
-require('./properties')(CSSStyleDeclaration.prototype);
+const implementedProperties = fs
+  .readdirSync(path.resolve(__dirname, 'properties'))
+  .reduce((props, filename) => {
+    const { definition } = require(`./properties/${filename}`);
+    const { name: camelCaseName } = path.parse(filename);
+    const dashedCaseName = camelToDashed(camelCaseName);
+    Object.defineProperty(CSSStyleDeclaration.prototype, camelCaseName, definition);
+    Object.defineProperty(CSSStyleDeclaration.prototype, dashedCaseName, definition);
+    return props.add(dashedCaseName);
+  }, new Set());
+
+module.exports.implementedProperties = implementedProperties;
 
 allProperties.forEach(function(property) {
   if (!implementedProperties.has(property)) {

lib/CSSStyleDeclaration.test.js

@@ -1,16 +1,16 @@
 'use strict';
 
-var { CSSStyleDeclaration } = require('./CSSStyleDeclaration');
+const { CSSStyleDeclaration, implementedProperties } = require('./CSSStyleDeclaration');
 
 var allProperties = require('./allProperties');
 var allExtraProperties = require('./allExtraProperties');
-var implementedProperties = require('./implementedProperties');
 var parsers = require('./parsers');
 
 var dashedProperties = [...allProperties, ...allExtraProperties];
 var allowedProperties = dashedProperties.map(parsers.dashedToCamelCase);
-implementedProperties = Array.from(implementedProperties).map(parsers.dashedToCamelCase);
-var invalidProperties = implementedProperties.filter(prop => !allowedProperties.includes(prop));
+const invalidProperties = Array.from(implementedProperties).filter(
+  prop => !allowedProperties.includes(parsers.dashedToCamelCase(prop))
+);
 
 describe('CSSStyleDeclaration', () => {
   test('has only valid properties implemented', () => {