Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Commit

Permalink
fix and un-skip tests at packages/resolver
Browse files Browse the repository at this point in the history
  • Loading branch information
Muhammad-Altabba committed Sep 20, 2023
1 parent 948ebc9 commit 556812c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
7 changes: 3 additions & 4 deletions packages/resolver/test/globalnpm.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,14 @@ import assert from "assert";
const detectInstalled: any = require("detect-installed");
const getInstalledPath: any = require("get-installed-path");
import * as sinon from "sinon";
import * as path from "path";
import * as fs from "fs";
import path from "path";
import fs from "fs";
import { describe, it } from "mocha";

import { GlobalNPM } from "../lib/sources/globalnpm";
const globalNpm = new GlobalNPM();

//todo web3js this fails but doesn't seem to be related (is it?) to the migration. `.require` returns null instead of `{}`
describe.skip("globalnpm", () => {
describe("globalnpm", () => {
describe("require function", () => {
let syncStub: sinon.SinonStub;
let getInstalledPathSyncStub: sinon.SinonStub;
Expand Down
3 changes: 1 addition & 2 deletions packages/resolver/test/npm.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@ import assert from "assert";
import { NPM } from "../lib/sources/npm";
const npm = new NPM("");

//todo web3js this fails but doesn't seem to be related (is it?) to the migration. `.require` returns null instead of `{}`
describe.skip("npm", function () {
describe("npm", function () {
describe("#require()", function () {
it("returns file contents from artifacts in build/contracts", function () {
let result = npm.require("package/Test", "./test/fixtures");
Expand Down

0 comments on commit 556812c

Please sign in to comment.