Breaking dangerous superuser burn into mock and test mocks.

Author: MoMannn

README.md

@@ -114,19 +114,6 @@ contract MyArtSale is
     super._setTokenUri(_tokenId, _uri);
   }
 
-  /**
-   * @dev Removes a NFT from owner.
-   * @param _tokenId Which NFT we want to remove.
-   */
-  function burn(
-    uint256 _tokenId
-  )
-    external
-    onlyOwner
-  {
-    super._burn(_tokenId);
-  }
-
 }
 ```
 

src/contracts/mocks/nf-token-mock.sol

@@ -26,17 +26,4 @@ contract NFTokenMock is
     super._mint(_to, _tokenId);
   }
 
-  /**
-   * @dev Removes a NFT from owner.
-   * @param _tokenId Which NFT we want to remove.
-   */
-  function burn(
-    uint256 _tokenId
-  )
-    external
-    onlyOwner
-  {
-    super._burn(_tokenId);
-  }
-
 }

src/tests/mocks/nf-token-test-mock.sol

@@ -0,0 +1,22 @@
+pragma solidity 0.5.2;
+
+import "../../contracts/mocks/nf-token-mock.sol";
+
+contract NFTokenTestMock is
+  NFTokenMock
+{
+
+ /**
+   * @dev Removes a NFT from owner.
+   * @param _tokenId Which NFT we want to remove.
+   */
+  function burn(
+    uint256 _tokenId
+  )
+    external
+    onlyOwner
+  {
+    super._burn(_tokenId);
+  }
+
+}

src/tests/tokens/nf-token.test.ts

@@ -39,8 +39,8 @@ spec.beforeEach(async (ctx) => {
 
 spec.beforeEach(async (ctx) => {
   const nfToken = await ctx.deploy({ 
-    src: './build/nf-token-mock.json',
-    contract: 'NFTokenMock',
+    src: './build/nf-token-test-mock.json',
+    contract: 'NFTokenTestMock',
   });
   ctx.set('nfToken', nfToken);
 });