Support contracts with the same name #1203

[melnikaite]

No description provided.

[jbcarpanelli]

Hey @melnikaite, thanks for tackling this one! Let me know when it's ready for review 👍.

[melnikaite]

Hi @jcarpanelli

I manage to successfully run oz push, but stuck with oz create, because it try to install contract/name as npm dependency, but obviously there is no such npm package and I didn't find the place where sdk check if it should run npm install

[spalladino]

Hey @melnikaite! Thanks for the contribution :-)
Is this the function you are looking for regarding installing an npm dependency?

openzeppelin-sdk/packages/cli/src/models/dependency/Dependency.ts

Lines 69 to 74 in 5140c69

	public static async install(nameAndVersion: string): Promise<Dependency> {
	Loggy.spin(__filename, 'install', `install-dependency-${nameAndVersion}`, `Installing ${nameAndVersion} via npm`);
	await npm.install([nameAndVersion], { save: true, cwd: process.cwd() });
	Loggy.succeed(`install-dependency-${nameAndVersion}`, `Dependency ${nameAndVersion} installed`);
	return this.fromNameWithVersion(nameAndVersion);
	}

[melnikaite]

Yes, but in fact my issue was in fromContractFullName

It seems functionality works fine for all commands, but currently I don't have time to fix all the tests