-
Notifications
You must be signed in to change notification settings - Fork 394
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add test for RealNNVectorizer #288
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, see comments
|
||
val expectedResult: Seq[OPVector] = Array(-1.0, -4.0, 5.0, -5.5, 0.1, 2.0, 0.0).map(Array(_).toOPVector) | ||
|
||
behavior of "RealNNVectorizerTest" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don’t think god need to specify the ‘behavior’ as the test name is set in the base class
import com.salesforce.op.test.{OpTransformerSpec, TestFeatureBuilder} | ||
|
||
|
||
class RealNNVectorizerTest extends OpTransformerSpec[OPVector, RealNNVectorizer] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add run with junit annotation
Codecov Report
@@ Coverage Diff @@
## master #288 +/- ##
==========================================
+ Coverage 86.4% 86.42% +0.01%
==========================================
Files 319 319
Lines 10453 10453
Branches 351 561 +210
==========================================
+ Hits 9032 9034 +2
+ Misses 1421 1419 -2
Continue to review full report at Codecov.
|
Related issues
Improve test coverage for transformers & estimators #278
Describe the proposed solution
Tests using
OpTransformerSpec
and also checks.vectorize
shortcutDescribe alternatives you've considered
None
Additional context
None