Add additional SystemWeb HttpRequset properties to C# test stubs#20513
Add additional SystemWeb HttpRequset properties to C# test stubs#20513michaelnebel merged 6 commits intogithub:mainfrom
Conversation
michaelnebel
left a comment
There was a problem hiding this comment.
Uh, thank you for doing this! 😄
Stub should not include code logic.
|
It looks like the change to the stubs doesn't compile with all the tests. |
|
@michaelnebel I don't have access to see which tests exactly in the "C# Language Tests Linux" pipeline is failing, I have started trying to kick them all off manually locally on my machine but I expect this process to take quite a while. I have found the "TaintedPath.qlref" test is failing which seems to be related to the .expected steps changing from this PR, but if you could provide me a full list so I can verify that would save me some time and hassle. Thank you! |
Oh, that makes it a lot harder to debug. Thank you for your persistence! |
|
@michaelnebel Thank you for the commit, I cherry picked it and have added it to the PR. Fingers crossed! |
Add additional HttpRequest properties to the System.Web.cs test stubs.