From 3fe58c67fd2cd52f7eac66c9a6287ec28a8e81bb Mon Sep 17 00:00:00 2001 From: Emanuele Stoppa Date: Sat, 18 May 2019 16:46:55 +0100 Subject: [PATCH 1/3] feat: added container to the API - added example - added test - bumped library to fix low vulnerability --- README.md | 3 ++- package-lock.json | 2 +- package.json | 2 +- src/index.js | 1 + tests/render.spec.js | 8 ++++++++ 5 files changed, 13 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 747cdd5..be4d3bf 100644 --- a/README.md +++ b/README.md @@ -116,7 +116,7 @@ import App from '../src/App.svelte' import {render} from 'svelte-testing-library' describe('App', () => { test('should render greeting', () => { - const {getByText} = render(App, {props: {name: 'world'}}) + const {getByText, container} = render(App, {props: {name: 'world'}}) expect(getByText('Hello world!')) }) @@ -129,6 +129,7 @@ describe('App', () => { const button = await waitForElement(() => getByText('Button Clicked')) expect(button).toBeInTheDocument() + expect(container.firstChild).toMatchSnapshot() }) }) ``` diff --git a/package-lock.json b/package-lock.json index 27d70ef..044b446 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,5 +1,5 @@ { - "name": "svlt-testing-library", + "name": "svelte-testing-library", "version": "0.0.0-semantically-released", "lockfileVersion": 1, "requires": true, diff --git a/package.json b/package.json index 5f5eca7..5491053 100644 --- a/package.json +++ b/package.json @@ -15,7 +15,7 @@ "jest": "^24.7.1", "jest-dom": "^3.1.2", "jest-transform-svelte": "^1.0.0", - "kcd-scripts": "^1.2.1", + "kcd-scripts": "^1.4.0", "npm-run-all": "^4.1.5", "rollup": "^1.10.1", "rollup-plugin-commonjs": "^9.3.4", diff --git a/src/index.js b/src/index.js index eff31a0..91c8286 100644 --- a/src/index.js +++ b/src/index.js @@ -17,6 +17,7 @@ export const render = (Component, {target, ...options} = {}) => { component, // eslint-disable-next-line no-console debug: (el = document.body) => console.log(prettyDOM(el)), + container: document.body, ...getQueriesForElement(document.body), } } diff --git a/tests/render.spec.js b/tests/render.spec.js index 4a65481..8b9ea36 100644 --- a/tests/render.spec.js +++ b/tests/render.spec.js @@ -74,4 +74,12 @@ describe('render', () => { expect(document.body.innerHTML).toBe('') }) + + test('should return a container object, which contains the DOM of the rendered component', () => { + const {container} = render(App, {props: {name: 'world'}}) + + expect(container.innerHTML).toBe(document.body.innerHTML) + + cleanup() + }) }) From efec95f2d0e0de57d1cd85a072252a1574772361 Mon Sep 17 00:00:00 2001 From: Emanuele Date: Sat, 18 May 2019 18:03:19 +0100 Subject: [PATCH 2/3] Update README.md Co-Authored-By: Ben Monro --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index be4d3bf..cfd3bb8 100644 --- a/README.md +++ b/README.md @@ -116,7 +116,7 @@ import App from '../src/App.svelte' import {render} from 'svelte-testing-library' describe('App', () => { test('should render greeting', () => { - const {getByText, container} = render(App, {props: {name: 'world'}}) + const {getByText} = render(App, {props: {name: 'world'}}) expect(getByText('Hello world!')) }) From 463d86d8e69c3f407c5d91b82697dc5860be8f98 Mon Sep 17 00:00:00 2001 From: Emanuele Date: Sat, 18 May 2019 18:03:26 +0100 Subject: [PATCH 3/3] Update README.md Co-Authored-By: Ben Monro --- README.md | 1 - 1 file changed, 1 deletion(-) diff --git a/README.md b/README.md index cfd3bb8..747cdd5 100644 --- a/README.md +++ b/README.md @@ -129,7 +129,6 @@ describe('App', () => { const button = await waitForElement(() => getByText('Button Clicked')) expect(button).toBeInTheDocument() - expect(container.firstChild).toMatchSnapshot() }) }) ```