Replace 'uuid' package with 'consecutive' package, for more predictab…

…le id attributes.
springload · Oct 6, 2017 · 15c0324 · 15c0324
1 parent 936d956
commit 15c0324
Show file tree

Hide file tree

Showing 4 changed files with 794 additions and 571 deletions.
diff --git a/package.json b/package.json
@@ -94,7 +94,7 @@
   },
   "dependencies": {
     "classnames": "^2.2.5",
-    "uuid": "^3.0.1"
+    "consecutive": "^5.0.4"
   },
   "peerDependencies": {
     "react": "^15.4.0 || ^16.0.0",

diff --git a/src/AccordionItem/accordion-item.js b/src/AccordionItem/accordion-item.js
@@ -2,9 +2,11 @@
 
 import React, { Component } from 'react';
 import type { Node } from 'react';
-import uuid from 'uuid';
+import consecutive from 'consecutive';
 import classNames from 'classnames';
 
+const nextUuid = consecutive();
+
 type AccordionItemProps = {
     accordion: boolean,
     expanded: boolean,
@@ -28,7 +30,7 @@ class AccordionItem extends Component<AccordionItemProps, AccordionItemState> {
     };
 
     state = {
-        itemUuid: uuid.v4(),
+        itemUuid: nextUuid(),
     };
 
     renderChildren() {

diff --git a/src/AccordionItem/accordion-item.spec.js b/src/AccordionItem/accordion-item.spec.js
@@ -7,9 +7,7 @@ import AccordionItemTitle from '../AccordionItemTitle/accordion-item-title';
 import AccordionItemBody from '../AccordionItemBody/accordion-item-body';
 import AccordionItem from './accordion-item';
 
-jest.mock('uuid', () => ({
-    v4: () => 'mock-uuid-hash',
-}));
+jest.mock('consecutive', () => () => () => 'mock-uuid-hash');
 
 jest.mock('../AccordionItemTitle/accordion-item-title', () => {
     const RealReact = require.requireActual('react');