Skip to content

chore: add tools/list integration test#4

Merged
mrizzi merged 4 commits intoguacsec:mainfrom
mrizzi:chore-add-test-tools-list
Jun 30, 2025
Merged

chore: add tools/list integration test#4
mrizzi merged 4 commits intoguacsec:mainfrom
mrizzi:chore-add-test-tools-list

Conversation

@mrizzi
Copy link
Copy Markdown
Contributor

@mrizzi mrizzi commented Jun 27, 2025

First and very basic test to list the tools available for the MCP server, done first of all to have a test approach for the MCP Server (replacing the simple cargo build executed previously).

It doesn't require any running trustify instance but for sure it's something that must be added soon to develop more tests that interact with trustify.

In an effort or reusing contains_subset from trustify, I faced this error:

error: environment variable `CARGO_WORKSPACE_ROOT` not defined at compile time
   --> /Users/mrizzi/.cargo/git/checkouts/trustify-094e5444376295b1/dd6f240/test-context/src/lib.rs:201:35
    |
201 |     let workspace_root: PathBuf = env!("CARGO_WORKSPACE_ROOT").into();
    |                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: Cargo sets build script variables at run time. Use `std::env::var("CARGO_WORKSPACE_ROOT")` instead

error: could not compile `trustify-test-context` (lib) due to 1 previous error

so I've pushed the branch chore-env-not-defined with the fix (?) that, before merging this PR, should be properly discussed and fixed in trustify itself.

mrizzi added 3 commits June 27, 2025 16:05
Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: mrizzi <mrizzi@redhat.com>
Signed-off-by: mrizzi <mrizzi@redhat.com>
@mrizzi mrizzi force-pushed the chore-add-test-tools-list branch from 45ab9c2 to e6ba285 Compare June 30, 2025 15:02
@mrizzi mrizzi marked this pull request as ready for review June 30, 2025 15:02
@mrizzi
Copy link
Copy Markdown
Contributor Author

mrizzi commented Jun 30, 2025

contains_subset fixed adding .cargo/config.toml in 3fb3046 just like trustify project has it (ref. config.toml)

@mrizzi mrizzi merged commit 3d06af3 into guacsec:main Jun 30, 2025
1 check passed
@mrizzi mrizzi deleted the chore-add-test-tools-list branch June 30, 2025 16:57
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