Added test id for menu

[SreehariJayaraj]

• Fixes Add data-cy label for Menu component #1015

Description

Checklist

• I have made corresponding changes to the documentation.
• I have updated the types definition of modified exports.
• I have verified the functionality in some of the neeto web-apps.
• I have added the necessary label (patch/minor/major - If package publish
  is required).

I have added data cy labels for all the menu buttons

Reviewers

[VarunSriram99]

@sreehari2003 A few questions,

Questions

• Why are we going through all these steps for adding a data-cy label? Button has option for other props. So just add the "data-cy" label in the buttonProps?
• Why are we adding a data-test-id label when we use data-cy labels in our tests?
• Even if you're adding a test-id, it should data-testid and not data-test-id
• Why are we not using the hyphenize utlity from neetoCist?

[SreehariJayaraj]

@sreehari2003 A few questions,

Questions

• Why are we going through all these steps for adding a data-cy label? Button has option for other props. So just add the "data-cy" label in the buttonProps?
• Why are we adding a data-test-id label when we use data-cy labels in our tests?
• Even if you're adding a test-id, it should data-testid and not data-test-id
• Why are we not using the hyphenize utlity from neetoCist?

hyphenate is not available to import

[VarunSriram99]

@sreehari2003 _a Please make sure you're taking care of all the review suggestions before re-assigning.

[VarunSriram99]

@sreehari2003 A few questions,
Questions

• Why are we going through all these steps for adding a data-cy label? Button has option for other props. So just add the "data-cy" label in the buttonProps?
• Why are we adding a data-test-id label when we use data-cy labels in our tests?
• Even if you're adding a test-id, it should data-testid and not data-test-id
• Why are we not using the hyphenize utlity from neetoCist?

hyphenate is not available to import

@sreehari2003 its hyphenate. Please go through all the neeto-cist and Playwright commons utilities carefully to make sure we're not repeating the logic.

[SreehariJayaraj]

• This Issue needs to be fixed first Fix all the audit failures #1027

[VarunSriram99]

@sreehari2003 _a LGTM