cozy-sharing 31.1.0 → 32.0.0

package/src/components/SharedDrive/helpers.spec.js

@@ -1,177 +0,0 @@
-import {
-  mergeAndDeduplicateRecipients,
-  moveRecipientToReadWrite,
-  moveRecipientToReadOnly,
-  formatRecipients,
-  RECIPIENT_INDEX_PREFIX
-} from './helpers'
-
-describe('mergeAndDeduplicateRecipients', () => {
-  it('should merge multiple arrays into one', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ id: '2', name: 'Bob' }]
-    ])
-    expect(result).toEqual([
-      { id: '1', _id: '1', name: 'Alice' },
-      { id: '2', _id: '2', name: 'Bob' }
-    ])
-  })
-
-  it('should deduplicate recipients with the same id', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ id: '1', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: '1', _id: '1', name: 'Alice' }])
-  })
-
-  it('should normalize _id to id for deduplication', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ _id: '1', name: 'Alice' }],
-      [{ _id: '2', name: 'Bob' }]
-    ])
-    expect(result).toEqual([
-      { _id: '1', id: '1', name: 'Alice' },
-      { _id: '2', id: '2', name: 'Bob' }
-    ])
-  })
-
-  it('should deduplicate recipients when one has id and the other has _id', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: '1', name: 'Alice' }],
-      [{ _id: '1', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: '1', _id: '1', name: 'Alice' }])
-  })
-
-  it('should prefer existing id over _id when both are present', () => {
-    const result = mergeAndDeduplicateRecipients([
-      [{ id: 'real-id', _id: 'other-id', name: 'Alice' }]
-    ])
-    expect(result).toEqual([{ id: 'real-id', _id: 'other-id', name: 'Alice' }])
-  })
-
-  it('should handle empty arrays', () => {
-    const result = mergeAndDeduplicateRecipients([[], []])
-    expect(result).toEqual([])
-  })
-})
-
-describe('moveRecipientToReadWrite', () => {
-  it('should move a recipient from readOnly to readWrite', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: [{ _id: '2', name: 'Bob' }]
-    }
-    const result = moveRecipientToReadWrite(state, '2')
-    expect(result.recipients).toEqual([
-      { _id: '1', name: 'Alice' },
-      { _id: '2', name: 'Bob' }
-    ])
-    expect(result.readOnlyRecipients).toEqual([])
-  })
-
-  it('should return unchanged state if recipient not found', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: []
-    }
-    const result = moveRecipientToReadWrite(state, '999')
-    expect(result).toBe(state)
-  })
-})
-
-describe('moveRecipientToReadOnly', () => {
-  it('should move a recipient from readWrite to readOnly', () => {
-    const state = {
-      recipients: [{ _id: '1', name: 'Alice' }],
-      readOnlyRecipients: [{ _id: '2', name: 'Bob' }]
-    }
-    const result = moveRecipientToReadOnly(state, '1')
-    expect(result.recipients).toEqual([])
-    expect(result.readOnlyRecipients).toEqual([
-      { _id: '2', name: 'Bob' },
-      { _id: '1', name: 'Alice' }
-    ])
-  })
-
-  it('should return unchanged state if recipient not found', () => {
-    const state = {
-      recipients: [],
-      readOnlyRecipients: [{ _id: '1', name: 'Alice' }]
-    }
-    const result = moveRecipientToReadOnly(state, '999')
-    expect(result).toBe(state)
-  })
-})
-
-describe('formatRecipients', () => {
-  it('should format readWrite and readOnly recipients with correct types', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '1',
-          displayName: 'Alice',
-          email: [{ address: 'alice@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: [
-        {
-          _id: '2',
-          displayName: 'Bob',
-          email: [{ address: 'bob@example.com', primary: true }]
-        }
-      ]
-    }
-    const result = formatRecipients(input)
-    expect(result).toHaveLength(2)
-    expect(result[0]).toMatchObject({
-      index: `${RECIPIENT_INDEX_PREFIX}1`,
-      type: 'two-way',
-      public_name: 'Alice'
-    })
-    expect(result[1]).toMatchObject({
-      index: `${RECIPIENT_INDEX_PREFIX}2`,
-      type: 'one-way',
-      public_name: 'Bob'
-    })
-  })
-
-  it('should flatten an object-shaped contact name into a string public_name', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '3',
-          name: { givenName: 'Arya', familyName: 'Stark' },
-          email: [{ address: 'arya@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: []
-    }
-    const result = formatRecipients(input)
-    expect(typeof result[0].public_name).toBe('string')
-    expect(result[0].public_name).toBe('Arya Stark')
-  })
-
-  it('should sort recipients by public_name', () => {
-    const input = {
-      recipients: [
-        {
-          _id: '1',
-          displayName: 'Zara',
-          email: [{ address: 'zara@example.com', primary: true }]
-        },
-        {
-          _id: '2',
-          displayName: 'Alice',
-          email: [{ address: 'alice@example.com', primary: true }]
-        }
-      ],
-      readOnlyRecipients: []
-    }
-    const result = formatRecipients(input)
-    expect(result[0].public_name).toBe('Alice')
-    expect(result[1].public_name).toBe('Zara')
-  })
-})

package/src/components/SharedStatus.jsx

@@ -1,50 +0,0 @@
-import cx from 'classnames'
-import React from 'react'
-
-import { useI18n } from 'twake-i18n'
-
-import { SharingTooltip, TooltipRecipientList } from './Tooltip'
-import LinkIcon from '../assets/icons/icon-link.svg'
-import { getDisplayName } from '../models'
-import styles from '../styles/status.styl'
-
-export const SharedStatus = ({ className, docId, recipients, link }) => {
-  const { t } = useI18n()
-  return (
-    <span className={cx(className, styles['shared-status'])}>
-      {recipients.length > 1 && (
-        <a
-          data-tip
-          data-for={`members${docId}`}
-          className={styles['shared-status-label']}
-        >
-          {t('Share.members.count', { smart_count: recipients.length })}
-        </a>
-      )}
-      {recipients.length > 1 && (
-        <SharingTooltip id={`members${docId}`}>
-          <TooltipRecipientList
-            recipientNames={recipients.map(recipient =>
-              getDisplayName(recipient)
-            )}
-          />
-        </SharingTooltip>
-      )}
-      {link && (
-        <>
-          <LinkIcon
-            style={{ fill: 'var(--iconTextColor)' }}
-            data-tip
-            data-for={`linkfor${docId}`}
-          />
-
-          <SharingTooltip id={`linkfor${docId}`}>
-            <span>{t('Files.share.sharedByLink')}</span>
-          </SharingTooltip>
-        </>
-      )}
-    </span>
-  )
-}
-
-export default SharedStatus

package/src/components/SharedStatus.spec.js

@@ -1,48 +0,0 @@
-import { render } from '@testing-library/react'
-import React from 'react'
-
-import { SharedStatus } from './SharedStatus'
-import AppLike from '../../test/AppLike'
-
-describe('SharedStatus component', () => {
-  it('should just render a span if no sharing', () => {
-    const { container } = render(
-      <AppLike>
-        <SharedStatus docId="1" recipients={[]} />
-      </AppLike>
-    )
-    expect(container).toMatchSnapshot()
-  })
-
-  it('should have the right display if there is several recipients', () => {
-    const { container, getByText } = render(
-      <AppLike>
-        <SharedStatus
-          docId="1"
-          recipients={[
-            {
-              _id: 1,
-              name: '1'
-            },
-            {
-              _id: 2,
-              name: '2'
-            }
-          ]}
-        />
-      </AppLike>
-    )
-    expect(container).toMatchSnapshot()
-    expect(getByText('2 members')).toBeTruthy()
-  })
-
-  it('should display the link if there is a link', () => {
-    const { container, getByText } = render(
-      <AppLike>
-        <SharedStatus docId="1" recipients={[]} link={true} />
-      </AppLike>
-    )
-    expect(container).toMatchSnapshot()
-    expect(getByText('Shared by link')).toBeTruthy()
-  })
-})

package/src/components/Sharesubmit.jsx

@@ -1,31 +0,0 @@
-import PropTypes from 'prop-types'
-import React from 'react'
-
-import Button from 'cozy-ui/transpiled/react/Buttons'
-
-const ShareSubmit = props => (
-  <Button
-    onClick={() => {
-      props.onSubmit()
-    }}
-    size="large"
-    busy={props.loading}
-    label={props.label}
-    disabled={props.disabled}
-    variant="ghost"
-  />
-)
-
-ShareSubmit.propTypes = {
-  onSubmit: PropTypes.func.isRequired,
-  label: PropTypes.string,
-  loading: PropTypes.bool,
-  disabled: PropTypes.bool
-}
-
-ShareSubmit.defaultProps = {
-  label: 'Submit',
-  loading: false
-}
-
-export default ShareSubmit

package/src/components/__snapshots__/SharedStatus.spec.js.snap

@@ -1,94 +0,0 @@
-// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
-
-exports[`SharedStatus component should display the link if there is a link 1`] = `
-<div>
-  <div
-    class="TwakeTheme--light u-dc"
-  >
-    <span
-      class="shared-status"
-    >
-      <svg
-        currentItem="false"
-        data-for="linkfor1"
-        data-tip="true"
-        height="16"
-        style="fill: var(--iconTextColor);"
-        viewBox="0 0 16 16"
-        width="16"
-        xmlns="http://www.w3.org/2000/svg"
-        xmlns:xlink="http://www.w3.org/1999/xlink"
-      >
-        <defs>
-          <path
-            d="M8.88 3.88a1 1 0 1 1-1.415-1.415L9.053.877A2.998 2.998 0 0 1 13.29.876l1.18 1.18a2.998 2.998 0 0 1-.007 4.24L10.999 9.76c-1.535 1.535-4.07 1.583-5.534.12A1 1 0 1 1 6.88 8.465c.67.67 1.94.646 2.705-.119l3.465-3.464a.998.998 0 0 0 .006-1.413l-1.18-1.18a.998.998 0 0 0-1.408.003L8.88 3.88zm-2.415 7.585A1 1 0 1 1 7.88 12.88l-1.587 1.587a2.998 2.998 0 0 1-4.237.002l-1.18-1.18a2.998 2.998 0 0 1 .007-4.24l3.464-3.464c1.534-1.535 4.07-1.583 5.533-.12A1 1 0 1 1 8.465 6.88c-.67-.67-1.94-.646-2.705.119l-3.464 3.464a.998.998 0 0 0-.006 1.413l1.18 1.18a.998.998 0 0 0 1.407-.003l1.588-1.588z"
-            id="link-1-path-1"
-          />
-        </defs>
-        <use
-          fill-rule="nonzero"
-          xlink:href="#link-1-path-1"
-        />
-      </svg>
-      <div
-        class="__react_component_tooltip place-bottom type-dark "
-        data-id="tooltip"
-        id="linkfor1"
-      >
-        <span>
-          Shared by link
-        </span>
-      </div>
-    </span>
-  </div>
-</div>
-`;
-
-exports[`SharedStatus component should have the right display if there is several recipients 1`] = `
-<div>
-  <div
-    class="TwakeTheme--light u-dc"
-  >
-    <span
-      class="shared-status"
-    >
-      <a
-        class="shared-status-label"
-        currentitem="false"
-        data-for="members1"
-        data-tip="true"
-      >
-        2 members
-      </a>
-      <div
-        class="__react_component_tooltip place-bottom type-dark "
-        data-id="tooltip"
-        id="members1"
-      >
-        <ul
-          class="shared-tooltip-list"
-        >
-          <li>
-            1
-          </li>
-          <li>
-            2
-          </li>
-        </ul>
-      </div>
-    </span>
-  </div>
-</div>
-`;
-
-exports[`SharedStatus component should just render a span if no sharing 1`] = `
-<div>
-  <div
-    class="TwakeTheme--light u-dc"
-  >
-    <span
-      class="shared-status"
-    />
-  </div>
-</div>
-`;

package/src/propTypes.js

@@ -1,29 +0,0 @@
-import PropTypes from 'prop-types'
-
-import { Contact, Group } from './models'
-
-export const contactsResponseType = PropTypes.shape({
-  count: PropTypes.number,
-  data: PropTypes.arrayOf(Contact.propType),
-  definition: PropTypes.object,
-  fetchMore: PropTypes.func,
-  fetchStatus: PropTypes.string,
-  hasMore: PropTypes.bool,
-  id: PropTypes.string,
-  lastError: PropTypes.string,
-  lastFetch: PropTypes.number,
-  lastUpdate: PropTypes.number
-})
-
-export const contactGroupsResponseType = PropTypes.shape({
-  count: PropTypes.number,
-  data: PropTypes.arrayOf(Group.propType),
-  definition: PropTypes.object,
-  fetchMore: PropTypes.func,
-  fetchStatus: PropTypes.string,
-  hasMore: PropTypes.bool,
-  id: PropTypes.string,
-  lastError: PropTypes.string,
-  lastFetch: PropTypes.number,
-  lastUpdate: PropTypes.number
-})