128 revival complete test coverage#137
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
black-format
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
[black-format] reported by reviewdog 🐶
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel | ||
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | |
| from monggregate.dollar import ( | |
| AggregationVariableEnum, | |
| Dollar, | |
| DollarDollar, | |
| CLUSTER_TIME, | |
| NOW, | |
| ROOT, | |
| CURRENT, | |
| REMOVE, | |
| DESCEND, | |
| PRUNE, | |
| KEEP, | |
| CONSTANTS, | |
| S, | |
| SS, | |
| ) |
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | ||
| from monggregate.fields import FieldName, FieldPath, Variable | ||
| from monggregate.pipeline import Pipeline, Match, Project | ||
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) | |
| from monggregate.utils import ( | |
| to_unique_list, | |
| validate_field_path, | |
| validate_field_paths, | |
| StrEnum, | |
| ) |
| @@ -0,0 +1,3 @@ | |||
| """Tests for `monggregate.operators` subpackage.""" | |||
|
|
|||
| from monggregate.operators.operator import Operator No newline at end of file | |||
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.operators.operator import Operator | |
| from monggregate.operators.operator import Operator |
| from monggregate.operators.accumulators.max import Max, max | ||
| from monggregate.operators.accumulators.min import Min, min | ||
| from monggregate.operators.accumulators.push import Push, push | ||
| from monggregate.operators.accumulators.sum import Sum, sum No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.operators.accumulators.sum import Sum, sum | |
| from monggregate.operators.accumulators.sum import Sum, sum |
| from monggregate.operators.arithmetic.divide import Divide | ||
| from monggregate.operators.arithmetic.multiply import Multiply | ||
| from monggregate.operators.arithmetic.pow import Pow | ||
| from monggregate.operators.arithmetic.subtract import Subtract No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.operators.arithmetic.subtract import Subtract | |
| from monggregate.operators.arithmetic.subtract import Subtract |
| path=path, | ||
| token_order="any", | ||
| fuzzy=fuzzy_options, | ||
| score={"boost": 2} |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| score={"boost": 2} | |
| score={"boost": 2}, |
| "fuzzy": { | ||
| "maxEdits": 1, | ||
| "prefixLength": 1, | ||
| "maxExpansions": 10 | ||
| }, | ||
| "score": {"boost": 2} |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| "fuzzy": { | |
| "maxEdits": 1, | |
| "prefixLength": 1, | |
| "maxExpansions": 10 | |
| }, | |
| "score": {"boost": 2} | |
| "fuzzy": {"maxEdits": 1, "prefixLength": 1, "maxExpansions": 10}, | |
| "score": {"boost": 2}, |
| actual_expression = autocomplete.expression | ||
|
|
||
| # Assert | ||
| assert actual_expression == expected_expression No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| assert actual_expression == expected_expression | |
| assert actual_expression == expected_expression |
| @@ -0,0 +1,24 @@ | |||
| import pytest | |||
| from monggregate.search.operators.equals import Equals | |||
|
|
|||
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| expected_expression = { | ||
| "equals": { | ||
| "path": path, | ||
| "value": value, | ||
| "score": score | ||
| } | ||
| } |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| expected_expression = { | |
| "equals": { | |
| "path": path, | |
| "value": value, | |
| "score": score | |
| } | |
| } | |
| expected_expression = {"equals": {"path": path, "value": value, "score": score}} |
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
Ruff
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.accumulator.Accumulator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.accumulator.AccumulatorEnum imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.avg.Average imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.avg.avg imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.count.Count imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.count.count imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.first.First imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.first.first imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.last.Last imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.last.last imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.max.Max imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.max.max imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.min.Min imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.min.min imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.push.Push imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.push.push imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.sum.Sum imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.sum.sum imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.add.Add imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.arithmetic.ArithmeticOperatorEnum imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.divide.Divide imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.multiply.Multiply imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.pow.Pow imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.arithmetic.subtract.Subtract imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.array_to_object.ArrayToObject imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.array.Operator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.filter.Filter imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.first.First imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.in_.In imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.is_array.IsArray imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.last.Last imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.max_n.MaxN imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.min_n.MinN imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.size.Size imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.sort_array.SortArray imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.and_.And imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.boolean.BooleanOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.not_.Not imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.or_.Or imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.cmp.Compare imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.comparator.Comparator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.eq.Equal imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.gt.GreatherThan imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.gte.GreatherThanOrEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.lt.LowerThan imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.lte.LowerThanOrEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.ne.NotEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.cond.Cond imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.conditional.ConditionalOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.if_null.IfNull imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.switch.Switch imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.custom.custom.CustomOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.data_size.data_size.DataSizeOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.date.date.DateOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.date.millisecond.Millisecond imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.merge_objects.MergeObjects imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_.ObjectOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_to_array.ObjectToArray imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.concat.Concat imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_from_string.DateFromString imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_to_string.DateToString imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.string.StringOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.type_.type_.Type_ imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.autocomplete.Autocomplete imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.clause.Clause imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.compound.Compound imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.equals.Equals imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.exists.Exists imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.more_like_this.MoreLikeThis imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.operator.SearchOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.range.Range imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.regex.Regex imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.text.Text imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.wildcard.Wildcard imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] datetime.datetime imported but unused
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket_auto.BucketAuto imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket.Bucket imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.count.Count imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.group.Group imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.limit.Limit imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.lookup.Lookup imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.match.Match imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.out.Out imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.project.Project imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.replace_root.ReplaceRoot imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sample.Sample imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.set.Set imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.skip.Skip imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort_by_count.SortByCount imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort.Sort imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.stage.Stage imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.union_with.UnionWith imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unset.Unset imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unwind.Unwind imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.vector_search.VectorSearch imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.BaseModel imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchBase imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchConfig imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search_meta.SearchMeta imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search.Search imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for the monggregate package.""" | ||
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.base.BaseModel imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for the monggregate package.""" | ||
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.base.Expression imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for the monggregate package.""" | ||
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.base.Singleton imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for the monggregate package.""" | ||
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.base.express imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for the monggregate package.""" | ||
| """Tests for `monggregate` package.""" | ||
|
|
||
| from monggregate.base import BaseModel, Expression, Singleton, express, isbasemodel |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.base.isbasemodel imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | ||
| from monggregate.fields import FieldName, FieldPath, Variable | ||
| from monggregate.pipeline import Pipeline, Match, Project | ||
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.utils.to_unique_list imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | ||
| from monggregate.fields import FieldName, FieldPath, Variable | ||
| from monggregate.pipeline import Pipeline, Match, Project | ||
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.utils.validate_field_path imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | ||
| from monggregate.fields import FieldName, FieldPath, Variable | ||
| from monggregate.pipeline import Pipeline, Match, Project | ||
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.utils.validate_field_paths imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.dollar import (AggregationVariableEnum,Dollar,DollarDollar,CLUSTER_TIME,NOW,ROOT,CURRENT,REMOVE,DESCEND,PRUNE,KEEP,CONSTANTS,S,SS,) | ||
| from monggregate.fields import FieldName, FieldPath, Variable | ||
| from monggregate.pipeline import Pipeline, Match, Project | ||
| from monggregate.utils import (to_unique_list,validate_field_path,validate_field_paths,StrEnum,) No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.utils.StrEnum imported but unused; consider removing, adding to __all__, or using a redundant alias
| @@ -0,0 +1,3 @@ | |||
| """Tests for `monggregate.operators` subpackage.""" | |||
|
|
|||
| from monggregate.operators.operator import Operator No newline at end of file | |||
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.operator.Operator imported but unused; consider removing, adding to __all__, or using a redundant alias
| assert not missing_operators_in_clause, ( | ||
| f"Operators in operator map but not in clause: {missing_operators_in_clause}" | ||
| ) | ||
| assert not missing_operators_in_operator_map, ( | ||
| f"Operators in clause but not in operator map: {missing_operators_in_operator_map}" | ||
| ) |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| assert not missing_operators_in_clause, ( | |
| f"Operators in operator map but not in clause: {missing_operators_in_clause}" | |
| ) | |
| assert not missing_operators_in_operator_map, ( | |
| f"Operators in clause but not in operator map: {missing_operators_in_operator_map}" | |
| ) | |
| assert ( | |
| not missing_operators_in_clause | |
| ), f"Operators in operator map but not in clause: {missing_operators_in_clause}" | |
| assert ( | |
| not missing_operators_in_operator_map | |
| ), f"Operators in clause but not in operator map: {missing_operators_in_operator_map}" |
| @@ -0,0 +1,19 @@ | |||
| from monggregate.search.operators.exists import Exists | |||
|
|
|||
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| expected_expression = { | ||
| "exists": { | ||
| "path": path | ||
| } | ||
| } |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| expected_expression = { | |
| "exists": { | |
| "path": path | |
| } | |
| } | |
| expected_expression = {"exists": {"path": path}} |
| actual_expression = exists_op.expression | ||
|
|
||
| # Assert | ||
| assert actual_expression == expected_expression No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| assert actual_expression == expected_expression | |
| assert actual_expression == expected_expression |
| @@ -0,0 +1,19 @@ | |||
| from monggregate.search.operators.more_like_this import MoreLikeThis | |||
|
|
|||
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| query="foo*bar?", | ||
| path="content", | ||
| allow_analyzed_field=True, | ||
| score={"boost": 5} |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| query="foo*bar?", | |
| path="content", | |
| allow_analyzed_field=True, | |
| score={"boost": 5} | |
| query="foo*bar?", path="content", allow_analyzed_field=True, score={"boost": 5} |
| "query": "foo*bar?", | ||
| "path": "content", | ||
| "allowAnalyzedField": True, | ||
| "score": {"boost": 5} |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| "score": {"boost": 5} | |
| "score": {"boost": 5}, |
| expression = wildcard_op.expression | ||
|
|
||
| # Assert | ||
| assert expression == expected No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| assert expression == expected | |
| assert expression == expected |
| from monggregate.stages.union_with import UnionWith | ||
| from monggregate.stages.unset import Unset | ||
| from monggregate.stages.unwind import Unwind | ||
| from monggregate.stages.vector_search import VectorSearch No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.stages.vector_search import VectorSearch | |
| from monggregate.stages.vector_search import VectorSearch |
|
|
||
| from monggregate.stages.search.base import BaseModel, SearchBase, SearchConfig | ||
| from monggregate.stages.search.search_meta import SearchMeta | ||
| from monggregate.stages.search.search import Search No newline at end of file |
There was a problem hiding this comment.
[black-format] reported by reviewdog 🐶
| from monggregate.stages.search.search import Search | |
| from monggregate.stages.search.search import Search |
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
Ruff
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.last.Last imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.max_n.MaxN imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.min_n.MinN imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.size.Size imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.sort_array.SortArray imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.and_.And imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.boolean.BooleanOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.not_.Not imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.boolean.or_.Or imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.cmp.Compare imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.comparator.Comparator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.eq.Equal imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.gt.GreatherThan imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.gte.GreatherThanOrEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.lt.LowerThan imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.lte.LowerThanOrEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.comparison.ne.NotEqual imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.cond.Cond imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.conditional.ConditionalOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.if_null.IfNull imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.conditional.switch.Switch imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.custom.custom.CustomOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.data_size.data_size.DataSizeOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.date.date.DateOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.date.millisecond.Millisecond imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.merge_objects.MergeObjects imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_.ObjectOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_to_array.ObjectToArray imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.concat.Concat imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_from_string.DateFromString imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_to_string.DateToString imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.string.StringOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.type_.type_.Type_ imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.autocomplete.Autocomplete imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.clause.Clause imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.compound.Compound imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.equals.Equals imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.exists.Exists imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.more_like_this.MoreLikeThis imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.operator.SearchOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.range.Range imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.regex.Regex imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.text.Text imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.wildcard.Wildcard imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] datetime.datetime imported but unused
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket_auto.BucketAuto imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket.Bucket imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.count.Count imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.group.Group imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.limit.Limit imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.lookup.Lookup imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.match.Match imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.out.Out imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.project.Project imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.replace_root.ReplaceRoot imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sample.Sample imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.set.Set imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.skip.Skip imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort_by_count.SortByCount imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort.Sort imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.stage.Stage imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.union_with.UnionWith imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unset.Unset imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unwind.Unwind imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.vector_search.VectorSearch imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.BaseModel imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchBase imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchConfig imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search_meta.SearchMeta imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search.Search imported but unused; consider removing, adding to __all__, or using a redundant alias
| @@ -0,0 +1,11 @@ | |||
| """Tests for `monggregate.operators.accumulators` subpackage.""" | |||
|
|
|||
| from monggregate.operators.accumulators.accumulator import Accumulator, AccumulatorEnum | |||
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.accumulator.Accumulator imported but unused; consider removing, adding to __all__, or using a redundant alias
| @@ -0,0 +1,11 @@ | |||
| """Tests for `monggregate.operators.accumulators` subpackage.""" | |||
|
|
|||
| from monggregate.operators.accumulators.accumulator import Accumulator, AccumulatorEnum | |||
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.accumulator.AccumulatorEnum imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for `monggregate.operators.accumulators` subpackage.""" | ||
|
|
||
| from monggregate.operators.accumulators.accumulator import Accumulator, AccumulatorEnum | ||
| from monggregate.operators.accumulators.avg import Average, avg |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.avg.Average imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for `monggregate.operators.accumulators` subpackage.""" | ||
|
|
||
| from monggregate.operators.accumulators.accumulator import Accumulator, AccumulatorEnum | ||
| from monggregate.operators.accumulators.avg import Average, avg |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.avg.avg imported but unused; consider removing, adding to __all__, or using a redundant alias
|
|
||
| from monggregate.operators.accumulators.accumulator import Accumulator, AccumulatorEnum | ||
| from monggregate.operators.accumulators.avg import Average, avg | ||
| from monggregate.operators.accumulators.count import Count, count |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.accumulators.count.Count imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for `monggregate.operators.array` subpackage.""" | ||
|
|
||
| from monggregate.operators.array.array_to_object import ArrayToObject | ||
| from monggregate.operators.array.array import Operator |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.array.Operator imported but unused; consider removing, adding to __all__, or using a redundant alias
|
|
||
| from monggregate.operators.array.array_to_object import ArrayToObject | ||
| from monggregate.operators.array.array import Operator | ||
| from monggregate.operators.array.filter import Filter |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.filter.Filter imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.array_to_object import ArrayToObject | ||
| from monggregate.operators.array.array import Operator | ||
| from monggregate.operators.array.filter import Filter | ||
| from monggregate.operators.array.first import First |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.first.First imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.array import Operator | ||
| from monggregate.operators.array.filter import Filter | ||
| from monggregate.operators.array.first import First | ||
| from monggregate.operators.array.in_ import In |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.in_.In imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.filter import Filter | ||
| from monggregate.operators.array.first import First | ||
| from monggregate.operators.array.in_ import In | ||
| from monggregate.operators.array.is_array import IsArray |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.is_array.IsArray imported but unused; consider removing, adding to __all__, or using a redundant alias
| @@ -0,0 +1,25 @@ | |||
| from monggregate.search.operators.wildcard import Wildcard | |||
|
|
|||
| def test_wildcard_expression_basic(): | |||
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
| @@ -0,0 +1,25 @@ | |||
| from monggregate.search.operators.wildcard import Wildcard | |||
|
|
|||
| def test_wildcard_expression_basic(): | |||
There was a problem hiding this comment.
📝 [mypy] reported by reviewdog 🐶
Use "-> None" if function does not return a value
| @@ -0,0 +1,24 @@ | |||
| from monggregate.search.operators.text import Text | |||
|
|
|||
| def test_text_expression_with_fuzzy_and_synonyms(): | |||
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
| @@ -0,0 +1,24 @@ | |||
| from monggregate.search.operators.text import Text | |||
|
|
|||
| def test_text_expression_with_fuzzy_and_synonyms(): | |||
There was a problem hiding this comment.
📝 [mypy] reported by reviewdog 🐶
Use "-> None" if function does not return a value
| @@ -0,0 +1,25 @@ | |||
| from monggregate.search.operators.regex import Regex | |||
|
|
|||
| def test_regex_expression_basic(): | |||
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
| from datetime import datetime | ||
| from monggregate.search.operators.range import Range | ||
|
|
||
| def test_range_expression_with_numeric_bounds(): |
There was a problem hiding this comment.
📝 [mypy] reported by reviewdog 🐶
Use "-> None" if function does not return a value
| import pytest | ||
| from monggregate.search.operators.equals import Equals | ||
|
|
||
| def test_equals_expression(): |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
| import pytest | ||
| from monggregate.search.operators.equals import Equals | ||
|
|
||
| def test_equals_expression(): |
There was a problem hiding this comment.
📝 [mypy] reported by reviewdog 🐶
Use "-> None" if function does not return a value
| from monggregate.search.operators.autocomplete import Autocomplete | ||
| from monggregate.search.commons import FuzzyOptions | ||
|
|
||
| def test_autocomplete_expression(): |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
| from monggregate.search.operators.autocomplete import Autocomplete | ||
| from monggregate.search.commons import FuzzyOptions | ||
|
|
||
| def test_autocomplete_expression(): |
There was a problem hiding this comment.
📝 [mypy] reported by reviewdog 🐶
Use "-> None" if function does not return a value
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
Ruff
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.string.StringOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.type_.type_.Type_ imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.autocomplete.Autocomplete imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.clause.Clause imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.compound.Compound imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.equals.Equals imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.exists.Exists imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.more_like_this.MoreLikeThis imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.operator.SearchOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.range.Range imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.regex.Regex imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.text.Text imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.search.operators.wildcard.Wildcard imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] pytest imported but unused
[Ruff] reported by reviewdog 🐶
F401 [*] datetime.datetime imported but unused
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket_auto.BucketAuto imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.bucket.Bucket imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.count.Count imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.group.Group imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.limit.Limit imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.lookup.Lookup imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.match.Match imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.out.Out imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.project.Project imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.replace_root.ReplaceRoot imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sample.Sample imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.set.Set imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.skip.Skip imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort_by_count.SortByCount imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.sort.Sort imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.stage.Stage imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.union_with.UnionWith imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unset.Unset imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.unwind.Unwind imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.vector_search.VectorSearch imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.BaseModel imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchBase imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.base.SearchConfig imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search_meta.SearchMeta imported but unused; consider removing, adding to __all__, or using a redundant alias
[Ruff] reported by reviewdog 🐶
F401 monggregate.stages.search.search.Search imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.first import First | ||
| from monggregate.operators.array.in_ import In | ||
| from monggregate.operators.array.is_array import IsArray | ||
| from monggregate.operators.array.last import Last |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.last.Last imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.in_ import In | ||
| from monggregate.operators.array.is_array import IsArray | ||
| from monggregate.operators.array.last import Last | ||
| from monggregate.operators.array.max_n import MaxN |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.max_n.MaxN imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.is_array import IsArray | ||
| from monggregate.operators.array.last import Last | ||
| from monggregate.operators.array.max_n import MaxN | ||
| from monggregate.operators.array.min_n import MinN |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.min_n.MinN imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.last import Last | ||
| from monggregate.operators.array.max_n import MaxN | ||
| from monggregate.operators.array.min_n import MinN | ||
| from monggregate.operators.array.size import Size |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.size.Size imported but unused; consider removing, adding to __all__, or using a redundant alias
| from monggregate.operators.array.max_n import MaxN | ||
| from monggregate.operators.array.min_n import MinN | ||
| from monggregate.operators.array.size import Size | ||
| from monggregate.operators.array.sort_array import SortArray No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.array.sort_array.SortArray imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for `monggregate.operators.objects` subpackage.""" | ||
|
|
||
| from monggregate.operators.objects.merge_objects import MergeObjects | ||
| from monggregate.operators.objects.object_ import ObjectOperator |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_.ObjectOperator imported but unused; consider removing, adding to __all__, or using a redundant alias
|
|
||
| from monggregate.operators.objects.merge_objects import MergeObjects | ||
| from monggregate.operators.objects.object_ import ObjectOperator | ||
| from monggregate.operators.objects.object_to_array import ObjectToArray No newline at end of file |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.objects.object_to_array.ObjectToArray imported but unused; consider removing, adding to __all__, or using a redundant alias
| @@ -0,0 +1,6 @@ | |||
| """Tests for `monggregate.operators.strings` subpackage.""" | |||
|
|
|||
| from monggregate.operators.strings.concat import Concat | |||
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.concat.Concat imported but unused; consider removing, adding to __all__, or using a redundant alias
| """Tests for `monggregate.operators.strings` subpackage.""" | ||
|
|
||
| from monggregate.operators.strings.concat import Concat | ||
| from monggregate.operators.strings.date_from_string import DateFromString |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_from_string.DateFromString imported but unused; consider removing, adding to __all__, or using a redundant alias
|
|
||
| from monggregate.operators.strings.concat import Concat | ||
| from monggregate.operators.strings.date_from_string import DateFromString | ||
| from monggregate.operators.strings.date_to_string import DateToString |
There was a problem hiding this comment.
[Ruff] reported by reviewdog 🐶
F401 monggregate.operators.strings.date_to_string.DateToString imported but unused; consider removing, adding to __all__, or using a redundant alias
| name : FacetName = "" | ||
|
|
||
| path: str | ||
| name: FacetName = "" |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Incompatible types in assignment (expression has type "str", variable has type "FacetName") [assignment]
| "facets":{} | ||
| } | ||
| } | ||
| _statement = {"facet": {"facets": {}}} |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Need type annotation for "_statement" [var-annotated]
| path=path, | ||
| boundaries=boundaries, | ||
| default=default | ||
| name=name, path=path, boundaries=boundaries, default=default |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Argument "boundaries" to "NumericFacet" of "Facet" has incompatible type "list[int | float] | list[datetime] | None"; expected "list[int | float]" [arg-type]
| path=path, | ||
| boundaries=boundaries, | ||
| default=default | ||
| name=name, path=path, boundaries=boundaries, default=default |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Argument "boundaries" to "DateFacet" of "Facet" has incompatible type "list[int | float] | list[datetime] | None"; expected "list[datetime]" [arg-type]
| } | ||
|
|
||
| return _constructors_map[operator_name] No newline at end of file | ||
| return _constructors_map[operator_name] |
There was a problem hiding this comment.
🚫 [mypy] reported by reviewdog 🐶
Incompatible return value type (got "function", expected "Callable[..., Self]") [return-value]
VianneyMI
left a comment
VianneyMI
left a comment
There was a problem hiding this comment.
Ok now the serious stuff begins !
2 participants
No description provided.