@openneuro/app 4.20.4 → 4.20.6-alpha.2

package/src/scripts/dataset/mutations/__tests__/deprecate-snapshot.spec.tsx

@@ -1,13 +1,13 @@
-import { describe, it, expect, vi } from 'vitest'
-import React from 'react'
-import { render, fireEvent, screen, waitFor } from '@testing-library/react'
-import { createMemoryHistory } from 'history'
-import { MockedProvider } from '@apollo/client/testing'
-import { Router, MemoryRouter } from 'react-router-dom'
-import { DeprecateVersion, DEPRECATE_VERSION } from '../deprecate-version'
+import { describe, expect, it, vi } from "vitest"
+import React from "react"
+import { fireEvent, render, screen, waitFor } from "@testing-library/react"
+import { createMemoryHistory } from "history"
+import { MockedProvider } from "@apollo/client/testing"
+import { MemoryRouter, Router } from "react-router-dom"
+import { DEPRECATE_VERSION, DeprecateVersion } from "../deprecate-version"
 
-describe('DeprecateVersion mutation', () => {
-  it('renders with typical props', () => {
+describe("DeprecateVersion mutation", () => {
+  it("renders with typical props", () => {
     const { asFragment } = render(
       <MemoryRouter>
         <MockedProvider>
@@ -21,14 +21,14 @@ describe('DeprecateVersion mutation', () => {
     )
     expect(asFragment()).toMatchSnapshot()
   })
-  it('calls the DEPRECATE_VERSION mutation when clicked and navigates to the snapshot on success', async () => {
-    const datasetId = 'test00001'
-    const tag = '1.0.0'
-    const reason = 'This is a test suite.'
+  it("calls the DEPRECATE_VERSION mutation when clicked and navigates to the snapshot on success", async () => {
+    const datasetId = "test00001"
+    const tag = "1.0.0"
+    const reason = "This is a test suite."
     const history = createMemoryHistory({
       initialEntries: [`/datasets/${datasetId}/versions/${tag}/deprecate`],
     })
-    const historyPushSpy = vi.spyOn(history, 'push')
+    const historyPushSpy = vi.spyOn(history, "push")
     const snapshotId = `${datasetId}:${tag}`
     const deprecateSnapshotMock = {
       request: {
@@ -42,12 +42,12 @@ describe('DeprecateVersion mutation', () => {
       result: {
         data: {
           deprecateSnapshot: {
-            __typename: 'Snapshot',
+            __typename: "Snapshot",
             id: snapshotId,
             deprecated: {
-              __typename: 'DeprecatedSnapshot',
+              __typename: "DeprecatedSnapshot",
               id: snapshotId,
-              user: '1245',
+              user: "1245",
               reason,
             },
           },

package/src/scripts/dataset/mutations/__tests__/deprecate-version.spec.tsx

@@ -1,18 +1,18 @@
-import { describe, it, expect, vi } from 'vitest'
-import React from 'react'
-import { render, fireEvent, screen, waitFor } from '@testing-library/react'
-import { MockedProvider } from '@apollo/client/testing'
+import { describe, expect, it, vi } from "vitest"
+import React from "react"
+import { fireEvent, render, screen, waitFor } from "@testing-library/react"
+import { MockedProvider } from "@apollo/client/testing"
 import {
   Router,
   unstable_HistoryRouter as HistoryRouter,
-} from 'react-router-dom'
-import { createMemoryHistory } from 'history'
-import { DeprecateVersion, DEPRECATE_VERSION } from '../deprecate-version'
+} from "react-router-dom"
+import { createMemoryHistory } from "history"
+import { DEPRECATE_VERSION, DeprecateVersion } from "../deprecate-version"
 
-describe('DeprecateVersion mutation', () => {
-  it('renders with typical props', () => {
-    const datasetId = 'test00001'
-    const tag = '1.0.0'
+describe("DeprecateVersion mutation", () => {
+  it("renders with typical props", () => {
+    const datasetId = "test00001"
+    const tag = "1.0.0"
     const history = createMemoryHistory({
       initialEntries: [`/datasets/${datasetId}/versions/${tag}/deprecate`],
     })
@@ -29,14 +29,14 @@ describe('DeprecateVersion mutation', () => {
     )
     expect(asFragment()).toMatchSnapshot()
   })
-  it('calls the DEPRECATE_VERSION mutation when clicked and navigates to the snapshot on success', async () => {
-    const datasetId = 'test00001'
-    const tag = '1.0.0'
-    const reason = 'This is a test suite.'
+  it("calls the DEPRECATE_VERSION mutation when clicked and navigates to the snapshot on success", async () => {
+    const datasetId = "test00001"
+    const tag = "1.0.0"
+    const reason = "This is a test suite."
     const history = createMemoryHistory({
       initialEntries: [`/datasets/${datasetId}/versions/${tag}/deprecate`],
     })
-    const historyPushSpy = vi.spyOn(history, 'push')
+    const historyPushSpy = vi.spyOn(history, "push")
     const snapshotId = `${datasetId}:${tag}`
     const deprecateSnapshotMock = {
       request: {
@@ -50,12 +50,12 @@ describe('DeprecateVersion mutation', () => {
       result: {
         data: {
           deprecateSnapshot: {
-            __typename: 'Snapshot',
+            __typename: "Snapshot",
             id: snapshotId,
             deprecated: {
-              __typename: 'DeprecatedSnapshot',
+              __typename: "DeprecatedSnapshot",
               id: snapshotId,
-              user: '1245',
+              user: "1245",
               reason,
             },
           },

package/src/scripts/dataset/mutations/__tests__/description.spec.jsx

@@ -1,23 +1,23 @@
-import React from 'react'
-import { render } from '@testing-library/react'
-import { MockedProvider } from '@apollo/client/testing'
+import React from "react"
+import { render } from "@testing-library/react"
+import { MockedProvider } from "@apollo/client/testing"
 import UpdateDescription, {
   mergeFieldValue,
   UPDATE_DESCRIPTION,
   UPDATE_DESCRIPTION_LIST,
-} from '../description.jsx'
+} from "../description.jsx"
 
 const mockMutation = vi.fn()
-vi.mock('@apollo/client/react/components', () => ({
-  ...vi.importActual('@apollo/client/react/components'),
-  Mutation: props => {
+vi.mock("@apollo/client/react/components", () => ({
+  ...vi.importActual("@apollo/client/react/components"),
+  Mutation: (props) => {
     mockMutation(props)
     return <>Mutation mock</>
   },
 }))
 
-describe('UpdateDescription mutation', () => {
-  it('renders with common props', () => {
+describe("UpdateDescription mutation", () => {
+  it("renders with common props", () => {
     const { asFragment } = render(
       <UpdateDescription
         datasetId="ds001"
@@ -29,7 +29,7 @@ describe('UpdateDescription mutation', () => {
     )
     expect(asFragment()).toMatchSnapshot()
   })
-  it('uses the scalar mutation for scalar values', () => {
+  it("uses the scalar mutation for scalar values", () => {
     render(
       <UpdateDescription
         datasetId="ds001"
@@ -43,12 +43,12 @@ describe('UpdateDescription mutation', () => {
       expect.objectContaining({ mutation: UPDATE_DESCRIPTION }),
     )
   })
-  it('uses the list mutation for array values', () => {
+  it("uses the list mutation for array values", () => {
     render(
       <UpdateDescription
         datasetId="ds001"
         field="Authors"
-        value={['John Doe', 'Jane Doe']}
+        value={["John Doe", "Jane Doe"]}
         done={vi.fn()}
       />,
       { wrapper: MockedProvider },
@@ -57,76 +57,76 @@ describe('UpdateDescription mutation', () => {
       expect.objectContaining({ mutation: UPDATE_DESCRIPTION_LIST }),
     )
   })
-  describe('mergeFieldValue()', () => {
-    it('merges in scalar fields', () => {
+  describe("mergeFieldValue()", () => {
+    it("merges in scalar fields", () => {
       expect(
         mergeFieldValue(
-          'ds001',
+          "ds001",
           {
-            __typename: 'Draft',
-            id: '1234',
-            created: '1999',
+            __typename: "Draft",
+            id: "1234",
+            created: "1999",
             description: {
-              Name: 'Old Name',
-              BIDSVersion: '1.2',
-              License: 'AGPL3',
+              Name: "Old Name",
+              BIDSVersion: "1.2",
+              License: "AGPL3",
             },
           },
-          { Name: 'New Name', BIDSVersion: '1.2', License: 'AGPL3' },
+          { Name: "New Name", BIDSVersion: "1.2", License: "AGPL3" },
           null,
         ),
       ).toEqual({
-        __typename: 'Dataset',
+        __typename: "Dataset",
         draft: {
-          __typename: 'Draft',
-          created: '1999',
+          __typename: "Draft",
+          created: "1999",
           description: {
-            BIDSVersion: '1.2',
-            License: 'AGPL3',
-            Name: 'New Name',
+            BIDSVersion: "1.2",
+            License: "AGPL3",
+            Name: "New Name",
           },
-          id: '1234',
+          id: "1234",
         },
-        id: 'ds001',
+        id: "ds001",
       })
     })
-    it('merges in array fields', () => {
+    it("merges in array fields", () => {
       expect(
         mergeFieldValue(
-          'ds001',
+          "ds001",
           {
-            __typename: 'Draft',
-            id: '1234',
-            created: '1999',
+            __typename: "Draft",
+            id: "1234",
+            created: "1999",
             description: {
-              Name: 'Old Name',
-              BIDSVersion: '1.2',
-              License: 'AGPL3',
+              Name: "Old Name",
+              BIDSVersion: "1.2",
+              License: "AGPL3",
               Authors: [],
             },
           },
           null,
           {
-            Name: 'New Name',
-            BIDSVersion: '1.2',
-            License: 'AGPL3',
-            Authors: ['One', 'Two'],
+            Name: "New Name",
+            BIDSVersion: "1.2",
+            License: "AGPL3",
+            Authors: ["One", "Two"],
           },
         ),
       ).toEqual({
-        __typename: 'Dataset',
+        __typename: "Dataset",
         draft: {
-          __typename: 'Draft',
-          created: '1999',
+          __typename: "Draft",
+          created: "1999",
           description: {
-            Authors: ['One', 'Two'],
-            BIDSVersion: '1.2',
-            License: 'AGPL3',
-            Name: 'New Name',
+            Authors: ["One", "Two"],
+            BIDSVersion: "1.2",
+            License: "AGPL3",
+            Name: "New Name",
           },
-          id: '1234',
+          id: "1234",
         },
-        id: 'ds001',
+        id: "ds001",
       })
     })
   })

package/src/scripts/dataset/mutations/__tests__/remove-permissions.spec.tsx

@@ -1,20 +1,20 @@
-import { userPermissionsFilter } from '../remove-permissions'
+import { userPermissionsFilter } from "../remove-permissions"
 
-describe('RemovePermissions mutation', () => {
-  describe('userPermissionsFilter()', () => {
-    it('accepts a permissions array object', () => {
+describe("RemovePermissions mutation", () => {
+  describe("userPermissionsFilter()", () => {
+    it("accepts a permissions array object", () => {
       expect(
         userPermissionsFilter(
-          [{ user: { id: '1234', email: 'example@example.com' }, level: 'ro' }],
-          '1234',
+          [{ user: { id: "1234", email: "example@example.com" }, level: "ro" }],
+          "1234",
         ),
       ).toEqual([])
     })
-    it('does not filter non-matching entries', () => {
+    it("does not filter non-matching entries", () => {
       const permissions = [
-        { user: { id: '5678', email: 'example@example.com' }, level: 'ro' },
+        { user: { id: "5678", email: "example@example.com" }, level: "ro" },
       ]
-      expect(userPermissionsFilter(permissions, '1234')).toEqual(permissions)
+      expect(userPermissionsFilter(permissions, "1234")).toEqual(permissions)
     })
   })
 })

package/src/scripts/dataset/mutations/__tests__/update-permissions.spec.jsx

@@ -1,10 +1,10 @@
-import React from 'react'
-import { render } from '@testing-library/react'
-import { MockedProvider } from '@apollo/client/testing'
-import { UpdateDatasetPermissions } from '../update-permissions'
+import React from "react"
+import { render } from "@testing-library/react"
+import { MockedProvider } from "@apollo/client/testing"
+import { UpdateDatasetPermissions } from "../update-permissions"
 
-describe('UpdateDatasetPermissions mutation', () => {
-  it('renders with default props', () => {
+describe("UpdateDatasetPermissions mutation", () => {
+  it("renders with default props", () => {
     const { asFragment } = render(
       <MockedProvider>
         <UpdateDatasetPermissions />
@@ -12,7 +12,7 @@ describe('UpdateDatasetPermissions mutation', () => {
     )
     expect(asFragment()).toMatchSnapshot()
   })
-  it('renders with typical props', () => {
+  it("renders with typical props", () => {
     const { asFragment } = render(
       <MockedProvider>
         <UpdateDatasetPermissions

package/src/scripts/dataset/mutations/admin-exports.jsx

@@ -1,9 +1,9 @@
-import React from 'react'
-import { gql, useMutation } from '@apollo/client'
-import PropTypes from 'prop-types'
-import { reexporterLogsURL } from '../../resources/kibana'
-import { Button } from '@openneuro/components/button'
-import styled from '@emotion/styled'
+import React from "react"
+import { gql, useMutation } from "@apollo/client"
+import PropTypes from "prop-types"
+import { reexporterLogsURL } from "../../resources/kibana"
+import { Button } from "@openneuro/components/button"
+import styled from "@emotion/styled"
 
 const ButtonRow = styled.div`
   display: grid;
@@ -13,13 +13,13 @@ const ButtonRow = styled.div`
 `
 
 const SuccessMessage = styled.p({
-  color: 'rgb(92, 184, 92)',
+  color: "rgb(92, 184, 92)",
 })
 const InProgressMessage = styled.p({
-  color: 'orange',
+  color: "orange",
 })
 const ErrorMessage = styled.p({
-  color: 'red',
+  color: "red",
 })
 
 const REEXPORT_REMOTES = gql`
@@ -29,8 +29,9 @@ const REEXPORT_REMOTES = gql`
 `
 
 const AdminExports = ({ dataset }) => {
-  const [reexportRemotes, { data, loading, error }] =
-    useMutation(REEXPORT_REMOTES)
+  const [reexportRemotes, { data, loading, error }] = useMutation(
+    REEXPORT_REMOTES,
+  )
   const success = data && data.reexportRemotes
   return (
     <div className="dataset-form">
@@ -55,7 +56,7 @@ const AdminExports = ({ dataset }) => {
           secondary={true}
           size="small"
           onClick={() => {
-            window.open(reexporterLogsURL, '_blank')
+            window.open(reexporterLogsURL, "_blank")
           }}
         />
       </ButtonRow>

package/src/scripts/dataset/mutations/cache-clear.jsx

@@ -1,8 +1,8 @@
-import React from 'react'
-import PropTypes from 'prop-types'
-import { gql } from '@apollo/client'
-import { Mutation } from '@apollo/client/react/components'
-import { Button } from '@openneuro/components/button'
+import React from "react"
+import PropTypes from "prop-types"
+import { gql } from "@apollo/client"
+import { Mutation } from "@apollo/client/react/components"
+import { Button } from "@openneuro/components/button"
 const CACHE_CLEAR = gql`
   mutation cacheClear($datasetId: ID!) {
     cacheClear(datasetId: $datasetId)
@@ -11,7 +11,7 @@ const CACHE_CLEAR = gql`
 
 const CacheClear = ({ datasetId }) => (
   <Mutation mutation={CACHE_CLEAR}>
-    {cacheClear => (
+    {(cacheClear) => (
       <span>
         <Button
           icon="fa fa-eraser"

package/src/scripts/dataset/mutations/cache-id.js

@@ -1,3 +1,3 @@
-export const datasetCacheId = datasetId => `Dataset:${datasetId}`
+export const datasetCacheId = (datasetId) => `Dataset:${datasetId}`
 
-export const draftCacheId = draftId => `Draft:${draftId}`
+export const draftCacheId = (draftId) => `Draft:${draftId}`

package/src/scripts/dataset/mutations/comment.jsx

@@ -1,11 +1,11 @@
-import React from 'react'
-import PropTypes from 'prop-types'
-import { gql } from '@apollo/client'
-import { Mutation } from '@apollo/client/react/components'
-import { convertToRaw } from 'draft-js'
-import withProfile from '../../authentication/withProfile.jsx'
-import { DATASET_COMMENTS } from '../fragments/comments-fragments'
-import { datasetCacheId } from './cache-id.js'
+import React from "react"
+import PropTypes from "prop-types"
+import { gql } from "@apollo/client"
+import { Mutation } from "@apollo/client/react/components"
+import { convertToRaw } from "draft-js"
+import withProfile from "../../authentication/withProfile.jsx"
+import { DATASET_COMMENTS } from "../fragments/comments-fragments"
+import { datasetCacheId } from "./cache-id.js"
 
 const NEW_COMMENT = gql`
   mutation addComment($datasetId: ID!, $parentId: ID, $comment: String!) {
@@ -30,12 +30,12 @@ const EDIT_COMMENT = gql`
  */
 export const commentStateFactory = (id, parentId, body, profile) => ({
   id,
-  parent: parentId ? { __typename: 'Comment', id: parentId } : null,
+  parent: parentId ? { __typename: "Comment", id: parentId } : null,
   text: JSON.stringify(convertToRaw(body)),
   createDate: new Date().toISOString(),
-  user: { __typename: 'User', ...profile },
+  user: { __typename: "User", ...profile },
   replies: [],
-  __typename: 'Comment',
+  __typename: "Comment",
 })
 
 /**
@@ -58,12 +58,12 @@ export const newCommentsReducer = (
   // If this is not a root level comment, add to replies
   if (parentId) {
     const parentIndex = nextCommentsState.findIndex(
-      comment => comment.id === parentId,
+      (comment) => comment.id === parentId,
     )
     const parentReplies = nextCommentsState[parentIndex].replies
     nextCommentsState[parentIndex] = {
       ...comments[parentIndex],
-      replies: [...parentReplies, { __typename: 'Comment', id: commentId }],
+      replies: [...parentReplies, { __typename: "Comment", id: commentId }],
     }
   }
   return nextCommentsState
@@ -81,7 +81,7 @@ export const modifyCommentsReducer = (comments, { commentId, comment }) => {
   // Must copy with freezeResults enabled
   const nextCommentsState = [...comments]
   const modifiedCommentIndex = nextCommentsState.findIndex(
-    c => c.id === commentId,
+    (c) => c.id === commentId,
   )
   const modifiedComment = nextCommentsState[modifiedCommentIndex]
   nextCommentsState[modifiedCommentIndex] = {
@@ -113,23 +113,24 @@ const CommentMutation = ({
         // Apply state reduction to cache for new comment changes
         const nextCommentsState = addComment
           ? newCommentsReducer(comments, {
-              parentId,
-              commentId: addComment,
-              comment,
-              profile,
-            })
+            parentId,
+            commentId: addComment,
+            comment,
+            profile,
+          })
           : modifyCommentsReducer(comments, { commentId, comment })
         cache.writeFragment({
           id: datasetCacheId(datasetId),
           fragment: DATASET_COMMENTS,
           data: {
-            __typename: 'Dataset',
+            __typename: "Dataset",
             id: datasetId,
             comments: nextCommentsState,
           },
         })
-      }}>
-      {newComment => (
+      }}
+    >
+      {(newComment) => (
         <button
           className="on-button on-button--primary  on-button--small "
           disabled={disabled}
@@ -143,7 +144,8 @@ const CommentMutation = ({
               },
             })
             done()
-          }}>
+          }}
+        >
           Submit Comment
         </button>
       )}

package/src/scripts/dataset/mutations/create-anonymous-reviewer.tsx

@@ -1,8 +1,8 @@
-import React, { FC } from 'react'
-import { gql, useMutation } from '@apollo/client'
-import { Link } from 'react-router-dom'
-import { Tooltip } from '@openneuro/components/tooltip'
-import { Button } from '@openneuro/components/button'
+import React, { FC } from "react"
+import { gql, useMutation } from "@apollo/client"
+import { Link } from "react-router-dom"
+import { Tooltip } from "@openneuro/components/tooltip"
+import { Button } from "@openneuro/components/button"
 
 const CREATE_LINK = gql`
   mutation createReviewer($datasetId: ID!) {
@@ -28,32 +28,38 @@ export const CreateReviewLink: FC<CreateReviewLinkProps> = ({ datasetId }) => {
   return (
     <>
       <div className="share-form-controls">
-        {error ? (
-          'An Error Occured'
-        ) : data ? (
-          <div className="reviewer-link">
-            <div>
-              <Tooltip
-                tooltip="Click to copy URL to clipboard"
-                flow="right"
-                className="tooltip">
-                <Button
-                  onClick={() => copyToClipboard(data.createReviewer.url)}
-                  icon="fas fa-clipboard"
-                  size="small"
-                  nobg={true}
-                  iconSize="18px"
-                  label="copy anonymous URL"
-                />
-              </Tooltip>
-              <pre>{data.createReviewer.url}</pre>
+        {error
+          ? (
+            "An Error Occured"
+          )
+          : data
+          ? (
+            <div className="reviewer-link">
+              <div>
+                <Tooltip
+                  tooltip="Click to copy URL to clipboard"
+                  flow="right"
+                  className="tooltip"
+                >
+                  <Button
+                    onClick={() => copyToClipboard(data.createReviewer.url)}
+                    icon="fas fa-clipboard"
+                    size="small"
+                    nobg={true}
+                    iconSize="18px"
+                    label="copy anonymous URL"
+                  />
+                </Tooltip>
+                <pre>{data.createReviewer.url}</pre>
+              </div>
+              <small className="alert-color">
+                Copy and save this reviewer link. It will disappear when you
+                leave the page. To get a new link click the Create Link button
+                below.
+              </small>
             </div>
-            <small className="alert-color">
-              Copy and save this reviewer link. It will disappear when you leave
-              the page. To get a new link click the Create Link button below.
-            </small>
-          </div>
-        ) : null}
+          )
+          : null}
         <Button
           className="btn-modal-action"
           primary={true}