Skip to content

Commit

Permalink
fix coverage for viem
Browse files Browse the repository at this point in the history
  • Loading branch information
fritx committed Apr 10, 2024
1 parent 5dd861b commit 61fc488
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,15 @@ java -jar target/playground-1.0-SNAPSHOT.jar
# Solidity
npm install # or pnpm install
REPORT_GAS=true npx hardhat test
npx hardhat coverage
# `SOLIDITY_COVERAGE=true` is required to fix for viem
SOLIDITY_COVERAGE=true npx hardhat coverage
```

Roadmap:
- [ ] benchmarks
- [ ] bigdata & spark & hadoop
- [ ] [lang cheatsheet](./lang_cheatsheet.md)
- [x] fix coverage for viem
- [x] solidity & hardhat
- [ ] c & c++ testing
- [x] mixed c & c++ with go
Expand Down
7 changes: 4 additions & 3 deletions test/Lock_viem.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
// Quick Start - hardhat.org
// https://hardhat.org/hardhat-runner/docs/getting-started#quick-start
import {
time,
loadFixture,
Expand All @@ -8,7 +6,10 @@ import { expect } from "chai";
import hre from "hardhat";
import { getAddress, parseGwei } from "viem";

describe.skip("Lock (viem-test) not passing w/ coverage", function () {
// Run gas report and coverage in e2e #4961
// https://github.com/NomicFoundation/hardhat/pull/4961
// describe.skip("Lock (viem)", function () {
describe("Lock (viem)", function () {
// We define a fixture to reuse the same setup in every test.
// We use loadFixture to run this setup once, snapshot that state,
// and reset Hardhat Network to that snapshot in every test.
Expand Down

0 comments on commit 61fc488

Please sign in to comment.