os.Remove: avoid double-wrapping err; fixes TODO in test

src/os/file.go

@@ -47,7 +47,7 @@ func Remove(path string) error {
 	}
 	err := fs.Remove(suffix)
 	if err != nil {
-		return &PathError{"remove", path, err}
+		return err
 	}
 	return nil
 }

src/os/os_unix_test.go

@@ -63,9 +63,8 @@ func TestRemove(t *testing.T) {
 				t.Errorf("TestRemove: PathError returned path %q, expected %q", pe.Path, f)
 			}
 		}
-		// TODO: make this pass.
 		if !IsNotExist(err) {
-			t.Logf("TestRemove: TODO: expected IsNotExist(err) true, got false; err %q", err.Error())
+			t.Errorf("TestRemove: expected IsNotExist(err) true, got false; err %q", err.Error())
 		}
 	}