add deletedns unit test

[Marketen]

The current edit() method merges the original service object with the partial updates returned by serviceEditor(service). Merging { ...service, ...updatedFields } doesn't remove properties from service unless they are explicitly overwritten. Properties that aren't included in updatedFields remain in the merged object.

private edit(serviceEditor: (service: ComposeService) => Partial<ComposeService>): void {
    const service = this.parent.compose.services[this.serviceName];
    this.parent.compose.services[this.serviceName] = {
      ...service,
      ...serviceEditor(service)
    };
}

This is why this implementation of removeDns() doesnt work

removeDns(): void {
    this.edit((service) => {
      if (!service.dns) return service;
      // eslint-disable-next-line @typescript-eslint/no-unused-vars
      const { dns, ...rest } = service;
      return rest;
    });
  }

and this one (that deletes the dns directly without passing through edit()) does:

  removeDns(): void {
    const service = this.get();
    if ('dns' in service) {
      delete service.dns;
    }
}  

[github-actions]

🚀 Deployed on https://6731bb5160aa1ad936f45844--objective-borg-9eeae5.netlify.app

[github-actions]

Dappnode bot has built and pinned the built packages to an IPFS node, for commit: 11aab0f

This is a development version and should only be installed for testing purposes.

1. Package dappmanager.dnp.dappnode.eth

Install link

Hash: /ipfs/QmcVbknaH8w5qX22qzw7CqzKc1EpdYH46KkAzY2dBFPYWg

(by dappnodebot/build-action)