llmaz icon indicating copy to clipboard operation
llmaz copied to clipboard
Published 3 months ago • verified publisher icon InftyAI

fix: add ut for backend runtime.

Open X1aoZEOuO opened this issue 9 months ago • 6 comments

What this PR does / why we need it

Add mode unit tests for backend runtime.

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

From:

github.com/inftyai/llmaz/pkg/controller_helper/backendruntime   0.016s  coverage: 35.9% of statements

To:

github.com/inftyai/llmaz/pkg/controller_helper/backendruntime   0.016s  coverage: 69.2% of statements

cc @kerthcet

Does this PR introduce a user-facing change?

Add ut for backend runtime.

X1aoZEOuO avatar May 22 '25 03:05 X1aoZEOuO

/kind cleanup

X1aoZEOuO avatar May 22 '25 03:05 X1aoZEOuO

Would you like to take a view first @googs1025

kerthcet avatar May 22 '25 03:05 kerthcet

/cc

sure. I'll take a look today 😄

googs1025 avatar May 22 '25 03:05 googs1025

/approve

Leave the LGTM to @googs1025

kerthcet avatar May 26 '25 04:05 kerthcet

kindly ping @X1aoZEOuO

kerthcet avatar May 28 '25 07:05 kerthcet

Sorry, I've been preparing for my final exams these past two days. I'll fix it this weekend. @kerthcet

X1aoZEOuO avatar May 29 '25 09:05 X1aoZEOuO

/lgtm Thanks @X1aoZEOuO welcome on board!

kerthcet avatar Jun 02 '25 11:06 kerthcet