Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Commit

Permalink
[TASK] Move the ExampleComponent into a separate partial to reduce th…
Browse files Browse the repository at this point in the history
…e duplicate code across the tests #8
  • Loading branch information
Inkdpixels committed Jun 27, 2015
1 parent 3470eb7 commit b6bf002
Show file tree
Hide file tree
Showing 12 changed files with 28 additions and 44 deletions.
9 changes: 1 addition & 8 deletions Tests/API/Element.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
var nodeProto = require('./../../Src/NodeProto.js');

// An ExampleClass for testing purposes.
class ExampleComponent extends nodeProto.Component {
constructor(el, props) {
super(el, props)
}
}
var ExampleComponent = require('./../ExampleComponent.js');

describe('NodeProto: Element API', () => {
it('should return a DOM element even if none was passed directly to the Constructor.', () => {
Expand Down
9 changes: 1 addition & 8 deletions Tests/API/Events.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
var nodeProto = require('./../../Src/NodeProto.js');

// An ExampleClass for testing purposes.
class ExampleComponent extends nodeProto.Component {
constructor(el, props) {
super(el, props)
}
}
var ExampleComponent = require('./../ExampleComponent.js');

describe('NodeProto: Events API', () => {
let instance;
Expand Down
14 changes: 1 addition & 13 deletions Tests/API/Prop.js
Original file line number Diff line number Diff line change
@@ -1,17 +1,5 @@
var nodeProto = require('./../../Src/NodeProto.js');

// An ExampleClass for testing purposes.
class ExampleComponent extends nodeProto.Component {
constructor(el, props, propTypes) {
super(el, props, propTypes)
}

getDefaultProps() {
return {
'anotherProp': 2
}
}
}
var ExampleComponent = require('./../ExampleComponent.js');

describe('NodeProto: Prop API', () => {
it('should return undefined if no prop was set.', () => {
Expand Down
9 changes: 1 addition & 8 deletions Tests/API/State.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
var nodeProto = require('./../../Src/NodeProto.js');

// An ExampleClass for testing purposes.
class ExampleComponent extends nodeProto.Component {
constructor(el, props) {
super(el, props)
}
}
var ExampleComponent = require('./../ExampleComponent.js');

describe('NodeProto: State API', () => {
let instance;
Expand Down
16 changes: 16 additions & 0 deletions Tests/ExampleComponent.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
var nodeProto = require('./../Src/NodeProto.js');

// An ExampleClass for testing purposes.
class ExampleComponent extends nodeProto.Component {
constructor(el, props, propTypes) {
super(el, props, propTypes)
}

getDefaultProps() {
return {
'anotherProp': 2
}
}
}

module.exports = ExampleComponent;
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsNumber/IsOptional.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../../Src/NodeProto.js';
var nodeProto = require('./../../../Src/NodeProto.js');

const propType = nodeProto.propTypes.isNumber.isOptional;

Expand Down
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsNumber/IsRequired.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../../Src/NodeProto.js';
var nodeProto = require('./../../../Src/NodeProto.js');
const propType = nodeProto.propTypes.isNumber.isRequired;

describe('NodeProto: propTypes.isNumber.isRequired', () => {
Expand Down
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsObject/IsOptional.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../../Src/NodeProto.js';
var nodeProto = require('./../../../Src/NodeProto.js');

const propType = nodeProto.propTypes.isObject.isOptional;

Expand Down
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsObject/IsRequired.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../../Src/NodeProto.js';
var nodeProto = require('./../../../Src/NodeProto.js');

const propType = nodeProto.propTypes.isObject.isRequired;

Expand Down
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsOptional.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../Src/NodeProto.js';
var nodeProto = require('./../../Src/NodeProto.js');

const propType = nodeProto.propTypes.isOptional;

Expand Down
2 changes: 1 addition & 1 deletion Tests/PropTypes/IsRequired.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import nodeProto from './../../Src/NodeProto.js';
var nodeProto = require('./../../Src/NodeProto.js');

const propType = nodeProto.propTypes.isRequired;

Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@
"testDirectoryName": "Tests",
"unmockedModulePathPatterns": [
"./Src",
"./Dist"
"./Dist",
"./Tests/ExampleComponent.js"
]
}
}

0 comments on commit b6bf002

Please sign in to comment.