From 556812c99e421ba8e334997db228e0851f0cd5b3 Mon Sep 17 00:00:00 2001
From: Muhammad-Altabba <24407834+Muhammad-Altabba@users.noreply.github.com>
Date: Wed, 20 Sep 2023 16:46:10 +0200
Subject: [PATCH] fix and un-skip tests at packages/resolver

---
 packages/resolver/test/globalnpm.ts | 7 +++----
 packages/resolver/test/npm.ts       | 3 +--
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/packages/resolver/test/globalnpm.ts b/packages/resolver/test/globalnpm.ts
index 33a70ed0733..5b930b73efc 100644
--- a/packages/resolver/test/globalnpm.ts
+++ b/packages/resolver/test/globalnpm.ts
@@ -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;
diff --git a/packages/resolver/test/npm.ts b/packages/resolver/test/npm.ts
index e562b61e714..03e8bde594d 100644
--- a/packages/resolver/test/npm.ts
+++ b/packages/resolver/test/npm.ts
@@ -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");