Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test-CadlRanch filter no longer works #5504

Open
JoshLove-msft opened this issue Jan 6, 2025 · 3 comments · May be fixed by #5548
Open

Test-CadlRanch filter no longer works #5504

JoshLove-msft opened this issue Jan 6, 2025 · 3 comments · May be fixed by #5548
Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Comments

@JoshLove-msft
Copy link
Contributor

Seems to be related to the changes in #5357.

Running
.\Test-CadlRanch.ps1 http/encode/bytes
Should generate the bytes project and run the corresponding tests.
@JoshLove-msft JoshLove-msft added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Jan 6, 2025
@JoshLove-msft
Copy link
Contributor Author

/cc @pshao25

@pshao25
Copy link
Contributor

pshao25 commented Jan 7, 2025

It changes to .\Test-CadlRanch.ps1 encode/bytes

@pshao25 pshao25 closed this as completed Jan 7, 2025
@JoshLove-msft JoshLove-msft reopened this Jan 8, 2025
@JoshLove-msft
Copy link
Contributor Author

JoshLove-msft commented Jan 8, 2025 •
edited
Loading

This generates the project, but the tests are still skipped. It looks like we need to update the folder structure to not have the http folder and the Http piece of the test namespaces. Reopening.

The tests are also all skipped if no filter is passed.

@pshao25 pshao25 linked a pull request Jan 9, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix the Test-CadlRanch script issue pshao25/typespec
2 participants
@JoshLove-msft @pshao25