specron / framework Goto Github PK

This allows us to make stability guarantees and also to use the ^ operator when installing Specron.

Similar to how ctx.reverts works a functionality to check for a throw would be convenient.

Open zeppelin example

Example:

{ // package.json
  ...
  "specron": {
    "require": [
      "ts-node/register"
    ]
  }
}

https://github.com/monpo/monorepo

Please prepare e logo for this organization in vectors and bitmaps and places them under /assets in master branch.

Consider changing default test command to:

rm -rf build && specron compile && specron test --require ts-node/register --match ./src/tests/**/*.test.ts

Once the #7 issue is resolved, set up a custom domain name specron.org for the website.

Compiler throws warning: Warning: SPDX license identifier not provided in source file. Before publishing, consider adding a comment containing "SPDX-License-Identifier: <SPDX-License>" to each source file. Use "SPDX-License-Identifier: UNLICENSED" for non-open-source code. Please see https://spdx.org for more information. even if license is correctly set.

We could probably upgrade the Reporter to print messages in NodeJS style. Currently, the design follows the MochaJS output style.

It would be nice to include that the user has to install web3 dependency(and which version) himself otherwise specron will not work.

This is a problem with package solidity-parser-antlr with is a dependency of sol-merger which flattener depends on.

Can someone please run

$ mkdir myProject
$ cd myProject
Initialize the project and install the dependencies.

$ specron init
$ npm install
Run tests to verify everything works as expected.

$ npm test

and then save that to specron/hello-world

Also add Travis integration.

This will help me quickly see if specron is right for me.

I have already complained to Truffle about the magic and boilerplates required there so specron could be a fresh relief!

trufflesuite/truffle#1382

When compiling solidity code if there are compiler warnings there are treated and shown as errors.

Example:

Use GitHub Pages to create a custom website template for this project.

Specron init creates files with code from the previous specron version in which there are breaking changes like: requireContract -> deploy so the default example does not work.

Add a new method signature on Context which returns signed data as shown below. For now only the web3.eth.sign will be supported.

import { Spec } from '@hayspec/spec';
import * as index from '..';

/**
 * Helper method.
 */

interface SignatureConfig {
  claim: string;
  signer: string;
  kind: number;
}

async function sign(cfg: SignatureConfig) {
  const signature = await ctx.web3.eth.sign(cfg.claim, cfg.signer);
  const signatureData = {
    r: signature.substr(0, 66),
    s: `0x${signature.substr(66, 64)}`,
    v: parseInt(`0x${signature.substr(130, 2)}`) + 27,
    kind: cfg.kind,
  };
  return ctx.tuple(signatureData);
}

/**
 * Testing module interface.
 */

const spec = new Spec();

spec.test('isHay() returns true', async (ctx) => {
  const signature = await sign({ 
    claim: '0x6baf6cc2efd4500d6531c276870153f10922e4d956240f7170711d79a67bb34e', 
    signer: '0x726bAa6F5caDFBa3cBf1596598656099693910e1',
    kind: 0,
  });
  ctx.is(signature.r, '0xa7c39c2c4850bf1602a61f4764ab895020a8b457752c6d05d8a852228dbd643d');
  ctx.is(signature.s, '0x1dd762054249d3c9ec92df34b6053c0710f9ab213ace30bf1cf9b05cf44b5da2');
  ctx.is(signature.v, '27');
});

export default spec;

Currently they are out of sync.

Perhaps delete all GitHub releases.

Or find a way to have them up-to-date with NPM releases.

Or use one of the fancy GitHub actions to do all that for us.

Current Specron uses old version

If you are checking revert message and the expected result does not match the actual error display the actual error.

Some errors fail with no information where a problem actually accrued. We should start by wrapping tests into try/catch.

It would be cool to have a specron flatten command that would flatten all the smart contracts and dependencies into one file.

The maximum gas value should be set automatically so a user doesn't have to set this property manually.