fix: change diff links for upgrade command

Author: thymikee

packages/cli/src/commands/upgrade/__tests__/upgrade.test.js

@@ -1,113 +1,113 @@
 // @flow
-import execa from 'execa';
-import path from 'path';
-import fs from 'fs';
-import snapshotDiff from 'snapshot-diff';
-import upgrade from '../upgrade';
-import {fetch} from '../helpers';
-import logger from '../../../tools/logger';
+import execa from "execa";
+import path from "path";
+import fs from "fs";
+import snapshotDiff from "snapshot-diff";
+import upgrade from "../upgrade";
+import { fetch } from "../helpers";
+import logger from "../../../tools/logger";
 
-jest.mock('https');
-jest.mock('fs');
-jest.mock('path');
-jest.mock('execa', () => {
+jest.mock("https");
+jest.mock("fs");
+jest.mock("path");
+jest.mock("execa", () => {
   const module = jest.fn((command, args) => {
-    mockPushLog('$', 'execa', command, args);
-    if (command === 'npm' && args[3] === '--json') {
+    mockPushLog("$", "execa", command, args);
+    if (command === "npm" && args[3] === "--json") {
       return Promise.resolve({
-        stdout: '{"react": "16.6.3"}',
+        stdout: '{"react": "16.6.3"}'
       });
     }
-    return Promise.resolve({stdout: ''});
+    return Promise.resolve({ stdout: "" });
   });
   return module;
 });
 jest.mock(
-  '/project/root/node_modules/react-native/package.json',
-  () => ({name: 'react-native', version: '0.57.8'}),
-  {virtual: true},
+  "/project/root/node_modules/react-native/package.json",
+  () => ({ name: "react-native", version: "0.57.8" }),
+  { virtual: true }
 );
 jest.mock(
-  '/project/root/package.json',
-  () => ({name: 'TestApp', dependencies: {'react-native': '^0.57.8'}}),
-  {virtual: true},
+  "/project/root/package.json",
+  () => ({ name: "TestApp", dependencies: { "react-native": "^0.57.8" } }),
+  { virtual: true }
 );
-jest.mock('../../../tools/PackageManager', () =>
+jest.mock("../../../tools/PackageManager", () =>
   jest.fn(() => ({
     install: args => {
-      mockPushLog('$ yarn add', ...args);
-    },
-  })),
+      mockPushLog("$ yarn add", ...args);
+    }
+  }))
 );
-jest.mock('../helpers', () => ({
-  ...jest.requireActual('../helpers'),
-  fetch: jest.fn(() => Promise.resolve('patch')),
+jest.mock("../helpers", () => ({
+  ...jest.requireActual("../helpers"),
+  fetch: jest.fn(() => Promise.resolve("patch"))
 }));
-jest.mock('../../../tools/logger', () => ({
-  info: jest.fn((...args) => mockPushLog('info', args)),
-  error: jest.fn((...args) => mockPushLog('error', args)),
-  warn: jest.fn((...args) => mockPushLog('warn', args)),
-  success: jest.fn((...args) => mockPushLog('success', args)),
-  log: jest.fn((...args) => mockPushLog(args)),
+jest.mock("../../../tools/logger", () => ({
+  info: jest.fn((...args) => mockPushLog("info", args)),
+  error: jest.fn((...args) => mockPushLog("error", args)),
+  warn: jest.fn((...args) => mockPushLog("warn", args)),
+  success: jest.fn((...args) => mockPushLog("success", args)),
+  log: jest.fn((...args) => mockPushLog(args))
 }));
 
-const currentVersion = '0.57.8';
-const newVersion = '0.58.4';
-const olderVersion = '0.56.0';
+const currentVersion = "0.57.8";
+const newVersion = "0.58.4";
+const olderVersion = "0.56.0";
 const ctx = {
-  root: '/project/root',
-  reactNativePath: '',
+  root: "/project/root",
+  reactNativePath: ""
 };
 const opts = {
-  legacy: false,
+  legacy: false
 };
 
 const samplePatch = jest
-  .requireActual('fs')
-  .readFileSync(path.join(__dirname, './sample.patch'), 'utf8');
+  .requireActual("fs")
+  .readFileSync(path.join(__dirname, "./sample.patch"), "utf8");
 
 let logs = [];
 const mockPushLog = (...args) =>
-  logs.push(args.map(x => (Array.isArray(x) ? x.join(' ') : x)).join(' '));
-const flushOutput = () => logs.join('\n');
+  logs.push(args.map(x => (Array.isArray(x) ? x.join(" ") : x)).join(" "));
+const flushOutput = () => logs.join("\n");
 
 beforeEach(() => {
   jest.clearAllMocks();
   // $FlowFixMe
-  fs.writeFileSync = jest.fn(filename => mockPushLog('[fs] write', filename));
+  fs.writeFileSync = jest.fn(filename => mockPushLog("[fs] write", filename));
   // $FlowFixMe
-  fs.unlinkSync = jest.fn((...args) => mockPushLog('[fs] unlink', args));
+  fs.unlinkSync = jest.fn((...args) => mockPushLog("[fs] unlink", args));
   logs = [];
 });
 
-test('uses latest version of react-native when none passed', async () => {
+test("uses latest version of react-native when none passed", async () => {
   await upgrade.func([], ctx, opts);
-  expect(execa).toBeCalledWith('npm', ['info', 'react-native', 'version']);
+  expect(execa).toBeCalledWith("npm", ["info", "react-native", "version"]);
 });
 
-test('errors when invalid version passed', async () => {
-  await upgrade.func(['next'], ctx, opts);
+test("errors when invalid version passed", async () => {
+  await upgrade.func(["next"], ctx, opts);
   expect(logger.error).toBeCalledWith(
-    'Provided version "next" is not allowed. Please pass a valid semver version',
+    'Provided version "next" is not allowed. Please pass a valid semver version'
   );
 });
 
-test('errors when older version passed', async () => {
+test("errors when older version passed", async () => {
   await upgrade.func([olderVersion], ctx, opts);
   expect(logger.error).toBeCalledWith(
-    `Trying to upgrade from newer version "${currentVersion}" to older "${olderVersion}"`,
+    `Trying to upgrade from newer version "${currentVersion}" to older "${olderVersion}"`
   );
 });
 
-test('warns when dependency upgrade version is in semver range', async () => {
+test("warns when dependency upgrade version is in semver range", async () => {
   await upgrade.func([currentVersion], ctx, opts);
   expect(logger.warn).toBeCalledWith(
-    `Specified version "${currentVersion}" is already installed in node_modules and it satisfies "^0.57.8" semver range. No need to upgrade`,
+    `Specified version "${currentVersion}" is already installed in node_modules and it satisfies "^0.57.8" semver range. No need to upgrade`
   );
 });
 
-test('fetches empty patch and installs deps', async () => {
-  (fetch: any).mockImplementation(() => Promise.resolve(''));
+test("fetches empty patch and installs deps", async () => {
+  (fetch: any).mockImplementation(() => Promise.resolve(""));
   await upgrade.func([newVersion], ctx, opts);
   expect(flushOutput()).toMatchInlineSnapshot(`
 "info Fetching diff between v0.57.8 and v0.58.4...
@@ -123,13 +123,13 @@ success Upgraded React Native to v0.58.4 🎉. Now you can review and commit the
 `);
 });
 
-test('fetches regular patch, adds remote, applies patch, installs deps, removes remote,', async () => {
+test("fetches regular patch, adds remote, applies patch, installs deps, removes remote,", async () => {
   (fetch: any).mockImplementation(() => Promise.resolve(samplePatch));
   await upgrade.func([newVersion], ctx, opts);
   expect(flushOutput()).toMatchInlineSnapshot(`
 "info Fetching diff between v0.57.8 and v0.58.4...
 [fs] write tmp-upgrade-rn.patch
-$ execa git remote add tmp-rn-diff-purge https://github.com/pvinis/rn-diff-purge.git
+$ execa git remote add tmp-rn-diff-purge https://github.com/react-native-community/rn-diff-purge.git
 $ execa git fetch --no-tags tmp-rn-diff-purge
 $ execa git apply --check tmp-upgrade-rn.patch --exclude=package.json -p2 --3way
 info Applying diff...
@@ -148,45 +148,45 @@ success Upgraded React Native to v0.58.4 🎉. Now you can review and commit the
 `);
   expect(
     snapshotDiff(samplePatch, fs.writeFileSync.mock.calls[0][1], {
-      contextLines: 1,
-    }),
-  ).toMatchSnapshot('RnDiffApp is replaced with app name (TestApp)');
+      contextLines: 1
+    })
+  ).toMatchSnapshot("RnDiffApp is replaced with app name (TestApp)");
 });
-test('cleans up if patching fails,', async () => {
+test("cleans up if patching fails,", async () => {
   (fetch: any).mockImplementation(() => Promise.resolve(samplePatch));
   (execa: any).mockImplementation((command, args) => {
-    mockPushLog('$', 'execa', command, args);
-    if (command === 'npm' && args[3] === '--json') {
+    mockPushLog("$", "execa", command, args);
+    if (command === "npm" && args[3] === "--json") {
       return Promise.resolve({
-        stdout: '{"react": "16.6.3"}',
+        stdout: '{"react": "16.6.3"}'
       });
     }
-    if (command === 'git' && args[0] === 'apply') {
+    if (command === "git" && args[0] === "apply") {
       return Promise.reject({
         code: 1,
-        stderr: 'error: .flowconfig: does not exist in index\n',
+        stderr: "error: .flowconfig: does not exist in index\n"
       });
     }
-    return Promise.resolve({stdout: ''});
+    return Promise.resolve({ stdout: "" });
   });
   try {
     await upgrade.func([newVersion], ctx, opts);
   } catch (error) {
     expect(error.message).toBe(
-      'Upgrade failed. Please see the messages above for details',
+      "Upgrade failed. Please see the messages above for details"
     );
   }
   expect(flushOutput()).toMatchInlineSnapshot(`
 "info Fetching diff between v0.57.8 and v0.58.4...
 [fs] write tmp-upgrade-rn.patch
-$ execa git remote add tmp-rn-diff-purge https://github.com/pvinis/rn-diff-purge.git
+$ execa git remote add tmp-rn-diff-purge https://github.com/react-native-community/rn-diff-purge.git
 $ execa git fetch --no-tags tmp-rn-diff-purge
 $ execa git apply --check tmp-upgrade-rn.patch --exclude=package.json -p2 --3way
 info Applying diff (excluding: package.json, .flowconfig)...
 $ execa git apply tmp-upgrade-rn.patch --exclude=package.json --exclude=.flowconfig -p2 --3way
 error: .flowconfig: does not exist in index
 error Automatically applying diff failed
-info Here's the diff we tried to apply: https://github.com/pvinis/rn-diff-purge/compare/version/0.57.8...version/0.58.4
+info Here's the diff we tried to apply: https://github.com/react-native-community/rn-diff-purge/compare/version/0.57.8...version/0.58.4
 info You may find release notes helpful: https://github.com/facebook/react-native/releases/tag/v0.58.4
 [fs] unlink tmp-upgrade-rn.patch
 warn Continuing after failure. Most of the files are upgraded but you will need to deal with some conflicts manually

packages/cli/src/commands/upgrade/upgrade.js

@@ -14,7 +14,7 @@ type FlagsT = {
   legacy: boolean,
 };
 
-const rnDiffPurgeUrl = 'https://github.com/pvinis/rn-diff-purge';
+const rnDiffPurgeUrl = 'https://github.com/react-native-community/rn-diff-purge';
 
 const getLatestRNVersion = async (): Promise<string> => {
   logger.info('No version passed. Fetching latest...');
@@ -52,7 +52,7 @@ const getPatch = async (currentVersion, newVersion, projectDir) => {
       `Failed to fetch diff for react-native@${newVersion}. Maybe it's not released yet?`,
     );
     logger.info(
-      'For available releases to diff see: https://github.com/pvinis/rn-diff-purge#version-changes',
+      'For available releases to diff see: https://github.com/react-native-community/rn-diff-purge#version-changes',
     );
     return null;
   }
@@ -187,7 +187,7 @@ async function upgrade(argv: Array<string>, ctx: ContextT, args: FlagsT) {
   if (args.legacy) {
     return legacyUpgrade.func(argv, ctx);
   }
-  const rnDiffGitAddress = 'https://github.com/pvinis/rn-diff-purge.git';
+  const rnDiffGitAddress = 'https://github.com/react-native-community/rn-diff-purge.git';
   const tmpRemote = 'tmp-rn-diff-purge';
   const tmpPatchFile = 'tmp-upgrade-rn.patch';
   const projectDir = ctx.root;