Skip to content

Fix rbs test use cases#2938

Open
HoneyryderChuck wants to merge 2 commits intoruby:masterfrom
HoneyryderChuck:fix-cli
Open

Fix rbs test use cases#2938
HoneyryderChuck wants to merge 2 commits intoruby:masterfrom
HoneyryderChuck:fix-cli

Conversation

@HoneyryderChuck
Copy link
Copy Markdown
Contributor

In the process of porting usage of rbs runtime checks via env vars onto rbs test in the httpx repository, there were some setbacks that were addressed by these fixes:

  • using --log-level wasn't working
  • manifest.yaml (a recommendation of RBS docs for libs) isn't being used when running tests from the lib repo. this fixes it by searching for a "manifest.yaml" file in the sig dirs.
  • I initially added a test to support RBS_TEST_RAISE via a --raise parameter, only to find out that it's now raising by default. I left the test around, but if you think it's not adding much, lmk and I can remove it.

this circumvents the need to be explicit about what to require via
RBS_TEST_OPT when a manifest.yaml is already present (as it's considered
a recommendation for gems as per the collection.md doc page).
the value was being parsed into a string, but was then being used as an
array index.
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.

1 participant