You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jun 5, 2025. It is now read-only.
Repository navigation
This repository was archived by the owner on Jun 5, 2025. It is now read-only.
Copilot Integration tests can generate false positives for insight.stacklok.com links #767
In attempting to add a utm_source=codegate querystring to links generated to insight.stacklok.com for malicious packages, I was able to make changes that caused the integration tests to pass (e.g. Copilot User asks about a malicious package). But this did not result in the build package providing the new link format.
NOTE: [The issue with the utm_source not being added has been fixed](- #765) but this issue is to track a possible issue with the integration testing approach.
The updated copilot integration tests passed (the response https://www.insight.stacklok.com/report/pypi/invokehttp?utm_source=codegate) but when I built the container locally, it generates the warning about invokehttp, but the links don't contain the string.
CodeGate version: v0.1.13-25c2368-dev, so it seems like I'm building and running the right container.
In order to get the desired behavior in the built container, it seems that the generated insight link needed to be updated in output.py.
But, the integration tests passed without doing this (just by updating default.yaml to have the querystring added and changing the integraton test expectations, but the actual result in the codegate response wasn't correctly changed without adjusting output.py.
I think we can close this one as completed. The integration test is checking for the new link (that has utm_source=codegate appended), so we should be okay 👍
Activity
rdimitrov commentedon Feb 17, 2025
I think we can close this one as completed. The integration test is checking for the new link (that has
utm_source=codegateappended), so we should be okay 👍jhrozek commentedon Feb 17, 2025
Closing for now. Reopen if needed @wright-io