Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Commit

Permalink
Revert "updated uuid version"
Browse files Browse the repository at this point in the history
This reverts commit 777bdee.
  • Loading branch information
sdadn committed Jul 11, 2023
1 parent 777bdee commit 7df22c8
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 28 deletions.
3 changes: 1 addition & 2 deletions packages/terra-clinical-detail-view/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@
## Unreleased

* Changed
* Updated DetailList to accept null as children.
* Locked `uuid` to `7.0.3` for consistency across Terra repos.
* Update Detail List to accept null as children

* Added
* Added accessibility guide.
Expand Down
2 changes: 1 addition & 1 deletion packages/terra-clinical-detail-view/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
"terra-clinical-label-value-view": "^3.29.0",
"terra-icon": "^3.52.0",
"terra-theme-context": "^1.0.0",
"uuid": "7.0.3"
"uuid": "8.2.0"
},
"scripts": {
"compile": "babel --root-mode upward src --out-dir lib --copy-files",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,3 @@ import DetailView from 'terra-clinical-detail-view';

## Detail List Item Props Table
<DetailListItemPropsTable />

## Testing

Menu uses `uuid` which changes the component's description id dynamically. To mock it with the Jest testing library, `jest.mock` can be used.

If Enzyme `shallow` rendering is being used for the tests then the mock may not be required. However, if `mount` is used then `uuid` should be mocked as shown below:

```js
// using a variable may result in failures. For best results, hardcode the mock return value.
jest.mock('uuid', () => ({ v4: () => '00000000-0000-0000-0000-000000000000' }));
1 change: 0 additions & 1 deletion packages/terra-clinical-result/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

* Changed
* Changed ResultTimeHeaderCell to remove the `<time>` html tag for the date and time. This was causing issues and technically does not have full screen reader support.
* Locked `uuid` to `7.0.3` for consistency across Terra repos.

## 1.18.0 - (June 22, 2023)

Expand Down
2 changes: 1 addition & 1 deletion packages/terra-clinical-result/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
"terra-mixins": "^1.0.0",
"terra-theme-context": "^1.0.0",
"terra-visually-hidden-text": "^2.36.0",
"uuid": "7.0.3"
"uuid": "8.2.0"
},
"scripts": {
"compile": "babel --root-mode upward src --out-dir lib --copy-files",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,16 +71,4 @@ import ClinicalResult from 'terra-clinical-result/lib/index';
<StatusEnteredInError title="Clinical Result Status Entered In Error" />

## Clinical Result Props Props
<ClinicalResultsProps />


## Testing

Menu uses `uuid` which changes the component's description id dynamically. To mock it with the Jest testing library, `jest.mock` can be used.

If Enzyme `shallow` rendering is being used for the tests then the mock may not be required. However, if `mount` is used then `uuid` should be mocked as shown below:

```js
// using a variable may result in failures. For best results, hardcode the mock return value.
jest.mock('uuid', () => ({ v4: () => '00000000-0000-0000-0000-000000000000' }));

<ClinicalResultsProps />

0 comments on commit 7df22c8

Please sign in to comment.