Skip to content

fix: intrinsic tests and add some safeguards for future adapters changes #1078

Open
jakelorocco wants to merge 3 commits into
generative-computing:mainfrom
jakelorocco:test/fix-intrinsic-tests
Open

fix: intrinsic tests and add some safeguards for future adapters changes #1078
jakelorocco wants to merge 3 commits into
generative-computing:mainfrom
jakelorocco:test/fix-intrinsic-tests

Conversation

@jakelorocco
Copy link
Copy Markdown
Contributor

@jakelorocco jakelorocco commented May 14, 2026

Misc PR

Type of PR

  • Bug Fix
  • New Feature
  • Documentation
  • Other

Description

  • Link to Issue: Fixes update intrinsic tests #1029

  • Adds back the uncertainty and requirement-check tests

  • Adds last_validated_commit for adapters we test for so that we can catch future version changes in our nightlies

Testing

  • Tests added to the respective file if code was changed
  • New code has 100% coverage if code as added
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

Attribution

  • AI coding assistants used

…nsics

Signed-off-by: Jake LoRocco <jake.lorocco@ibm.com>
Assisted-by: CLAUDE:OPUS
Signed-off-by: Jake LoRocco <jake.lorocco@ibm.com>
Assisted-by: CLAUDE:OPUS
@github-actions github-actions Bot added the bug Something isn't working label May 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

The PR description has been updated. Please fill out the template for your PR to be reviewed.

@jakelorocco jakelorocco linked an issue May 14, 2026 that may be closed by this pull request
…ests; small nits

Signed-off-by: Jake LoRocco <jake.lorocco@ibm.com>
Assisted-by: CLAUDE:OPUS
@jakelorocco jakelorocco force-pushed the test/fix-intrinsic-tests branch from 84f4af5 to 33401f4 Compare May 18, 2026 17:57
@jakelorocco jakelorocco marked this pull request as ready for review May 18, 2026 18:32
@jakelorocco jakelorocco requested a review from a team as a code owner May 18, 2026 18:32
@jakelorocco
Copy link
Copy Markdown
Contributor Author

jakelorocco commented May 18, 2026

@frreiss, this PR will require your review when you get a chance. Thank you. I think the main part is to make sure I added support for req-check and certainty correctly. I think I have previously mentioned the versioning checks to you.

@jakelorocco jakelorocco requested a review from frreiss May 18, 2026 18:33
@jakelorocco jakelorocco changed the title fix: intrinsic tests and add some safeguards for future adapters changes fix: intrinsic tests and add some safeguards for future adapters changes May 18, 2026
@jakelorocco jakelorocco added bug Something isn't working and removed bug Something isn't working labels May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update intrinsic tests

1 participant