From 80bee424103afa95a6fe93c30518f7586df20d4f Mon Sep 17 00:00:00 2001
From: DD <liudi08@gmail.com>
Date: Wed, 17 Oct 2018 14:33:21 -0400
Subject: [PATCH] Remove child rendered test and add no load test if fonts are
not loaded
---
test/unit/util/vm-manager-hoc.test.jsx | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/test/unit/util/vm-manager-hoc.test.jsx b/test/unit/util/vm-manager-hoc.test.jsx
index 1738b9dd1..39015a4de 100644
--- a/test/unit/util/vm-manager-hoc.test.jsx
+++ b/test/unit/util/vm-manager-hoc.test.jsx
@@ -98,16 +98,24 @@ describe('VMManagerHOC', () => {
// nextTick needed since vm.loadProject is async, and we have to wait for it :/
process.nextTick(() => expect(mockedOnLoadedProject).toHaveBeenLastCalledWith(LoadingState.LOADING_VM_WITH_ID));
});
- test('if there is projectData, the child is rendered', () => {
+ test('if the fontsLoaded prop is false, project data is never loaded', () => {
+ vm.loadProject = jest.fn(() => Promise.resolve());
+ const mockedOnLoadedProject = jest.fn();
const Component = () => <div />;
const WrappedComponent = vmManagerHOC(Component);
const mounted = mount(
<WrappedComponent
- projectData="100"
+ isLoadingWithId
store={store}
vm={vm}
+ onLoadedProject={mockedOnLoadedProject}
/>
);
- expect(mounted.find('div').length).toBe(1);
+ mounted.setProps({
+ loadingState: LoadingState.LOADING_VM_WITH_ID,
+ projectData: '100'
+ });
+ expect(vm.loadProject).toHaveBeenCalledTimes(0);
+ process.nextTick(() => expect(mockedOnLoadedProject).toHaveBeenCalledTimes(0));
});
});
--
GitLab