Adds command listitem attachment add

[MathijsVerbeeck]

Closes #5219

[milanholemans]

Thank you @MathijsVerbeeck we'll try to review it ASAP!

[Jwaegebaert]

Hey @MathijsVerbeeck, would you mind updating your PR to utilize ESM? Here's a breakdown of the new changes:

• Make sure that all logging is done asynchronously and awaited

  • cli-microsoft365/src/m365/spo/commands/listitem/listitem-attachment-list.ts

    Line 126 in 515b009

    await logger.log(attachmentFiles.AttachmentFiles);

  • cli-microsoft365/src/m365/spo/commands/listitem/listitem-attachment-list.spec.ts

    Lines 65 to 81 in 515b009

    	beforeEach(() => {
    	log = [];
    	logger = {
    	log: async (msg: string) => {
    	log.push(msg);
    	},
    	logRaw: async (msg: string) => {
    	log.push(msg);
    	},
    	logToStderr: async (msg: string) => {
    	log.push(msg);
    	}
    	};
    	loggerLogSpy = sinon.spy(logger, 'log');
    	loggerLogToStderrSpy = sinon.spy(logger, 'logToStderr');
    	sinon.stub(cli, 'getSettingWithDefaultValue').callsFake(((settingName, defaultValue) => defaultValue));
    	});

• Adjust the way commands are exported
  

  cli-microsoft365/src/m365/spo/commands/listitem/listitem-attachment-list.ts

  Line 140 in 515b009

  export default new SpoListItemAttachmentListCommand();

• Replace all instances of require with import
  

  cli-microsoft365/src/m365/spo/commands/listitem/listitem-attachment-list.spec.ts

  Line 16 in 515b009

  import command from './listitem-attachment-list.js';

• Modify all file imports within the project to have a .js extension

[Jwaegebaert]

Nice work @MathijsVerbeeck, I've left a few suggestions you can take a look at.

I've also noticed a bit of a vague error. When you want to upload a file with a name that already exists you get this error:

Error: The specified name is already in use.

The document or folder name was not changed.  To change the name to a different value, close this dialog and edit the properties of the document or folder.

Could we modify this so the message is clearer and doesn't return any steps that aren't relevant to CLI usage?

[Jwaegebaert]

Splendid work @MathijsVerbeeck! Noticed a small detail in the examples but resolved that one myself during the merging.

[Jwaegebaert]

For fileName, we don't need to define the extension as this is done in code.

Suggested change

	m365 spo listitem attachment add --webUrl https://contoso.sharepoint.com/sites/project-x --listUrl "/sites/project-x/Lists/DemoList" --listItemId 147 --filePath "C:/Reports/File1.jpg" --fileName "File2.jpg"
	m365 spo listitem attachment add --webUrl https://contoso.sharepoint.com/sites/project-x --listUrl "/sites/project-x/Lists/DemoList" --listItemId 147 --filePath "C:/Reports/File1.jpg" --fileName "File2"