Skip to content

Conversation

@MohanLaksh
Copy link
Collaborator

📝 TEST CASE UPDATE - Coverage 65 %

Updated the test cases for the following utils along with catalog service. Coverage improved to 65%.

  1. mcpgateway/services/catalog_service.py
  2. mcpgateway/utils/jwt_config_helper.py
  3. mcpgateway/utils/sqlalchemy_modifier.py
  4. mcpgateway/utils/validate_signature.py

Following is the TEST SUMMARY:

  1. make serve - PASS
  2. make test - PASS - === 3635 passed, 271 skipped, 2 warnings in 266.78s (0:04:26) === - COVERAGE - 65 %
  3. make autoflake isort black flake8 - PASS
  4. make doctest - PASS - 876 passed, 48 skipped in 24.85s
  5. make pylint - PASS - Your code has been rated at 10.00/10
  6. make smoketest` - PASS - ✅ Smoketest passed!

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
@crivetimihai crivetimihai merged commit 1a7e1a6 into IBM:main Nov 13, 2025
44 checks passed
Nayana-R-Gowda pushed a commit that referenced this pull request Nov 14, 2025
* Utils test case update

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

* Utils and catalog test case updates

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

* Test case update for utils

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

---------

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
Co-authored-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
p4yl04d3r pushed a commit to p4yl04d3r/mcp-context-forge that referenced this pull request Nov 19, 2025
* Utils test case update

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

* Utils and catalog test case updates

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

* Test case update for utils

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>

---------

Signed-off-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
Co-authored-by: Mohan Lakshmaiah <mohalaks@in.ibm.com>
Signed-off-by: p4yl04d3r <cvogan@gmail.com>
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.

3 participants