Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Fix Functional Tests #13585
Fix Functional Tests #13585
Changes from 22 commits
12e6774
5a2c811
11641b8
0300cce
52789ec
3030e5f
1e2ffdc
fd3df04
dc80a9d
b05cc9d
a0d4faa
d07eb90
015fe69
7bb18c5
2ed3181
69e0930
588d0e4
8917b54
8b52d55
fef0796
2ee6574
092452f
2aec165
b5e6359
339848e
69f3fe2
98b3209
47cea9d
ad13fb9
072f2ee
b0266a2
3099c8a
79555d4
874e2cc
0cf3696
321b7f5
b81d1e7
d5f2b53
c8fa43e
7a691c4
965579e
b9a1e75
a6ce51f
0f08ceb
3203cec
f9d8135
74674b0
d506707
abaae61
7ca0f0e
8999ff2
efcb2c2
d6cc9e5
871f9ef
fac7d82
2d8a1cb
3616e48
0aed742
c362f41
9b55de6
398d1b4
828739d
6396940
c141efa
5ea1996
65087f3
8cb8bc3
b4f2ef2
ca8ee49
928b5ec
cd476b0
223832e
9dcee87
0428ef9
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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 tried something similar but it didn't work for me, if that's working then it's great. That's weird though that it broke at some point, maybe they have an issue with their base images.
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.
Could it be part of the js script? Or could the js script try to perfix dotnet with this path (which is what I had tried I think).
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.
Yes, will see
Now it fails because it doesn't find powershell ;)
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 will check
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.
Failing only with SQLServer, not sure why
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.
Because for testing I only updated the SQLServer test
So it means that we would also need to update the PATH for powershell or use directly the available command line in the containers we have initialized.
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.
Why change this new code? It's displaying error messages