fix: address code review

compiler/noirc_frontend/src/graph/mod.rs

@@ -93,11 +93,8 @@ mod crate_name {
     }
 
     #[test]
-    fn it_can_serialize_and_deserialize() {
-        let crate_name = "test_crate".parse::<CrateName>().unwrap();
-        let serialized = serde_json::to_string(&crate_name).unwrap();
-        let deserialized = serde_json::from_str::<CrateName>(&serialized).unwrap();
-        assert_eq!(crate_name, deserialized);
+    fn it_rejects_bad_crate_names_when_deserializing() {
+        assert!(serde_json::from_str::<CrateName>("bad-name").is_err());
     }
 }
 

compiler/wasm/test/node/index.test.ts

@@ -12,9 +12,6 @@ import { join, resolve } from 'node:path';
 import { compile } from '@noir-lang/noir_wasm';
 import { initializeResolver } from '@noir-lang/source-resolver';
 
-// const absoluteNoirSourcePath = ;
-// const absoluteNargoArtifactPath = join(__dirname, nargoArtifactPath);
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 async function getPrecompiledSource(path: string): Promise<any> {
   const compiledData = readFileSync(resolve(__dirname, path)).toString();
@@ -36,6 +33,7 @@ describe('noir wasm compilation', () => {
 
   describe('can compile scripts with dependencies', () => {
     beforeEach(() => {
+      // this test requires a custom resolver in order to correctly resolve dependencies
       initializeResolver((file) => {
         switch (file) {
           case '/script/main.nr':