Skip to content

Comments

fix: Fix failing tests#4840

Merged
hezyin merged 2 commits intoapache:mainfrom
merico-ai:hotfix_failing_python_plugin_test
Apr 1, 2023
Merged

fix: Fix failing tests#4840
hezyin merged 2 commits intoapache:mainfrom
merico-ai:hotfix_failing_python_plugin_test

Conversation

@CamilleTeruel
Copy link
Contributor

Commit 4c07a81 renamed two JSON tags of server/services/remote/models/models.go:ScopeModel fields.
This made the python_plugin_test.go tests fail for all builds.

⚠️ Pre Checklist

  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Commit 4c07a81 renamed json two JSON tags of server/services/remote/models/models.go:ScopeModel fields.
This made the python_plugin_test.go tests fail.
keon94
keon94 previously approved these changes Apr 1, 2023
Apparently a gorm bug prevent to have two uint64 primary keys in MySQL.
Copy link
Contributor

@hezyin hezyin left a comment

Choose a reason for hiding this comment

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

LGTM

@hezyin hezyin merged commit cc67265 into apache:main Apr 1, 2023
@CamilleTeruel CamilleTeruel deleted the hotfix_failing_python_plugin_test branch April 1, 2023 19:33
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