Updating Tests to latest Prettier tests (#448)

* using defineProperty() instead of directly assigning run_spec to global

* Using the latest run_spec for testing

* fix call to run_spec and new snapshots

* support for prettier/standalone in tests

* running prettier

* only collect coverage in FULL_TEST

* cleaning up the scripts in package.json

* adding testing to the README

* .eslintignore and .prettierignore take care of the ignored folders now

* Avoid using !!

Author: Janther

.eslintignore

@@ -0,0 +1,2 @@
+tests_config/**/*.js
+src/prettier-comments/**/*.js

.prettierignore

@@ -1 +1,3 @@
 tests/**/*.sol
+tests_config/**/*.js
+src/prettier-comments/**/*.js

README.md

@@ -152,8 +152,9 @@ If you want a different configuration for your javascript and solidity files, yo
 1. [Fork it](https://github.com/prettier-solidity/prettier-plugin-solidity/fork)
 2. Create your feature branch (`git checkout -b feature/fooBar`)
 3. Commit your changes (`git commit -am 'Add some fooBar'`)
-4. Push to the branch (`git push origin feature/fooBar`)
-5. Create a new Pull Request
+4. All existing test and coverage must pass (`npm run test:all`), if coverage drops below 100% add missing tests.
+5. Push to the branch (`git push origin feature/fooBar`)
+6. Create a new Pull Request
 
 ## License
 

jest.config.js

@@ -1,5 +1,7 @@
+const FULL_TEST = Boolean(process.env.FULL_TEST);
+
 module.exports = {
-  collectCoverage: true,
+  collectCoverage: FULL_TEST,
   collectCoverageFrom: [
     'scripts/**/*.js',
     '!scripts/generateIndexes.js',
@@ -16,8 +18,11 @@ module.exports = {
       statements: 100
     }
   },
-  setupFiles: ['<rootDir>/tests_config/run_spec.js'],
-  snapshotSerializers: ['<rootDir>/tests_config/raw-serializer.js'],
+  setupFiles: ['<rootDir>/tests_config/setup.js'],
+  snapshotSerializers: [
+    'jest-snapshot-serializer-raw',
+    'jest-snapshot-serializer-ansi'
+  ],
   testEnvironment: 'node',
   testRegex: 'jsfmt\\.spec\\.js$|__tests__/.*\\.js$|scripts/.*\\.test\\.js$',
   transform: {},