Skip to content

Allow aliases in Parameter Filters blocks#1240

Merged
nohwnd merged 8 commits intopester:masterfrom
renehernandez:allow-parameter-aliases
Feb 13, 2019
Merged

Allow aliases in Parameter Filters blocks#1240
nohwnd merged 8 commits intopester:masterfrom
renehernandez:allow-parameter-aliases

Conversation

@renehernandez
Copy link
Copy Markdown
Contributor

1. General summary of the pull request

Fix #1128

Rene Hernandez added 3 commits February 7, 2019 15:19
Remove top-level brackets from final bracket
Fix position calculations for start and end of variable
Comment thread Functions/Mock.ps1
@renehernandez
Copy link
Copy Markdown
Contributor Author

@nohwnd 21 tests failed in the powershell 3 job. They are related to Gherkin, which are not related to these changes. I looked in the job history and those Gherkin tests failed in the job associated to the previous pull request as well.

@renehernandez
Copy link
Copy Markdown
Contributor Author

The powershell 2 job is apparently stuck waiting for an agent to run, but these changes shouldn't be validated against powershell 2 anyways. Any inputs?

@renehernandez
Copy link
Copy Markdown
Contributor Author

Ok I realized now that the pull request should be targeted against v5 branch instead of master. Will close and open a new pull request

@renehernandez
Copy link
Copy Markdown
Contributor Author

Ok I realized now that the pull request should be targeted against v5 branch instead of master. Will close and open a new pull request

@nohwnd Actually I am not sure how to proceed here. I checked the latest status of mocking in v5 and it doesn't have yet the previous mocking fixes that I have implemented

@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 9, 2019

@renehernandez This should target the master, not v5. The agent running powershell v2 is stuck for some reason, I just restarted the VM so hopefully it will come back and start building. Otherwise I need to investigate. The tests & code that is PowerShell 3+ specific should go into an if that will use the GetPesterPSVersion function (it's called something like that). So it can still work like before on v2. If tests started failing in Gherkin then there is probably some behaviour that has changed by your changes, and it needs to be investigated an addressed. I should be able to help with that later.

@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 9, 2019

Okay restart did not help, but I solve it. The problem is that VSTS agent is not compatible with PowerShell 2 so what I am doing is patching the app via compatibility shimming to think it runs on v3 and then giving it PowerShell 6 instead, so it is all just a big hack. The problem was that I specified a version of the Agent.Listener executable in the shim and the agent updated and so it started seeing the real powershell version and refused to start up.

@renehernandez
Copy link
Copy Markdown
Contributor Author

If tests started failing in Gherkin then there is probably some behaviour that has changed by your changes, and it needs to be investigated an addressed. I should be able to help with that later.

@nohwnd The gherkins testst were already failing in the previous build job before mine, as can be checked here. I have also looked at the stacktrace of the error by they don't seem to be hitting anything related to my mocking changes

Rene Hernandez added 2 commits February 9, 2019 13:09
@renehernandez
Copy link
Copy Markdown
Contributor Author

@nohwnd Now only the PowerShell 3 job is failing in the Gherkins tests

@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 10, 2019

@renehernandez okay will have a look, that error looks weirdly unrelated. I can also see that the master started failing, so maybe it's not a problem of this PR at all.

@renehernandez
Copy link
Copy Markdown
Contributor Author

@nohwnd Any update on the Gherkin issues?? How can I help with that?

@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 13, 2019

I looked at the failed builds and frankly I have no idea why they started failing. I need to have a closer look on it, hopefully I’ll be able to do that today.

@renehernandez renehernandez force-pushed the allow-parameter-aliases branch from cdb107b to 9f7002f Compare February 13, 2019 11:50
@renehernandez
Copy link
Copy Markdown
Contributor Author

renehernandez commented Feb 13, 2019

@nohwnd I guess I should have looked into the exceptions sooner. I had ran into that issue before and I found a solution at this uservoice thread

The problem seems to be that older version of powershell weren't capable of properly managing different encodings with BOM between the current process and a triggered background job

Comment thread Functions/Mock.ps1
@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 13, 2019

@renehernandez aaah, that's why there is apparently no change, especially no change in the xmls.

@nohwnd nohwnd merged commit 0159d38 into pester:master Feb 13, 2019
@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Feb 13, 2019

@renehernandez merged, thanks!

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.

2 participants