Manifest Command Unit Tests

[astarinmymind]

One line in logManifestError is not covered because it is never called by manifest.ts . This is because the error parameter is optional and it is never used throughout the program.

[astarinmymind]

Now you're cooking with gas! 😄

One general comment in addition to the inline: would it be possible to spy on e.g. logManifestWarning and logManifestError instead console.warn and console.error? This is a genuine question; we don't want to do that at the expense of too much extra work.

This is because logManifestWarning and logManifestError are internal functions. SpyOn is used to mock functions from other modules.

[rekmarks]

This is because logManifestWarning and logManifestError are internal functions. SpyOn is used to mock functions from other modules.

Oh, I thought they were in a different file. That's fine then, let's just keep things as they are!