From 873ebc805580ecc54cbe3479a91160a8bee70886 Mon Sep 17 00:00:00 2001
From: Leon Xu <xleo@amazon.com>
Date: Wed, 26 Jun 2024 10:42:47 -0700
Subject: [PATCH] Normalize newlines in public-api-test (#45145)

Summary:
Tests were failing on windows due to parsing CRLF line endings. This change enables the API tests for windows by normalizing line endings before parsing the file.

## Changelog:
[INTERNAL] [ADDED] - `public-api-test` now runs on Windows.

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests

Pull Request resolved: https://github.com/facebook/react-native/pull/45145

Test Plan: Build-time-only change; relying on CircleCI

Reviewed By: cipolleschi

Differential Revision: D59001867

Pulled By: huntie

fbshipit-source-id: a7a41945e8c93288be1d5b7b59df7f621e467657
---
 .../Libraries/__tests__/public-api-test.js             | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/packages/react-native/Libraries/__tests__/public-api-test.js b/packages/react-native/Libraries/__tests__/public-api-test.js
index 47f4496228e..88a2ab81059 100644
--- a/packages/react-native/Libraries/__tests__/public-api-test.js
+++ b/packages/react-native/Libraries/__tests__/public-api-test.js
@@ -66,14 +66,6 @@ const sourceFiles = [
 ];
 
 describe('public API', () => {
-  if (os.platform() === 'win32') {
-    // TODO(huntie): Re-enable once upstream flow-api-translator fixes are made
-    // eslint-disable-next-line jest/no-focused-tests
-    test.only('skipping tests on win32', () => {
-      console.log('skipping tests');
-    });
-  }
-
   describe('should not change unintentionally', () => {
     test.each(sourceFiles)('%s', async (file: string) => {
       const source = await fs.readFile(path.join(PACKAGE_ROOT, file), 'utf-8');
@@ -111,6 +103,8 @@ describe('public API', () => {
 });
 
 async function translateFlowToExportedAPI(source: string): Promise<string> {
+  // Normalize newlines
+  source = source.replace(/\r\n?/g, '\n');
   // Convert to Flow typedefs
   const typeDefSource = await translate.translateFlowToFlowDef(source);