Fix/streamonomics step zero

contracts/PoolStreamer2.sol

@@ -256,7 +256,7 @@ contract PoolStreamer2 is AccessControl {
         // downgrade amount from super token to underlying token
         _superToken(pool).downgrade(amount);
         // transfer amount to msg.sender
-        _superToken(pool).transfer(msg.sender, amount);
+        _underlyingToken(pool).transfer(msg.sender, amount);
     }
 
     function superToken(ISuperFluidPool pool) external view returns (address) {
@@ -288,4 +288,4 @@ contract PoolStreamer2 is AccessControl {
     }
 
 }
-
+

contracts/Streamonomics.sol

@@ -32,6 +32,7 @@ contract Streamonomics is Ownable {
         delete streamonomics;
         uint256 total;
         for(uint i = 0; i < percentage.length; i++) {
+            require(step[i] > 0, "!step");
             streamonomics.push(Streamonomic(percentage[i], start[i], step[i], limit[i]));
             emit StreamonomicAdded(percentage[i], start[i], step[i], limit[i]);
             total += percentage[i];
@@ -43,4 +44,4 @@ contract Streamonomics is Ownable {
         return streamonomics;
     }
 
-}
+}

test/streamonomics.js

@@ -0,0 +1,18 @@
+const { expect } = require("chai");
+const { ethers } = require("hardhat");
+
+describe("Streamonomics", function () {
+  it("reverts when step is zero", async function () {
+    const Streamonomics = await ethers.getContractFactory("Streamonomics");
+    const stream = await Streamonomics.deploy();
+    await stream.deployed();
+
+    const percentage = [10];
+    const start = [1];
+    const step = [0];
+    const limit = [1];
+
+    await expect(stream.setStreamonomics(percentage, start, step, limit))
+      .to.be.revertedWith("!step");
+  });
+});