Oracle v2 dev (#256)

* allows report to be pushed if past is expired (naguib's original change)

* tests median oracle push report for all expiration/delay combinations

* adds one more push report test to oracle

* Apply suggestions from code review

Author: nms-7

contracts/MedianOracle.sol

@@ -108,13 +108,21 @@ contract MedianOracle is OwnableUpgradeable, IOracle {
         Report[2] storage reports = providerReports[providerAddress];
         uint256[2] memory timestamps = [reports[0].timestamp, reports[1].timestamp];
 
+        // Checks that this providerAddress is already whitelisted
         require(timestamps[0] > 0);
 
         uint8 index_recent = timestamps[0] >= timestamps[1] ? 0 : 1;
         uint8 index_past = 1 - index_recent;
 
+        uint256 minValidTimestamp = block.timestamp - reportExpirationTimeSec;
+        uint256 maxValidTimestamp = block.timestamp - reportDelaySec;
+
         // Check that the push is not too soon after the last one.
-        require(timestamps[index_recent] + reportDelaySec <= block.timestamp);
+        // unless past one is already expired
+        require(
+            timestamps[index_past] < minValidTimestamp ||
+                timestamps[index_recent] <= maxValidTimestamp
+        );
 
         reports[index_past].timestamp = block.timestamp;
         reports[index_past].payload = payload;
@@ -127,6 +135,7 @@ contract MedianOracle is OwnableUpgradeable, IOracle {
      */
     function purgeReports() external {
         address providerAddress = msg.sender;
+        // Check that this providerAddress is already whitelisted
         require(providerReports[providerAddress][0].timestamp > 0);
         providerReports[providerAddress][0].timestamp = 1;
         providerReports[providerAddress][1].timestamp = 1;

test/unit/MedianOracle.ts

@@ -92,11 +92,55 @@ describe('MedianOracle:pushReport', async function () {
   it('should only push from authorized source', async function () {
     await expect(oracle.connect(A).pushReport(payload)).to.be.reverted
   })
-  it('should fail if reportDelaySec did not pass since the previous push', async function () {
+
+  it('should pass if reportDelaySec did not pass since the previous push, but previous report non-existent', async function () {
+    await oracle.addProvider(await A.getAddress())
+    await oracle.connect(A).pushReport(payload)
+    await oracle.connect(A).pushReport(payload)
+  })
+
+  it('should pass if reportDelaySec did pass since the previous push, and previous report non-existent', async function () {
     await oracle.addProvider(await A.getAddress())
     await oracle.connect(A).pushReport(payload)
+    await increaseTime(20)
+    await oracle.connect(A).pushReport(payload)
+  })
+
+  it('should pass if reportDelaySec did pass since the previous push, but previous report expired', async function () {
+    await oracle.addProvider(await A.getAddress())
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(70)
+    await oracle.connect(A).pushReport(payload)
+    await oracle.connect(A).pushReport(payload)
+  })
+
+  it('should pass if reportDelaySec did pass since the previous push, and previous report expired', async function () {
+    await oracle.addProvider(await A.getAddress())
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(70)
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(20)
+    await oracle.connect(A).pushReport(payload)
+  })
+
+  it('should fail if reportDelaySec did not pass since the previous push, and previous report not expired', async function () {
+    await oracle.addProvider(await A.getAddress())
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(20)
+    await oracle.connect(A).pushReport(payload)
+    await expect(oracle.connect(A).pushReport(payload)).to.be.reverted
+  })
+
+  it('should pass if reportDelaySec did pass since the previous push, and previous report not expired', async function () {
+    await oracle.addProvider(await A.getAddress())
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(20)
+    await oracle.connect(A).pushReport(payload)
+    await increaseTime(20)
+    await oracle.connect(A).pushReport(payload)
     await expect(oracle.connect(A).pushReport(payload)).to.be.reverted
   })
+
   it('should emit ProviderReportPushed message', async function () {
     oracle.addProvider(await A.getAddress())
     const tx = await oracle.connect(A).pushReport(payload)