Merge pull request #22 from biancacamargomachado/clean-code

Clean code

.gitignore

@@ -7,3 +7,4 @@ target/*
 /target
 npm-debug.log
 /bin/
+client/*

client/server.js

@@ -243,7 +243,6 @@ function runDevServer(port, protocol) {
     // clearConsole();
     console.log(chalk.cyan('Starting the development server...'));
     console.log();
-    // openBrowser(port, protocol);
   });
 }
 

client/src/components/owners/NewOwnerPage.tsx

@@ -14,68 +14,4 @@ const newOwner = (): IOwner => ({
   pets: []
 });
 
-export default () => <OwnerEditor initialOwner={newOwner()} />;
-
-// export default class NewOwnerPage extends React.Component<void, INewOwnerPageState> {
-
-//   context: IRouterContext;
-
-//   static contextTypes = {
-//     router: React.PropTypes.object.isRequired
-//   };
-
-//   constructor() {
-//     super();
-//     this.onInputChange = this.onInputChange.bind(this);
-//     this.onSubmit = this.onSubmit.bind(this);
-
-//     this.state = { owner: newOwner() };
-//   }
-
-//   onSubmit(event) {
-//     event.preventDefault();
-
-//     const { owner } = this.state;
-
-//     submitForm('/api/owner', owner, (status, response) => {
-//       if (status === 201) {
-//         const newOwner = response as IOwner;
-//         this.context.router.push({
-//           pathname: '/owners/' + newOwner.id
-//         });
-//       } else {
-//         console.log('ERROR?!...', response);
-//         this.setState({ error: response });
-//       }
-//     });
-//   }
-
-//   onInputChange(name: string, value: string) {
-//     const { owner } = this.state;
-//     const modifiedOwner = Object.assign({}, owner, { [name]: value });
-//     this.setState({ owner: modifiedOwner });
-//   }
-
-//   render() {
-//     const { owner, error } = this.state;
-//     return (
-//       <span>
-//         <h2>New Owner</h2>
-//         <form className='form-horizontal' method='POST' action={url('/api/owner')}>
-//           <div className='form-group has-feedback'>
-//             <Input object={owner} error={error} label='First Name' name='firstName' onChange={this.onInputChange} />
-//             <Input object={owner} error={error} label='Last Name' name='lastName' onChange={this.onInputChange} />
-//             <Input object={owner} error={error} label='Address' name='address' onChange={this.onInputChange} />
-//             <Input object={owner} error={error} label='City' name='city' onChange={this.onInputChange} />
-//             <Input object={owner} error={error} label='Telephone' name='telephone' onChange={this.onInputChange} />
-//           </div>
-//           <div className='form-group'>
-//             <div className='col-sm-offset-2 col-sm-10'>
-//               <button className='btn btn-default' type='submit' onClick={this.onSubmit}>Add Owner</button>
-//             </div>
-//           </div>
-//         </form>
-//       </span>
-//     );
-//   }
-// }
+export default () => <OwnerEditor initialOwner={newOwner()} />;

client/tests/__tests__/fetch-mock.js

@@ -54,14 +54,6 @@ fetch.mockResponseOnce = (body, init) => {
   );
 };
 
-// fetch.mockResponses = (...responses) => {
-//   responses.forEach(([body, init]) => {
-//     fetch.mockImplementationOnce(
-//       () => Promise.resolve(new ResponseWrapper(body, init))
-//     );
-//   })
-// };
-
 // Default mock is just a empty string.
 fetch.mockResponse('');