Merge pull request #392 from chintan9/fix/docs

Update docs
chintan9 · Oct 26, 2020 · cd40f31 · cd40f31
2 parents e1ab2ad + 662cd53
commit cd40f31
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -66,7 +66,7 @@ class MyComponent extends Component {
   render() {
     return (
       <>
-        <Plyr ref={(player) => (this.player = player)} />
+        <Plyr ref={(player) => (this.player.current = player)} />
       </>
     )
   }

diff --git a/test/Plyr.test.tsx b/test/Plyr.test.tsx
@@ -27,6 +27,15 @@ describe('<Plyr />', () => {
     expect(ref.current.plyr).toBeDefined()
   })
 
+  it('should render and have a plyr instance in ref.current when using a ref callback', () => {
+    const ref = createRef<any>() as any
+    const wrapper = mount(<Plyr ref={(player) => (ref.current = player)} />)
+
+    expect(wrapper).toBeDefined()
+    expect(ref.current).toBeDefined()
+    expect(ref.current.plyr).toBeDefined()
+  })
+
   it('should render and keep a plyr instance after a rerender', () => {
     const ref = createRef<any>()
     const wrapper = mount(<Plyr ref={ref} />)