[Step Extension] Fix parsing of xml step extension

[poorna2152]

Purpose

Update parsing of xml step extension to match the spec.
https://ballerina.io/spec/lang/master/#xml-step-extend

Check List

• Read the Contributing Guide
• Updated Change Log
• Checked Tooling Support (#)
• Added necessary tests
  • Unit Tests
  • Spec Conformance Tests
  • Integration Tests
  • Ballerina By Example Tests
• Increased Test Coverage
• Added necessary documentation
  • API documentation
  • Module documentation in Module.md files
  • Ballerina By Examples

[codecov]

Codecov Report

Attention: Patch coverage is 78.90295% with 50 lines in your changes missing coverage. Please review.

Please upload report for BASE (xml_step_extension@0c727ce). Learn more about missing BASE report.

❗ Current head cc8f3ec differs from pull request most recent head 18206e3

Please upload reports for the commit 18206e3 to get more accurate results.

Files	Patch %	Lines
...o/ballerina/compiler/syntax/tree/TreeModifier.java	0.00%	16 Missing ⚠️
.../compiler/internal/parser/tree/STTreeModifier.java	0.00%	9 Missing ⚠️
...ternal/parser/tree/STXMLStepIndexedExtendNode.java	58.82%	7 Missing ⚠️
...nal/parser/tree/STXMLStepMethodCallExtendNode.java	58.82%	7 Missing ⚠️
...a/compiler/internal/parser/tree/STNodeVisitor.java	0.00%	4 Missing ⚠️
...mpiler/internal/parser/tree/STNodeTransformer.java	0.00%	2 Missing ⚠️
...allerina/compiler/syntax/tree/NodeTransformer.java	0.00%	2 Missing ⚠️
...r/internal/parser/BallerinaParserErrorHandler.java	94.73%	1 Missing ⚠️
...compiler/syntax/tree/XMLStepIndexedExtendNode.java	96.55%	1 Missing ⚠️
...piler/syntax/tree/XMLStepMethodCallExtendNode.java	96.55%	1 Missing ⚠️

Additional details and impacted files

@@                  Coverage Diff                  @@
##             xml_step_extension   #42722   +/-   ##
=====================================================
  Coverage                      ?   77.16%           
  Complexity                    ?    51257           
=====================================================
  Files                         ?     2928           
  Lines                         ?   204223           
  Branches                      ?    26628           
=====================================================
  Hits                          ?   157583           
  Misses                        ?    38126           
  Partials                      ?     8514           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

This PR has been open for more than 15 days with no activity. This will be closed in 3 days unless the stale label is removed or commented.

[lochana-chathura]

function-call-expr vs method-call-expr: they have slightly different grammar.
e.g.

• function call allows qualified identifier. Method call does not allow it.
• method-name allows special-method-name := map | join | start. Function reference does not allow it

So I think we better parse . method-name ( arg-list ) items separately.
i.e.

parseMethodCallExprRhs() {
   parseDotToken()
   parseMethodName()
   parseParenthesizedArgList()
}

[lochana-chathura]

are the years manually updated? You need to add an entry to syntax_node_metadata.json. It will automatically pick the year from there.