NatSpec for MedianOracle

contracts/MedianOracle.sol

@@ -122,9 +122,10 @@ contract MedianOracle is Ownable, IOracle {
         providerReports[providerAddress][0].timestamp=1;
         providerReports[providerAddress][1].timestamp=1;
     }
+
     /**
-    * @notice Computes median of provider reports that their timestamps are in
-    * the valid timestamp range.
+    * @notice Computes median of provider reports whose timestamps are in the
+    * valid timestamp range.
     * @return AggregatedValue: Median of providers reported values.
     *         valid: Boolean indicating an aggregated value was computed successfully.
     */

contracts/lib/Select.sol

@@ -1,10 +1,19 @@
 pragma solidity ^0.4.24;
 import "openzeppelin-solidity/contracts/math/SafeMath.sol";
 
+/**
+ * @title Select
+ * @dev Median Selection Library
+ */
 library Select {
     using SafeMath for uint256;
 
-    // Sorts the input array up to the denoted size, and returns the median.
+    /**
+     * @dev Sorts the input array up to the denoted size, and returns the median.
+     * @param array Input array to compute its median.
+     * @param size Number of elements in array to compute the median for.
+     * @return Median of array.
+     */
     function computeMedian(uint256[] array, uint256 size)
         internal
         pure

test/unit/select.js

@@ -1,5 +1,3 @@
-// TODO(naguib): Fail tests if gas utilization changes
-// TODO(naguib): Consider Adding more test scenarios
 const SelectMock = artifacts.require('SelectMock');
 
 const BigNumber = web3.BigNumber;