kredits/contracts
kredits/contracts
9
1
Fork 0
Code Issues 21 Pull Requests Actions Projects 1 Releases 8 Activity

Contributor app tests #143

Merged
haythem96 merged 26 commits from tests/contracts-contributor into master 2019-07-22 06:55:23 +00:00
Conversation 39 Commits 26 Files Changed 17 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 924cfedad4 - Show all commits

4
apps/contributor/test/contributor.js
View File

@ -66,8 +66,8 @@ contract('Contributor app', (accounts) => {
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
});
describe("Owner default space permissions", async () => {
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
it('check owner is token issuer', async () => {
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
describe("Owner default permissions", async () => {
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
it('check owner is contributors manager', async () => {
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
let manageContributorPermission = await acl.hasPermission(root, contributor.address, await contributor.MANAGE_CONTRIBUTORS_ROLE());
// eslint-disable-next-line no-undef
assert.equal(manageContributorPermission, true);

bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
bumi commented 2019-07-06 15:22:31 +00:00 (Migrated from github.com)
Review
Copy Link

we should be able to use the namehash from ethers:
const namehash = require('ethers').utils.namehash;
as ethers.js is already a dependency we do not need to add another dependency

we should be able to use the namehash from ethers: `const namehash = require('ethers').utils.namehash;` as ethers.js is already a dependency we do not need to add another dependency
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:07:41 +00:00 (Migrated from github.com)
Review
Copy Link

cool will change that

cool will change that
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...
haythem96 commented 2019-07-06 16:10:49 +00:00 (Migrated from github.com)
Review
Copy Link

for the ganache script, it is basically the same aragon one, but I changed

run_tests() {
  echo "Running tests..."
  npx truffle test --network rpc "$@"
}

to

run_tests() {
  echo "Running tests..."
  npx aragon contracts test --network rpc "$@"
}

Because npx truffle test --network rpc "$@" didn't work for me for no reason...

for the ganache script, it is basically the same aragon one, but I changed ``` run_tests() { echo "Running tests..." npx truffle test --network rpc "$@" } ``` to ``` run_tests() { echo "Running tests..." npx aragon contracts test --network rpc "$@" } ``` Because `npx truffle test --network rpc "$@"` didn't work for me for no reason...