Sync with BioPortal release 2.6.0 #9
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…into ncbo codebase
Brings ontoportal-lirmm#7 into ncbo codebase
Updated PURL-related tests to assert only the HTTP 302 status and Location header, without following the redirect to the BioPortal UI. This avoids triggering Cloudflare bot protection during CI runs. We do not want to test UI behavior here — we only verify that the correct redirect is served by the PURL resolver. Testing how the UI handles or rewrites URLs (e.g., with query parameters or concept ID fragments) is out of scope for these tests. Split PURL resolution tests into a new file test_class_purl.rb to separate concerns from core class model tests in test_class.rb Resolves /issues/41
…parate file" This reverts commit c3be85b.
… UI redirects Updated tests for the `.purl` method to: - Assert the constructed URL matches the expected output based on client config - Confirm the PURL resolves via HTTP 302 to the correct UI target - Avoid following the UI-level redirect to prevent Cloudflare issues in CI - Used `@@purl_prefix` from config.purl_prefix.
Fix: Avoid following PURL redirects in unit tests
Feature/enhanced skos support
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
https://github.com/ncbo/ontologies_api_ruby_client/releases/tag/v2.6.0