Merge pull request #127 from nodezoo/coverallssearch

Coveralls in search page

Author: mihaidma

CHANGELOG.md

@@ -1,4 +1,4 @@
-* Coveralls integration
+* Integrated Coveralls on search and info pages
 * Set Up Mocha and Instabul
 
 ## 11 April 2016 - 6.1.0

client/assets/css/components/module.styl

@@ -71,6 +71,11 @@
     &.passed
       background-color: #56A93C
 
+.img-badge
+  height: 24px
+  width: auto
+  padding: 1px 0
+
 #query-results-container
   &.panel
     padding: $ui-spacing-half

client/components/module-source.js

@@ -1,21 +1,21 @@
 'use strict'
 
 import React from 'react'
+import classNames from 'classnames'
 
 export const ModuleSource = React.createClass({
   propTypes: {
     data: React.PropTypes.object.isRequired,
-    sourceName: React.PropTypes.string.isRequired
+    sourceName: React.PropTypes.string.isRequired,
+    imgSrc: React.PropTypes.string
   },
   render () {
-    let className = `logo logo-${this.props.sourceName} ${this.props.className}`
-    let source = this.props.data[this.props.sourceName]
-    let url = source ? source.url : ''
+    let {imgSrc, sourceName, className, data: {[sourceName]: source}, data: {[sourceName]: {url: url}}} = this.props
+    let logoClass = classNames({[`logo logo-${sourceName} ${className}`]: !imgSrc})
+    let img = imgSrc && <img src={imgSrc} alt={sourceName} className="img-badge"/>
 
     return (
-      source && source.connected
-        ? <a href={url} target='_blank' className={className} target="_blank"></a>
-        : null
+      source && source.connected && <a href={url} target='_blank' target="_blank" className={logoClass}>{img}</a>
     )
   }
 })

client/components/search-result.js

@@ -31,6 +31,9 @@ export const SearchResult = React.createClass({
             <ModuleSource data={result} sourceName="npm"/>
             {this.hasBuild(result.travis) ? null :
               <ModuleSource data={result} sourceName="travis"/>}
+            {result.coveralls && result.coveralls.badgeUrl &&
+              <ModuleSource data={result} sourceName="coveralls" imgSrc={result.coveralls.badgeUrl}/>
+            }
           </div>
         </div>
         <p className="module-description">

package.json

@@ -60,7 +60,8 @@
     "style-loader": "0.13.x",
     "superagent": "1.6.x",
     "vidi-metrics": "0.1.0",
-    "vidi-seneca-metrics": "0.2.0"
+    "vidi-seneca-metrics": "0.2.0",
+    "classnames": "2.2.x"
   },
   "devDependencies": {
     "babel-core": "6.4.x",