Skip to content

New batch of fixes for Opencypher TCK tests #3366

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.

Sign up for GitHub

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

Merged
lvca merged 17 commits into from
Feb 7, 2026

fix: opencypher more code fixed thank to the tck

900a178
Select commit
Loading
Failed to load commit list.
Merged

New batch of fixes for Opencypher TCK tests #3366

fix: opencypher more code fixed thank to the tck
900a178
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Feb 7, 2026 in 0s

11 new issues (0 max.) of at least severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 11
           

Complexity increasing per file
==============================
- engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/OrderByStep.java  33
- engine/src/main/java/com/arcadedb/query/opencypher/ast/ComparisonExpression.java  12
- engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/SetStep.java  24
- engine/src/main/java/com/arcadedb/query/opencypher/parser/CypherASTBuilder.java  5
- engine/src/main/java/com/arcadedb/query/opencypher/ast/StringMatchExpression.java  3
- engine/src/main/java/com/arcadedb/query/opencypher/executor/CypherFunctionFactory.java  49
- engine/src/main/java/com/arcadedb/query/opencypher/ast/PropertyAccessExpression.java  15
- engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/CreateStep.java  2
- engine/src/main/java/com/arcadedb/query/opencypher/executor/CypherExecutionPlan.java  11
- engine/src/main/java/com/arcadedb/query/opencypher/ast/InExpression.java  8
- engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java  22
- engine/src/main/java/com/arcadedb/query/opencypher/ast/SetClause.java  4
- engine/src/main/java/com/arcadedb/query/opencypher/Labels.java  4
- engine/src/main/java/com/arcadedb/query/opencypher/ast/ListIndexExpression.java  8
- engine/src/main/java/com/arcadedb/query/opencypher/parser/CypherSemanticValidator.java  49
- engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java  67
         

Complexity decreasing per file
==============================
+ engine/src/main/java/com/arcadedb/query/opencypher/temporal/CypherTime.java  -3
+ engine/src/main/java/com/arcadedb/query/opencypher/temporal/CypherDateTime.java  -3
+ engine/src/main/java/com/arcadedb/query/opencypher/temporal/CypherDuration.java  -2
+ engine/src/main/java/com/arcadedb/query/opencypher/temporal/CypherLocalDateTime.java  -3
+ engine/src/main/java/com/arcadedb/query/opencypher/temporal/CypherLocalTime.java  -4

See the complete overview on Codacy

Annotations

Check warning on line 202 in engine/src/main/java/com/arcadedb/query/opencypher/ast/ComparisonExpression.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/ast/ComparisonExpression.java#L202 

Avoid using a branching statement as the last in a loop.

Check notice on line 589 in engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java#L589 

Unnecessary use of fully qualified name 'java.util.Map' due to existing import 'java.util.Map'

Check notice on line 591 in engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java#L591 

Unnecessary use of fully qualified name 'java.util.Map' due to existing import 'java.util.Map'

Check notice on line 596 in engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java#L596 

Unnecessary use of fully qualified name 'java.util.Map.Entry' due to existing import 'java.util.Map'

Check notice on line 623 in engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java#L623 

Unnecessary use of fully qualified name 'java.util.List' due to existing import 'java.util.List'

Check notice on line 624 in engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/executor/steps/MergeStep.java#L624 

Unnecessary use of fully qualified name 'java.util.ArrayList' due to existing import 'java.util.ArrayList'

Check notice on line 716 in engine/src/main/java/com/arcadedb/query/opencypher/parser/CypherSemanticValidator.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/parser/CypherSemanticValidator.java#L716 

Avoid unused private methods such as 'checkAggregationInOrderBy(Expression)'.

Check notice on line 459 in engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java#L459 

Avoid unused private methods such as 'toInstant(CypherTemporalValue)'.

Check notice on line 463 in engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java#L463 

Unnecessary use of fully qualified name 'java.time.ZoneOffset.UTC' due to existing import 'java.time.ZoneOffset'

Check notice on line 465 in engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java#L465 

Unnecessary use of fully qualified name 'java.time.ZoneOffset.UTC' due to existing import 'java.time.ZoneOffset'

Check notice on line 469 in engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

engine/src/main/java/com/arcadedb/query/opencypher/temporal/TemporalUtil.java#L469 

Unnecessary use of fully qualified name 'java.time.ZoneOffset.UTC' due to existing import 'java.time.ZoneOffset'
View more details on Codacy Production