Skip to content

Conversation

@j-luong
Copy link
Contributor

@j-luong j-luong commented Oct 21, 2025

Manual testing:
As described in CONTRIBUTING.md:

  • download and configure teamcity
  • build the plugin as described in the local testing section
  • upload and enable the local version of the plugin
  • configure a test project
  • configure the build job by adding a Snyk plugin step with the following config
Screenshot 2025-10-21 at 16 24 47 - run build job

@snyk-io
Copy link

snyk-io bot commented Oct 21, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@j-luong j-luong force-pushed the feat/cli-1082_supportSBOMCommand branch 2 times, most recently from 3b6bee5 to 9dbb907 Compare October 22, 2025 09:33
@j-luong j-luong force-pushed the feat/cli-1082_supportSBOMCommand branch 7 times, most recently from ed86483 to ade3b7f Compare October 23, 2025 13:58
@j-luong j-luong force-pushed the feat/cli-1082_supportSBOMCommand branch from ade3b7f to 5ef71af Compare October 23, 2025 14:09
@@ -0,0 +1,183 @@
package io.snyk.plugins.teamcity.agent.commands;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: These tests have questionable value, as the command is not actually executed. The TeamCity framework makes proper testing of the command difficult because of things like final methods & protected access requiring extensive mocking.

open to suggestions on how to better do this.

@j-luong j-luong marked this pull request as ready for review October 23, 2025 14:13
@j-luong j-luong requested review from a team as code owners October 23, 2025 14:13
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.

2 participants