chore: migrate jest-serializer to TypeScript

[thymikee]

Summary

Jumping on a TS train 🚂

Details

diff --git a/packages/jest-serializer/build/index.js b/packages/jest-serializer/build/index.js
index b6a60150d..1c0d8cdb7 100644
--- a/packages/jest-serializer/build/index.js
+++ b/packages/jest-serializer/build/index.js
@@ -3,8 +3,6 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- *
  */
 'use strict';

@@ -49,7 +47,7 @@ const JS_TYPE = '__$t__';
 const JS_VALUE = '__$v__';
 const JS_VF = '__$f__';

-function replacer(key, value) {
+function replacer(_key, value) {
   // NaN cannot be in a switch statement, because NaN !== NaN.
   if (Number.isNaN(value)) {
     return {
@@ -110,7 +108,7 @@ function replacer(key, value) {
   return value;
 }

-function reviver(key, value) {
+function reviver(_key, value) {
   if (!value || (typeof value !== 'object' && !value.hasOwnProperty(JS_TYPE))) {
     return value;
   }
@@ -157,15 +155,13 @@ function jsonStringify(content) {
   /* eslint-disable no-extend-native */

   try {
-    // $FlowFixMe: intentional removal of "toJSON" property.
-    Date.prototype.toJSON = undefined; // $FlowFixMe: intentional removal of "toJSON" property.
+    // @ts-ignore intentional removal of "toJSON" property.
+    Date.prototype.toJSON = undefined; // @ts-ignore intentional removal of "toJSON" property.

     Buffer.prototype.toJSON = undefined;
     return JSON.stringify(content, replacer);
   } finally {
-    // $FlowFixMe: intentional assignment of "toJSON" property.
-    Date.prototype.toJSON = dateToJSON; // $FlowFixMe: intentional assignment of "toJSON" property.
-
+    Date.prototype.toJSON = dateToJSON;
     Buffer.prototype.toJSON = bufferToJSON;
   }
   /* eslint-enable no-extend-native */
@@ -176,28 +172,24 @@ function jsonParse(content) {
 } // In memory functions.

 function deserialize(buffer) {
-  // $FlowFixMe - Node 8+ only
   return _v().default.deserialize
     ? _v().default.deserialize(buffer)
     : jsonParse(buffer.toString('utf8'));
 }

 function serialize(content) {
-  // $FlowFixMe - Node 8+ only
   return _v().default.serialize
     ? _v().default.serialize(content)
     : Buffer.from(jsonStringify(content));
 } // Synchronous filesystem functions.

 function readFileSync(filePath) {
-  // $FlowFixMe - Node 8+ only
   return _v().default.deserialize
     ? _v().default.deserialize(_fs().default.readFileSync(filePath))
     : jsonParse(_fs().default.readFileSync(filePath, 'utf8'));
 }

 function writeFileSync(filePath, content) {
-  // $FlowFixMe - Node 8+ only
   return _v().default.serialize
     ? _fs().default.writeFileSync(filePath, _v().default.serialize(content))
     : _fs().default.writeFileSync(filePath, jsonStringify(content), 'utf8');

Test plan

Green CI

[SimenB]

🎉

[SimenB]

Changelog?

[thymikee]

Good call!

[milesj]

Instead of disabling all those node 8 checks, you should mybe patch the node types.

declare namespace NodeJS {
  interface v8 {
    // ...
  }
}

[thymikee]

Thanks! Added global.d.ts, hope it's ok? I'm new to TS :P

[codecov-io]

Codecov Report

Merging #7841 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7841      +/-   ##
==========================================
- Coverage    64.8%   64.63%   -0.18%     
==========================================
  Files         230      229       -1     
  Lines        8657     8615      -42     
  Branches        5        5              
==========================================
- Hits         5610     5568      -42     
  Misses       3045     3045              
  Partials        2        2

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d96ff2c...3123acf. Read the comment docs.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.