Merge pull request #10 from macklinu/pull-requests-title

Provide optional PullRequests title

Author: Raphaël Benitte

src/components/PullRequests.jsx

@@ -33,11 +33,18 @@ class PullRequests extends Component {
 
     render() {
         let { pullRequests } = this.state;
+        let { repository, title } = this.props
+
+        let titleNode = title === undefined ? (
+            <span>
+                <span className="widget__header__subject">{repository}</span> Pull Requests
+            </span>
+        ) : title;
 
         return (
             <div>
                 <div className="widget__header">
-                    Pull requests
+                    {titleNode}
                     <span className="widget__header__count">
                         {pullRequests.length}
                     </span>
@@ -54,7 +61,8 @@ class PullRequests extends Component {
 }
 
 PullRequests.propTypes = {
-    repository: PropTypes.string.isRequired
+    repository: PropTypes.string.isRequired,
+    title:      PropTypes.string
 };
 
 reactMixin(PullRequests.prototype, ListenerMixin);

src/tests/PullRequests.test.js

@@ -7,6 +7,8 @@ import mockery      from 'mockery';
 var PullRequests;
 var pullRequests;
 
+const repository = 'plouc/mozaik';
+
 describe('Github — PullRequests', () => {
 
     let sandbox;
@@ -23,7 +25,7 @@ describe('Github — PullRequests', () => {
 
     beforeEach(() => {
         sandbox = sinon.sandbox.create();
-        pullRequests = TestUtils.renderIntoDocument(<PullRequests repository="plouc/mozaik" />);
+        pullRequests = TestUtils.renderIntoDocument(<PullRequests repository={repository} />);
     });
 
     afterEach(() => {
@@ -38,9 +40,9 @@ describe('Github — PullRequests', () => {
 
     it('should return correct api request', () => {
         expect(pullRequests.getApiRequest()).to.eql({
-            id:     'github.pullRequests.plouc/mozaik',
+            id:     `github.pullRequests.${repository}`,
             params: {
-                repository: 'plouc/mozaik'
+                repository: repository
             }
         });
     });
@@ -80,4 +82,16 @@ describe('Github — PullRequests', () => {
         let count = TestUtils.findRenderedDOMComponentWithClass(pullRequests, 'widget__header__count');
         expect(count.getDOMNode().textContent).to.equal('3');
     });
-});
+
+    it('renders default title `repository Pull Requests`', () => {
+        let title = TestUtils.findRenderedDOMComponentWithClass(pullRequests, 'widget__header');
+        expect(title.getDOMNode().textContent).to.contain(`${repository} Pull Requests`);
+    });
+
+    it('renders custom title when supplied', () => {
+        let customTitle = 'Custom Title';
+        pullRequests = TestUtils.renderIntoDocument(<PullRequests repository={repository} title={customTitle} />)
+        let title = TestUtils.findRenderedDOMComponentWithClass(pullRequests, 'widget__header');
+        expect(title.getDOMNode().textContent).to.contain(customTitle);
+    });
+});