From 37aa5538590532c1cf050b82f452991e58cd5665 Mon Sep 17 00:00:00 2001
From: Paul Kaplan <pkaplan@media.mit.edu>
Date: Tue, 27 Nov 2018 11:22:53 -0500
Subject: [PATCH] Remove unused containers.
The functionality was all moved to the Controls container, these containers are no longer used
---
src/containers/green-flag.jsx | 60 -------------------
src/containers/stop-all.jsx | 55 -----------------
.../__snapshots__/green-flag.test.jsx.snap | 21 -------
test/unit/containers/green-flag.test.jsx | 49 ---------------
4 files changed, 185 deletions(-)
delete mode 100644 src/containers/green-flag.jsx
delete mode 100644 src/containers/stop-all.jsx
delete mode 100644 test/unit/containers/__snapshots__/green-flag.test.jsx.snap
delete mode 100644 test/unit/containers/green-flag.test.jsx
diff --git a/src/containers/green-flag.jsx b/src/containers/green-flag.jsx
deleted file mode 100644
index a725b7e1b..000000000
--- a/src/containers/green-flag.jsx
+++ /dev/null
@@ -1,60 +0,0 @@
-import bindAll from 'lodash.bindall';
-import PropTypes from 'prop-types';
-import React from 'react';
-
-import VM from 'scratch-vm';
-
-import GreenFlagComponent from '../components/green-flag/green-flag.jsx';
-
-class GreenFlag extends React.Component {
- constructor (props) {
- super(props);
- bindAll(this, [
- 'handleClick',
- 'onProjectRunStart',
- 'onProjectRunStop'
- ]);
- this.state = {projectRunning: false};
- }
- componentDidMount () {
- this.props.vm.addListener('PROJECT_RUN_START', this.onProjectRunStart);
- this.props.vm.addListener('PROJECT_RUN_STOP', this.onProjectRunStop);
- }
- componentWillUnmount () {
- this.props.vm.removeListener('PROJECT_RUN_START', this.onProjectRunStart);
- this.props.vm.removeListener('PROJECT_RUN_STOP', this.onProjectRunStop);
- }
- onProjectRunStart () {
- this.setState({projectRunning: true});
- }
- onProjectRunStop () {
- this.setState({projectRunning: false});
- }
- handleClick (e) {
- e.preventDefault();
- if (e.shiftKey) {
- this.props.vm.setTurboMode(!this.props.vm.runtime.turboMode);
- } else {
- this.props.vm.greenFlag();
- }
- }
- render () {
- const {
- vm, // eslint-disable-line no-unused-vars
- ...props
- } = this.props;
- return (
- <GreenFlagComponent
- active={this.state.projectRunning}
- onClick={this.handleClick}
- {...props}
- />
- );
- }
-}
-
-GreenFlag.propTypes = {
- vm: PropTypes.instanceOf(VM)
-};
-
-export default GreenFlag;
diff --git a/src/containers/stop-all.jsx b/src/containers/stop-all.jsx
deleted file mode 100644
index 0f7641a20..000000000
--- a/src/containers/stop-all.jsx
+++ /dev/null
@@ -1,55 +0,0 @@
-import bindAll from 'lodash.bindall';
-import PropTypes from 'prop-types';
-import React from 'react';
-import VM from 'scratch-vm';
-
-import StopAllComponent from '../components/stop-all/stop-all.jsx';
-
-class StopAll extends React.Component {
- constructor (props) {
- super(props);
- bindAll(this, [
- 'handleClick',
- 'onProjectRunStart',
- 'onProjectRunStop'
- ]);
- this.state = {projectRunning: false};
- }
- componentDidMount () {
- this.props.vm.addListener('PROJECT_RUN_START', this.onProjectRunStart);
- this.props.vm.addListener('PROJECT_RUN_STOP', this.onProjectRunStop);
- }
- componentWillUnmount () {
- this.props.vm.removeListener('PROJECT_RUN_START', this.onProjectRunStart);
- this.props.vm.removeListener('PROJECT_RUN_STOP', this.onProjectRunStop);
- }
- onProjectRunStart () {
- this.setState({projectRunning: true});
- }
- onProjectRunStop () {
- this.setState({projectRunning: false});
- }
- handleClick (e) {
- e.preventDefault();
- this.props.vm.stopAll();
- }
- render () {
- const {
- vm, // eslint-disable-line no-unused-vars
- ...props
- } = this.props;
- return (
- <StopAllComponent
- active={!this.state.projectRunning}
- onClick={this.handleClick}
- {...props}
- />
- );
- }
-}
-
-StopAll.propTypes = {
- vm: PropTypes.instanceOf(VM)
-};
-
-export default StopAll;
diff --git a/test/unit/containers/__snapshots__/green-flag.test.jsx.snap b/test/unit/containers/__snapshots__/green-flag.test.jsx.snap
deleted file mode 100644
index f8dbdbf82..000000000
--- a/test/unit/containers/__snapshots__/green-flag.test.jsx.snap
+++ /dev/null
@@ -1,21 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`GreenFlag Container renders active state 1`] = `
-<img
- className="undefined"
- draggable={false}
- onClick={[Function]}
- src="test-file-stub"
- title="Go"
-/>
-`;
-
-exports[`GreenFlag Container renders inactive state 1`] = `
-<img
- className=""
- draggable={false}
- onClick={[Function]}
- src="test-file-stub"
- title="Go"
-/>
-`;
diff --git a/test/unit/containers/green-flag.test.jsx b/test/unit/containers/green-flag.test.jsx
deleted file mode 100644
index 4744ebaab..000000000
--- a/test/unit/containers/green-flag.test.jsx
+++ /dev/null
@@ -1,49 +0,0 @@
-import React from 'react';
-import {shallow} from 'enzyme';
-import GreenFlag from '../../../src/containers/green-flag';
-import renderer from 'react-test-renderer';
-import VM from 'scratch-vm';
-
-describe('GreenFlag Container', () => {
- let vm;
- beforeEach(() => {
- vm = new VM();
- });
-
- test('renders active state', () => {
- const component = renderer.create(
- <GreenFlag
- active
- vm={vm}
- />
- );
- expect(component.toJSON()).toMatchSnapshot();
- });
-
- test('renders inactive state', () => {
- const component = renderer.create(
- <GreenFlag
- active={false}
- vm={vm}
- />
- );
- expect(component.toJSON()).toMatchSnapshot();
- });
-
- test('triggers onClick when active', () => {
- const onClick = jest.fn();
- const componentShallowWrapper = shallow(
- <GreenFlag
- active
- vm={vm}
- onClick={onClick}
- />
- );
- componentShallowWrapper.simulate('click');
- expect(onClick).toHaveBeenCalled();
- });
-
- // @todo: Test for handles key events.
- // @todo: Test project run start.
- // @todo: Test project run stop.
-});
--
GitLab