Skip to content

Conversation

@pinglin
Copy link
Member

@pinglin pinglin commented Jan 30, 2026

Because

  • Integration test file naming was inconsistent with other repositories (used underscore _ instead of dash -)
  • CE now only supports Basic Auth; JWT authentication is EE-only
  • Proto files were outdated and needed to match current protobuf definitions

This commit

  • Rename all integration test files from underscore to dash format for consistency:
    • grpc_create_model.jsgrpc-create-model.js
    • rest_create_model.jsrest-create-model.js
    • etc.
  • Rename JWT test files to Basic Auth:
    • rest_create_model_with_jwt.jsrest-create-model-with-basic-auth.js
    • rest_deploy_model_with_jwt.jsrest-deploy-model-with-basic-auth.js
    • rest_model_card_with_jwt.jsrest-model-card-with-basic-auth.js
    • rest_publish_model_with_jwt.jsrest-publish-model-with-basic-auth.js
    • rest_query_model_with_jwt.jsrest-query-model-with-basic-auth.js
    • rest_update_model_with_jwt.jsrest-update-model-with-basic-auth.js
    • rest_with_jwt.jsrest-with-basic-auth.js
  • Add getBasicAuthHeader() helper function for Basic Auth encoding
  • Update setup() functions in grpc.js and rest.js to use Basic Auth instead of JWT login
  • Update gRPC client proto loading paths
  • Update health check endpoint from /v1alpha/health/model to /v1beta/health/model
  • Update proto files to match current protobuf definitions

@pinglin pinglin merged commit f732758 into main Jan 30, 2026
5 checks passed
@pinglin pinglin deleted the pinglin/refactor-test-naming-basic-auth branch January 30, 2026 10:35
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