-
Notifications
You must be signed in to change notification settings - Fork 2
Adds test automation and fixes misc. readme issues #38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🧪 Validation Test Results
Workflow run: 20761734863 |
🧪 Validation Test ResultsWorkflow run: 20761995216 |
5870d34 to
7eafd6e
Compare
🧪 Validation Test ResultsWorkflow run: 20762294078 |
7eafd6e to
626bdf1
Compare
🧪 Validation Test ResultsWorkflow run: 20762597414 |
626bdf1 to
9c70074
Compare
🧪 Validation Test ResultsWorkflow run: 20762686345 |
| if trx_file: | ||
| tree = ET.parse(trx_file) | ||
| root = tree.getroot() | ||
| ns = {'ns': 'http://microsoft.com/schemas/VisualStudio/TeamTest/2010'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this script do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still iterating on this- the current form is not working.
In short- Trying to unify the test runs for all languages to have the same output. This would allow me to call out "Test Foo" is passing in Javascript and Python, but not Dotnet. The goal of this script is to comment on the PR with a matrix of tests x languages.
trx is a Visual Studio test result file that supposedly I can get each language to output. While it was working locally- it doesn't appear to be working within the Github Action.
In short- this needs to be changed.
🧪 Validation Test ResultsWorkflow run: 20868428871 |
No description provided.