When wrapping with Emmet, consider the document language when setting the syntax.

[matt-april]

doWrapping was previously setting syntax = 'html when wrapping with abbreviations. This was resulting in bad class attribute names for jsx files.

• Added a call to an existing function that determines the syntax based on the args parameter passed to doWrapping.
• It's possible for this args parameter to be undefined, so added a guard clause to ensure that it is
  • not undefined, and
  • the language property is set by the current editor.document.languageId.

Now the syntax value is based on the return of getSyntaxFromArgs, and if the result of this function is undefined, default to 'html'.

This fixes #67098.

[ramya-rao-a]

@mattkwiecien Nice! Do you mind adding a test case for this in wrapWithAbbreviation.test.ts file

[matt-april]

Absolutely! Will do either tonight or tomorrow morning.

[msftclas]

All CLA requirements met.

[matt-april]

Added two unit tests for wrapWithAbbreviation and wrapIndividualLinesWithAbbreviation in wrapWithAbbreviation.test.ts for when the file extension is 'jsx'.

While I was in there, I noticed that a lot of the unit tests were re-writing existing logic in the testWrapWithAbbreviation
function (and the newly added testWrapIndividualLinesWithAbbreviation function), so I refactored those tests to call that function instead of copying the same logic within each unit test.

[matt-april]

@octref / @ramya-rao-a
Any more changes needed to make on this PR?

[octref]

Just a small nitpick, looks good to me. Thanks!

[octref]

Awesome contribution, thank you!

[matt-april]

Thanks, really looking forward to contributing more to this great project :)