Skip to content

Fix merge-junit-xml and merge-nunit-xml issue#2235

Merged
AnatolyPristensky merged 3 commits intomicrosoft:masterfrom
ankasani:patch-1
Mar 14, 2023
Merged

Fix merge-junit-xml and merge-nunit-xml issue#2235
AnatolyPristensky merged 3 commits intomicrosoft:masterfrom
ankasani:patch-1

Conversation

@ankasani
Copy link
Copy Markdown
Contributor

@ankasani ankasani commented Mar 9, 2023

Using the option --merged-file-name "output.xml" --test-output-dir JunitTestResult, facing the below issue.

Same for --merge-junit-xml "output.xml" --test-output-dir JunitTestResult

i can see .zip was was downloaded but merged files is not working!

Downloaded artifacts to JunitTestResult\junit_xml_zip.zip

Error: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of Document

Similar issue with nunit for xamarin uitest: #1814

Using the option  --merged-file-name "output.xml" --test-output-dir JunitTestResult, facing the below issue, i can see .zip was was downloaded but merged files is not working!

Downloaded artifacts to JunitTestResult\junit_xml_zip.zip
Error: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of Document

Similer issue with nunit for xamarin uitest: #1814
@ankasani ankasani requested a review from a team as a code owner March 9, 2023 14:42
@ankasani ankasani changed the title Fix merge-nunit-xml issue Fix merge-junit-xml issue Mar 9, 2023
Using the option merged-file-name "output.xml" --test-output-dir JunitTestResult, facing the below issue, i can see .zip was was downloaded but merged files is not working!

Downloaded artifacts to JunitTestResult\junit_xml_zip.zip

Error: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of Document
@ankasani ankasani changed the title Fix merge-junit-xml issue Fix merge-junit-xml and merge-nunit-xml issue Mar 9, 2023
@AnatolyPristensky
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@AnatolyPristensky
Copy link
Copy Markdown
Contributor

Hello, @ankasani,
CI is failing with the following error message:

/home/vsts/work/1/s/src/commands/test/run/appium.ts
62:5 error Delete ···· prettier/prettier
63:1 error Delete ···· prettier/prettier

Could you please have a look?

/home/vsts/work/1/s/src/commands/test/run/appium.ts
62:5 error Delete ···· prettier/prettier
63:1 error Delete ···· prettier/prettier

Issue was with file format, fix it!
@ankasani
Copy link
Copy Markdown
Contributor Author

@AnatolyPristensky , fixed it can you check now!

@AnatolyPristensky AnatolyPristensky merged commit ec234e7 into microsoft:master Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants