From dc048ef0476d8ab4490bc42702f3cfed245a031d Mon Sep 17 00:00:00 2001
From: Pavel Mineev <pavel@mineev.me>
Date: Sun, 21 Aug 2022 20:53:48 -0700
Subject: [PATCH] fix: anon cannot request data removal

---
 .../app/components/profile/profile.spec.tsx         | 13 ++++++++++++-
 .../remark42/app/components/profile/profile.tsx     |  5 +++--
 frontend/apps/remark42/package.json                 |  1 +
 frontend/pnpm-lock.yaml                             |  2 ++
 4 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/frontend/apps/remark42/app/components/profile/profile.spec.tsx b/frontend/apps/remark42/app/components/profile/profile.spec.tsx
index 42f6f662..0cb3a1b5 100644
--- a/frontend/apps/remark42/app/components/profile/profile.spec.tsx
+++ b/frontend/apps/remark42/app/components/profile/profile.spec.tsx
@@ -1,5 +1,5 @@
 import '@testing-library/jest-dom';
-import { waitFor, fireEvent } from '@testing-library/preact';
+import { waitFor, fireEvent, screen } from '@testing-library/preact';
 
 import { render } from 'tests/utils';
 import * as api from 'common/api';
@@ -190,4 +190,15 @@ describe('<Profile />', () => {
     fireEvent.click(await findByRole('button', { name: /load more/i }));
     expect(queryByTestId('preloader')).not.toBeInTheDocument();
   });
+
+  it('should not render removal button for anonymous user', async () => {
+    jest
+      .spyOn(api, 'getUserComments')
+      .mockImplementation(async () => ({ comments: new Array(10).fill(commentStub), count: 15 }));
+    jest.spyOn(pq, 'parseQuery').mockImplementation(() => ({ ...userParamsStub, name: 'anonymous_1' }));
+
+    render(<Profile />);
+
+    expect(screen.queryByText(/request my data removal/i)).not.toBeInTheDocument();
+  });
 });
diff --git a/frontend/apps/remark42/app/components/profile/profile.tsx b/frontend/apps/remark42/app/components/profile/profile.tsx
index b113eee5..5b4fead2 100644
--- a/frontend/apps/remark42/app/components/profile/profile.tsx
+++ b/frontend/apps/remark42/app/components/profile/profile.tsx
@@ -8,6 +8,7 @@ import { parseQuery } from 'utils/parse-query';
 import { requestDeletion } from 'utils/email';
 import { setStyles } from 'utils/set-dom-props';
 import { Avatar } from 'components/avatar';
+import { isUserAnonymous } from 'utils/isUserAnonymous';
 import { postMessageToParent } from 'utils/post-message';
 import { Comment } from 'components/comment';
 import { Preloader } from 'components/preloader';
@@ -18,7 +19,7 @@ import { CrossIcon } from 'components/icons/cross';
 import { IconButton } from 'components/icon-button/icon-button';
 import { Button } from 'components/auth/components/button';
 import { messages as authMessages } from 'components/auth/auth.messsages';
-import type { Comment as CommentType, Theme } from 'common/types';
+import type { Comment as CommentType, Theme, User } from 'common/types';
 
 import styles from './profile.module.css';
 import { Counter } from './components/counter';
@@ -219,7 +220,7 @@ export function Profile() {
           {comments === null && isCommentsLoading && <Preloader className={styles.preloader} />}
           {comments !== null && commentsJSX}
         </section>
-        {isCurrent ? (
+        {isCurrent && !isUserAnonymous(user as unknown as User) ? (
           <footer className={clsx('profile-footer', styles.footer)}>
             <Button kind="hollow" size="sm" onClick={handleClickRequestRemoveData}>
               <FormattedMessage id="profile.request-to-delete-data" defaultMessage="Request my data removal" />
diff --git a/frontend/apps/remark42/package.json b/frontend/apps/remark42/package.json
index e6ede276..25e2a208 100644
--- a/frontend/apps/remark42/package.json
+++ b/frontend/apps/remark42/package.json
@@ -51,6 +51,7 @@
     "@formatjs/cli": "^5.0.2",
     "@mavrin/stylelint-declaration-use-css-custom-properties": "^2.0.0",
     "@prefresh/babel-plugin": "^0.4.3",
+    "@prefresh/core": "^1.3.4",
     "@prefresh/webpack": "^3.3.4",
     "@size-limit/file": "^7.0.8",
     "@swc/core": "1.2.205",
diff --git a/frontend/pnpm-lock.yaml b/frontend/pnpm-lock.yaml
index a5bf91b7..34fbacc3 100644
--- a/frontend/pnpm-lock.yaml
+++ b/frontend/pnpm-lock.yaml
@@ -24,6 +24,7 @@ importers:
       '@github/text-expander-element': ^2.2.2
       '@mavrin/stylelint-declaration-use-css-custom-properties': ^2.0.0
       '@prefresh/babel-plugin': ^0.4.3
+      '@prefresh/core': ^1.3.4
       '@prefresh/webpack': ^3.3.4
       '@size-limit/file': ^7.0.8
       '@swc/core': 1.2.205
@@ -137,6 +138,7 @@ importers:
       '@formatjs/cli': 5.0.2
       '@mavrin/stylelint-declaration-use-css-custom-properties': 2.0.0_stylelint@14.9.1
       '@prefresh/babel-plugin': 0.4.3
+      '@prefresh/core': 1.3.4_preact@10.6.2
       '@prefresh/webpack': 3.3.4_aygq4fhb36ivjwsmiykvhlqsje
       '@size-limit/file': 7.0.8_size-limit@7.0.8
       '@swc/core': 1.2.205