Skip to content

Commit 330ee3d

Browse files
committed
working tests
1 parent 2e1b8cd commit 330ee3d

5 files changed

Lines changed: 9 additions & 19 deletions

File tree

packages/algebra-transformer-1-1/lib/algebraToAst/sparql.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import type * as RDF from '@rdfjs/types';
22
import {
3-
43
Factory as AstFactory,
54
} from '@traqula/rules-sparql-1-1';
65
import type {
@@ -56,7 +55,6 @@ import type {
5655
Term,
5756
Sparql11Nodes,
5857
Quads,
59-
6058
DatasetClauses,
6159
PatternFilter,
6260
} from '@traqula/rules-sparql-1-1';
@@ -849,7 +847,7 @@ F.gen(),
849847
const asCasted = <UpdateOperationDeleteWhere & { where?: unknown; delete?: unknown }> <unknown> updates[0];
850848
asCasted.data = updates[0].delete;
851849
delete asCasted.where;
852-
delete op.delete;
850+
delete asCasted.delete;
853851
asCasted.subType = 'deletewhere';
854852
}
855853

packages/algebra-transformer-1-1/lib/sparqlAlgebra.ts

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import {
66
findPatternBoundedVars,
77
} from '@traqula/rules-sparql-1-1';
88
import type {
9-
109
BasicGraphPattern,
1110
Expression,
1211
GraphQuads,
@@ -29,14 +28,12 @@ import type {
2928
TermIri,
3029
PatternFilter,
3130
PatternValues,
32-
3331
ExpressionAggregate,
3432
PatternBind,
3533
Update,
3634
UpdateOperation,
3735
UpdateOperationLoad,
3836
UpdateOperationCreate,
39-
4037
DatasetClauses,
4138
GraphRef,
4239
GraphRefAll,
@@ -49,7 +46,6 @@ import type {
4946
Path,
5047
UpdateOperationClear,
5148
UpdateOperationDrop,
52-
5349
ContextDefinition,
5450
} from '@traqula/rules-sparql-1-1';
5551
import equal from 'fast-deep-equal/es6';
@@ -117,6 +113,7 @@ class QueryTranslator {
117113
public constructor(private readonly factory: Factory) {}
118114

119115
public translateQuery(sparql: SparqlQuery, quads?: boolean, blankToVariable?: boolean): Algebra.Operation {
116+
const F = this.astFactory;
120117
this.variables = new Set();
121118
this.varCount = 0;
122119
this.useQuads = quads ?? false;
@@ -126,14 +123,14 @@ class QueryTranslator {
126123
// Find ALL variables here to fill `variables` array - needed to create fresh variables
127124
this.findAllVariables(sparql);
128125

129-
if (this.astFactory.isQuery(sparql)) {
126+
if (F.isQuery(sparql)) {
130127
this.registerContextDefinitions(sparql.context);
131128
// Group and where are identical, having only 1 makes parsing easier, can be undefined in DESCRIBE
132-
const group: PatternGroup = sparql.where ?? this.astFactory.patternGroup([], this.astFactory.sourceLocation());
129+
const group: PatternGroup = sparql.where ?? F.patternGroup([], F.gen());
133130
result = this.translateGraphPattern(group);
134131
// 18.2.4 Converting Groups, Aggregates, HAVING, final VALUES clause and SELECT Expressions
135132
result = this.translateAggregates(sparql, result);
136-
} else if (sparql.type === 'update') {
133+
} else if (F.isUpdate(sparql)) {
137134
result = this.translateUpdate(sparql);
138135
}
139136
if (blankToVariable) {

packages/algebra-transformer-1-1/lib/util.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import type * as RDF from '@rdfjs/types';
22
import { someTermsNested } from 'rdf-terms';
3-
import type { Wildcard } from 'sparqljs';
43
import type * as A from './algebra';
54
import { expressionTypes, Types } from './algebra';
65
import Factory from './factory';
@@ -681,10 +680,6 @@ expr.separator,
681680
public static hasQuadVariables(quad: RDF.Quad): boolean {
682681
return someTermsNested(quad, term => term.termType === 'Variable');
683682
}
684-
685-
public static isWildcard(term: any): term is Wildcard {
686-
return term.termType === 'Wildcard';
687-
}
688683
}
689684

690685
/**

packages/algebra-transformer-1-1/package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,14 @@
2929
"@traqula/core": "^0.0.0",
3030
"@traqula/rules-sparql-1-1": "^0.0.0",
3131
"fast-deep-equal": "^3.1.3",
32-
"minimist": "^1.2.6",
3332
"rdf-data-factory": "^2.0.1",
3433
"rdf-isomorphic": "^2.0.0",
3534
"rdf-string": "^2.0.0",
36-
"rdf-terms": "^2.0.0",
37-
"sparqljs": "^3.7.1"
35+
"rdf-terms": "^2.0.0"
3836
},
3937
"devDependencies": {
38+
"@rdfjs/types": "^2.0.1",
39+
"@traqula/generator-sparql-1-1": "^0.0.0",
4040
"@traqula/parser-sparql-1-1": "^0.0.0"
4141
}
4242
}

yarn.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7165,7 +7165,7 @@ source-map@^0.6.1:
71657165
resolved "https://registry.yarnpkg.com/source-map/-/source-map-0.6.1.tgz#74722af32e9614e9c287a8d0bbde48b5e2f1a263"
71667166
integrity sha512-UjgapumWlbMhkBgzT7Ykc5YXUT46F0iKu8SGXq0bcwP5dz/h0Plj6enJqjz1Zbq2l5WaqYnrVbwWOWMyF3F47g==
71677167

7168-
sparqljs@^3.7.1, sparqljs@^3.7.3:
7168+
sparqljs@^3.7.3:
71697169
version "3.7.3"
71707170
resolved "https://registry.yarnpkg.com/sparqljs/-/sparqljs-3.7.3.tgz#075821d51ef4954284e36569503fe5558cfb71b0"
71717171
integrity sha512-FQfHUhfwn5PD9WH6xPU7DhFfXMgqK/XoDrYDVxz/grhw66Il0OjRg3JBgwuEvwHnQt7oSTiKWEiCZCPNaUbqgg==

0 commit comments

Comments
 (0)