vertex in this graph
* @method getInEdges
*/
- getInEdges(vertex) {}
+ getInEdges(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Collection view of the outgoing edges incident to vertex
* in this graph.
@@ -60,7 +60,7 @@ module.exports = class Graph extends Hypergraph {
* to vertex in this graph
* @method getOutEdges
*/
- getOutEdges(vertex) {}
+ getOutEdges(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Collection view of the predecessors of vertex
* in this graph. A predecessor of vertex is defined as a vertex v
@@ -73,7 +73,7 @@ module.exports = class Graph extends Hypergraph {
* vertex in this graph
* @method getPredecessors
*/
- getPredecessors(vertex) {}
+ getPredecessors(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Collection view of the successors of vertex
* in this graph. A successor of vertex is defined as a vertex v
@@ -86,7 +86,7 @@ module.exports = class Graph extends Hypergraph {
* vertex in this graph
* @method getSuccessors
*/
- getSuccessors(vertex) {}
+ getSuccessors(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of incoming edges incident to vertex.
* Equivalent to getInEdges(vertex).size().
@@ -95,7 +95,7 @@ module.exports = class Graph extends Hypergraph {
* @return the number of incoming edges incident to vertex
* @method inDegree
*/
- inDegree(vertex) {}
+ inDegree(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of outgoing edges incident to vertex.
* Equivalent to getOutEdges(vertex).size().
@@ -104,7 +104,7 @@ module.exports = class Graph extends Hypergraph {
* @return the number of outgoing edges incident to vertex
* @method outDegree
*/
- outDegree(vertex) {}
+ outDegree(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if v1 is a predecessor of v2 in this graph.
* Equivalent to v1.getPredecessors().contains(v2).
@@ -114,7 +114,7 @@ module.exports = class Graph extends Hypergraph {
* @return true if v1 is a predecessor of v2, and false otherwise.
* @method isPredecessor
*/
- isPredecessor(v1, v2) {}
+ isPredecessor(v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if v1 is a successor of v2 in this graph.
* Equivalent to v1.getSuccessors().contains(v2).
@@ -124,7 +124,7 @@ module.exports = class Graph extends Hypergraph {
* @return true if v1 is a successor of v2, and false otherwise.
* @method isSuccessor
*/
- isSuccessor(v1, v2) {}
+ isSuccessor(v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of predecessors that vertex has in this graph.
* Equivalent to vertex.getPredecessors().size().
@@ -133,7 +133,7 @@ module.exports = class Graph extends Hypergraph {
* @return the number of predecessors that vertex has in this graph
* @method getPredecessorCount
*/
- getPredecessorCount(vertex) {}
+ getPredecessorCount(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of successors that vertex has in this graph.
* Equivalent to vertex.getSuccessors().size().
@@ -142,7 +142,7 @@ module.exports = class Graph extends Hypergraph {
* @return the number of successors that vertex has in this graph
* @method getSuccessorCount
*/
- getSuccessorCount(vertex) {}
+ getSuccessorCount(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* If directed_edge is a directed edge in this graph, returns the source;
* otherwise returns null.
@@ -155,7 +155,7 @@ module.exports = class Graph extends Hypergraph {
* @return the source of directed_edge if it is a directed edge in this graph, or null otherwise
* @method getSource
*/
- getSource(directed_edge) {}
+ getSource(directed_edge) { } //NOSONAR -- Method is intended to be abstract
/**
* If directed_edge is a directed edge in this graph, returns the destination;
* otherwise returns null.
@@ -169,7 +169,7 @@ module.exports = class Graph extends Hypergraph {
* @return the destination of directed_edge if it is a directed edge in this graph, or null otherwise
* @method getDest
*/
- getDest(directed_edge) {}
+ getDest(directed_edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if vertex is the source of edge.
* Equivalent to getSource(edge).equals(vertex).
@@ -179,7 +179,7 @@ module.exports = class Graph extends Hypergraph {
* @return true iff vertex is the source of edge
* @method isSource
*/
- isSource(vertex, edge) {}
+ isSource(vertex, edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if vertex is the destination of edge.
* Equivalent to getDest(edge).equals(vertex).
@@ -188,7 +188,7 @@ module.exports = class Graph extends Hypergraph {
* @param edge the edge to be queried
* @return true iff vertex is the destination of edge
*/
- isDest(vertex, edge) {}
+ isDest(vertex, edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Adds edge e to this graph such that it connects
* vertex v1 to v2.
@@ -209,7 +209,7 @@ module.exports = class Graph extends Hypergraph {
* @see Hypergraph#addEdge(Object, Collection)
* @see #addEdge(Object, Object, Object, EdgeType)
*/
- addEdge(e, v1, v2) {}
+ addEdge(e, v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the vertex at the other end of edge from vertex.
* (That is, returns the vertex incident to edge which is not vertex.)
@@ -219,5 +219,5 @@ module.exports = class Graph extends Hypergraph {
* @return the vertex at the other end of edge from vertex
* @method getOpposite
*/
- getOpposite(vertex, edge) {}
+ getOpposite(vertex, edge) { } //NOSONAR -- Method is intended to be abstract
};
diff --git a/src/com/eduworks/ec/graph/Hypergraph.js b/src/com/eduworks/ec/graph/Hypergraph.js
index 2baa51d5..e0f0032c 100644
--- a/src/com/eduworks/ec/graph/Hypergraph.js
+++ b/src/com/eduworks/ec/graph/Hypergraph.js
@@ -48,7 +48,7 @@ module.exports = class Hypergraph {
* @return a Array view of all edges in this graph
* @method getEdges
*/
- getEdges() {}
+ getEdges() { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a view of all vertices in this graph. In general, this obeys the
* Array contract, and therefore makes no guarantees about the
@@ -57,7 +57,7 @@ module.exports = class Hypergraph {
* @return a Array view of all vertices in this graph
* @method getVerticies
*/
- getVertices() {}
+ getVertices() { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if this graph's vertex collection contains
* vertex. Equivalent to
@@ -67,7 +67,7 @@ module.exports = class Hypergraph {
* @return true iff this graph contains a vertex vertex
* @method containsVertex
*/
- containsVertex(vertex) {}
+ containsVertex(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if this graph's edge collection contains edge.
* Equivalent to getEdges().contains(edge).
@@ -76,21 +76,21 @@ module.exports = class Hypergraph {
* @return true iff this graph contains an edge edge
* @method containsEdge
*/
- containsEdge(edge) {}
+ containsEdge(edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of edges in this graph.
*
* @return the number of edges in this graph
* @method getEdgeCount
*/
- getEdgeCount() {}
+ getEdgeCount() { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of vertices in this graph.
*
* @return the number of vertices in this graph
* @method getVertexCount
*/
- getVertexCount() {}
+ getVertexCount() { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the collection of vertices which are connected to
* vertex via any edges in this graph. If vertex
@@ -103,7 +103,7 @@ module.exports = class Hypergraph {
* is not present
* @method getNeighbors
*/
- getNeighbors(vertex) {}
+ getNeighbors(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the collection of edges in this graph which are connected to
* vertex.
@@ -114,7 +114,7 @@ module.exports = class Hypergraph {
* is not present
* @method getIncidentEdges
*/
- getIncidentEdges(vertex) {}
+ getIncidentEdges(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the collection of vertices in this graph which are connected to
* edge. Note that for some graph types there are guarantees
@@ -129,7 +129,7 @@ module.exports = class Hypergraph {
* not present
* @method getIncidentVertices
*/
- getIncidentVertices(edge) {}
+ getIncidentVertices(edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns an edge that connects this vertex to v. If this edge
* is not uniquely defined (that is, if the graph contains more than one
@@ -158,7 +158,7 @@ module.exports = class Hypergraph {
* @method findEdge
* @see Hypergraph#findEdgeSet(Object, Object)
*/
- findEdge(v1, v2) {}
+ findEdge(v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns all edges that connects this vertex to v. If this
* edge is not uniquely defined (that is, if the graph contains more than
@@ -186,7 +186,7 @@ module.exports = class Hypergraph {
* @method findEdgeSet
* @see Hypergraph#findEdge(Object, Object)
*/
- findEdgeSet(v1, v2) {}
+ findEdgeSet(v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Adds vertex to this graph. Fails if vertex is
* null or already in the graph.
@@ -197,7 +197,7 @@ module.exports = class Hypergraph {
* @throws IllegalArgumentException if vertex is null
* @method addVertex
*/
- addVertex(vertex) {}
+ addVertex(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Adds edge to this graph. Fails under the following
* circumstances:
@@ -221,7 +221,7 @@ module.exports = class Hypergraph {
* legal vertex set for edge
* @method addHyperEdge
*/
- addHyperEdge(edge, vertices) {}
+ addHyperEdge(edge, vertices) { } //NOSONAR -- Method is intended to be abstract
/**
* Removes vertex from this graph. As a side effect, removes
* any edges e incident to vertex if the removal
@@ -242,7 +242,7 @@ module.exports = class Hypergraph {
* false otherwise
* @method removeVertex
*/
- removeVertex(vertex) {}
+ removeVertex(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Removes edge from this graph. Fails if edge is
* null, or is otherwise not an element of this graph.
@@ -252,7 +252,7 @@ module.exports = class Hypergraph {
* false otherwise
* @method removeEdge
*/
- removeEdge(edge) {}
+ removeEdge(edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if v1 and v2 share an
* incident edge. Equivalent to getNeighbors(v1).contains(v2).
@@ -263,7 +263,7 @@ module.exports = class Hypergraph {
* incident edge
* @method isNeighbor
*/
- isNeighbor(v1, v2) {}
+ isNeighbor(v1, v2) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns true if vertex and edge
* are incident to each other. Equivalent to
@@ -276,7 +276,7 @@ module.exports = class Hypergraph {
* are incident to each other
* @method isIncident
*/
- isIncident(vertex, edge) {}
+ isIncident(vertex, edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of edges incident to vertex. Special
* cases of interest:
@@ -300,7 +300,7 @@ module.exports = class Hypergraph {
* @method degree
* @see Hypergraph#getNeighborCount(Object)
*/
- degree(vertex) {}
+ degree(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of vertices that are adjacent to vertex
* (that is, the number of vertices that are incident to edges in
@@ -313,7 +313,7 @@ module.exports = class Hypergraph {
* @return the number of neighboring vertices
* @method getNeighborCount
*/
- getNeighborCount(vertex) {}
+ getNeighborCount(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of vertices that are incident to edge.
* For hyperedges, this can be any nonnegative integer; for edges this must
@@ -326,7 +326,7 @@ module.exports = class Hypergraph {
* @return the number of vertices that are incident to edge.
* @method getIncidentCount
*/
- getIncidentCount(edge) {}
+ getIncidentCount(edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the edge type of edge in this graph.
*
@@ -335,14 +335,14 @@ module.exports = class Hypergraph {
* null if edge has no defined type
* @method getEdgeType
*/
- getEdgeType(edge) {}
+ getEdgeType(edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the default edge type for this graph.
*
* @return the default edge type for this graph
* @method getDefaultEdgeType
*/
- getDefaultEdgeType() {}
+ getDefaultEdgeType() { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the collection of edges in this graph which are of type
* edge_type.
@@ -354,7 +354,7 @@ module.exports = class Hypergraph {
* @method getEdgesOfType
* @see EdgeType
*/
- getEdgesOfType(edge_type) {}
+ getEdgesOfType(edge_type) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of edges of type edge_type in this graph.
*
@@ -362,7 +362,7 @@ module.exports = class Hypergraph {
* @return the number of edges of type edge_type in this graph
* @method getEdgeCountOfType
*/
- getEdgeCountOfType(edge_type) {}
+ getEdgeCountOfType(edge_type) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Array view of the incoming edges incident to
* vertex in this graph.
@@ -372,7 +372,7 @@ module.exports = class Hypergraph {
* vertex in this graph
* @method getInEdges
*/
- getInEdges(vertex) {}
+ getInEdges(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Array view of the outgoing edges incident to
* vertex in this graph.
@@ -382,7 +382,7 @@ module.exports = class Hypergraph {
* vertex in this graph
* @method getOutEdges
*/
- getOutEdges(vertex) {}
+ getOutEdges(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of incoming edges incident to vertex.
* Equivalent to getInEdges(vertex).size().
@@ -391,7 +391,7 @@ module.exports = class Hypergraph {
* @return the number of incoming edges incident to vertex
* @method inDegree
*/
- inDegree(vertex) {}
+ inDegree(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns the number of outgoing edges incident to vertex.
* Equivalent to getOutEdges(vertex).size().
@@ -400,7 +400,7 @@ module.exports = class Hypergraph {
* @return the number of outgoing edges incident to vertex
* @method outDegree
*/
- outDegree(vertex) {}
+ outDegree(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* If directed_edge is a directed edge in this graph, returns
* the source; otherwise returns null. The source of a directed
@@ -413,7 +413,7 @@ module.exports = class Hypergraph {
* in this graph, or null otherwise
* @method getSource
*/
- getSource(directed_edge) {}
+ getSource(directed_edge) { } //NOSONAR -- Method is intended to be abstract
/**
* If directed_edge is a directed edge in this graph, returns
* the destination; otherwise returns null. The destination of
@@ -427,7 +427,7 @@ module.exports = class Hypergraph {
* edge in this graph, or null otherwise
* @method getDest
*/
- getDest(directed_edge) {}
+ getDest(directed_edge) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Array view of the predecessors of
* vertex in this graph. A predecessor of vertex
@@ -441,7 +441,7 @@ module.exports = class Hypergraph {
* vertex in this graph
* @method getPredecessors
*/
- getPredecessors(vertex) {}
+ getPredecessors(vertex) { } //NOSONAR -- Method is intended to be abstract
/**
* Returns a Array view of the successors of
* vertex in this graph. A successor of vertex is
@@ -455,5 +455,5 @@ module.exports = class Hypergraph {
* vertex in this graph
* @method getSuccessors
*/
- getSuccessors(vertex) {}
+ getSuccessors(vertex) { } //NOSONAR -- Method is intended to be abstract
};
diff --git a/src/com/eduworks/ec/promises/helpers.js b/src/com/eduworks/ec/promises/helpers.js
index 7a612dfd..2cb48f4e 100644
--- a/src/com/eduworks/ec/promises/helpers.js
+++ b/src/com/eduworks/ec/promises/helpers.js
@@ -12,7 +12,6 @@ module.exports = {
cassReturnAsPromise: function (o, success, failure, error) {
let p = new Promise((resolve, reject) => {
if (o === undefined || o == null) {
- // console.trace(error);
reject(new Error(error));
} else
resolve(o);
diff --git a/src/com/eduworks/ec/remote/EcRemote.js b/src/com/eduworks/ec/remote/EcRemote.js
index 270c15e3..f6c43df6 100644
--- a/src/com/eduworks/ec/remote/EcRemote.js
+++ b/src/com/eduworks/ec/remote/EcRemote.js
@@ -15,20 +15,20 @@ if (isNode)
let undici = eval("require('undici');");
if (undici != null)
{
- var {setGlobalDispatcher,Agent,fetch} = undici;
+ var {setGlobalDispatcher,Agent,fetch} = undici; //NOSONAR - Intentional use of var to allow for global scope.
setGlobalDispatcher(new Agent({
allowH2: process.env.HTTP2 != null ? process.env.HTTP2.trim() == 'true' : true
}))
}
}
if (typeof window !== 'undefined' && window.fetch != null)
- var fetch = window.fetch;
+ var fetch = window.fetch; //NOSONAR - Intentional use of var to allow for global scope.
if (isNode)
{
try{
- var dns = require('node:dns');
- if (dns && dns.setDefaultResultOrder)
+ var dns = require('node:dns'); //NOSONAR - Intentional use of var to allow for global scope.
+ if (dns?.setDefaultResultOrder)
{
//Support for Node 18 using Docker containers with a network that doesn't support ipv6 loopback.
dns.setDefaultResultOrder('ipv4first');
@@ -154,7 +154,7 @@ module.exports = class EcRemote {
if (DEBUG)
console.log("POST " + server + "" + (service || "") + " " + headers + fd);
let url = server;
- if (!url.endsWith("/") && service != null && !("" == service)) {
+ if (!url.endsWith("/") && service != null && "" != service) {
url += "/";
}
if (service != null) {
@@ -362,13 +362,11 @@ module.exports = class EcRemote {
}
static upgradeHttpToHttps(url) {
if (typeof window !== "undefined")
- if (window != null) {
- if (window.location != null) {
- if (url.indexOf(window.location.protocol) == -1) {
- if (window.location.protocol.startsWith("https")) {
- if (!url.startsWith("https:")) {
- url = url.replace("http:", "https:");
- }
+ if (window?.location != null) {
+ if (url.indexOf(window.location.protocol) == -1) {
+ if (window.location.protocol.startsWith("https")) {
+ if (!url.startsWith("https:")) {
+ url = url.replace("http:", "https:");
}
}
}
diff --git a/src/com/eduworks/ec/task/EcAsyncTask.js b/src/com/eduworks/ec/task/EcAsyncTask.js
index 3c184806..04bfdb90 100644
--- a/src/com/eduworks/ec/task/EcAsyncTask.js
+++ b/src/com/eduworks/ec/task/EcAsyncTask.js
@@ -53,7 +53,7 @@ module.exports = class EcAsyncTask {
return this.start;
}
doTask(_arguments) {
- let ret = undefined;
+ let ret;
if (!this.isStarted) {
this.start = new Date();
this.isStarted = true;
@@ -91,35 +91,35 @@ module.exports = class EcAsyncTask {
}
invoke0 = function() {
let that = this;
- let ret = this.job.call(this, function() {
+ let ret = this.job.call(this, function() { //NOSONAR -- Not taking that chance.
that.setComplete();
});
return ret;
};
invoke1 = function(arg1) {
let that = this;
- let ret = this.job.call(this, arg1, function() {
+ let ret = this.job.call(this, arg1, function () { //NOSONAR -- Not taking that chance.
that.setComplete();
});
return ret;
};
invoke2 = function(arg1, arg2) {
let that = this;
- let ret = this.job.call(this, arg1, arg2, function() {
+ let ret = this.job.call(this, arg1, arg2, function () { //NOSONAR -- Not taking that chance.
that.setComplete();
});
return ret;
};
invoke3 = function(arg1, arg2, arg3) {
let that = this;
- let ret = this.job.call(this, arg1, arg2, arg3, function() {
+ let ret = this.job.call(this, arg1, arg2, arg3, function () { //NOSONAR -- Not taking that chance.
that.setComplete();
});
return ret;
};
invoke4 = function(arg1, arg2, arg3, arg4) {
let that = this;
- let ret = this.job.call(this, arg1, arg2, arg3, arg4, function() {
+ let ret = this.job.call(this, arg1, arg2, arg3, arg4, function() { //NOSONAR -- Not taking that chance.
that.setComplete();
});
return ret;
diff --git a/src/com/eduworks/ec/task/EcAsyncTaskManager.js b/src/com/eduworks/ec/task/EcAsyncTaskManager.js
index e42ef1a0..41e72cc4 100644
--- a/src/com/eduworks/ec/task/EcAsyncTaskManager.js
+++ b/src/com/eduworks/ec/task/EcAsyncTaskManager.js
@@ -18,8 +18,7 @@ module.exports = class EcAsyncTaskManager {
addTasks(tasks) {
let retMap = {};
if (tasks != undefined) {
- for (let i = 0; i < tasks.length; i++) {
- let task = tasks[i];
+ for (let task of tasks) {
let key = this.addTask(task);
retMap[key] = task;
}
@@ -62,9 +61,9 @@ module.exports = class EcAsyncTaskManager {
assignNewTask(task) {
let min = 10000;
let theGuy = null;
- for (let i = 0; i < this.workerList.length; i++) {
- if (this.workerList[i].getAssignedCount() < min) {
- theGuy = this.workerList[i];
+ for (let worker of this.workerList) {
+ if (worker.getAssignedCount() < min) {
+ theGuy = worker;
min = theGuy.getAssignedCount();
}
}
diff --git a/src/com/eduworks/ec/task/EcAsyncTaskWorker.js b/src/com/eduworks/ec/task/EcAsyncTaskWorker.js
index 3a8b66cf..eb0d390e 100644
--- a/src/com/eduworks/ec/task/EcAsyncTaskWorker.js
+++ b/src/com/eduworks/ec/task/EcAsyncTaskWorker.js
@@ -52,9 +52,7 @@ module.exports = class EcAsyncTaskWorker {
that.invoke();
}
if (
- that.currentTask != undefined &&
- that.currentTask != null &&
- that.currentTask.getIsComplete()
+ that.currentTask?.getIsComplete()
) {
that.currentTask = that.assignedList.shift();
}
diff --git a/src/com/eduworks/ec/task/Task.js b/src/com/eduworks/ec/task/Task.js
index 655a50bd..32c77a96 100644
--- a/src/com/eduworks/ec/task/Task.js
+++ b/src/com/eduworks/ec/task/Task.js
@@ -7,7 +7,7 @@
*/
let Task = null;
module.exports = Task = class Task {
- static constructor() {
+ static() {
Task.updateFrame();
}
static desiredFps = 2;
diff --git a/src/org/cass/competency/EcAlignment.js b/src/org/cass/competency/EcAlignment.js
index b9197681..2182cbd6 100644
--- a/src/org/cass/competency/EcAlignment.js
+++ b/src/org/cass/competency/EcAlignment.js
@@ -1,5 +1,6 @@
const EcRepository = require("../../cassproject/ebac/repository/EcRepository.js");
const Relation = require("../../cassproject/schema/cass/competency/Relation.js");
+const EcRemoteLinkedData = require("../../cassproject/schema/general/EcRemoteLinkedData.js");
/**
* Implementation of an alignment object with methods for interacting with CASS
* services on a server.
diff --git a/src/org/cass/competency/EcCompetency.js b/src/org/cass/competency/EcCompetency.js
index 0bf79a4d..f3c38c59 100644
--- a/src/org/cass/competency/EcCompetency.js
+++ b/src/org/cass/competency/EcCompetency.js
@@ -1,5 +1,8 @@
const EcRepository = require("../../cassproject/ebac/repository/EcRepository.js");
const Competency = require("../../cassproject/schema/cass/competency/Competency.js");
+const EcAlignment = require("./EcAlignment.js");
+const EcLevel = require("./EcLevel.js");
+const EcRollupRule = require("./EcRollupRule.js");
require("../../cassproject/general/AuditLogger.js");
/**
* Implementation of a Competency object with methods for interacting with CASS
diff --git a/src/org/cass/competency/EcFramework.js b/src/org/cass/competency/EcFramework.js
index 71fa5878..8fd8e131 100644
--- a/src/org/cass/competency/EcFramework.js
+++ b/src/org/cass/competency/EcFramework.js
@@ -104,20 +104,14 @@ module.exports = class EcFramework extends Framework {
addCompetency(id) {
id = EcRemoteLinkedData.trimVersionFromUrl(id);
if (this.competency == null) this.competency = [];
- for (let i = 0; i < this.competency.length; i++)
- if (
- EcRemoteLinkedData.trimVersionFromUrl(
- this.competency[i]
- ) == id
- )
+ for (let compId of this.competency)
+ if (EcRemoteLinkedData.trimVersionFromUrl(compId) == id)
return;
this.competency.push(id);
}
/**
* Removes a competency ID from the framework's list, also removes any
* levels and relations associated with that competency
- * - * TODO: remove rollup rules? should we add flag to remove these extras * * @param {String} id * ID of the competency to remove @@ -132,10 +126,7 @@ module.exports = class EcFramework extends Framework { let shortId = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.competency == null) this.competency = []; for (let i = 0; i < this.competency.length; i++) - if ( - this.competency[i].equals(shortId) || - this.competency[i].equals(id) - ) + if (this.competency[i] == shortId || this.competency[i] == id) this.competency.splice(i, 1); if ( (this.relation == null || this.relation.length == 0) && @@ -294,10 +285,8 @@ module.exports = class EcFramework extends Framework { addRelation(id) { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.relation == null) this.relation = []; - for (let i = 0; i < this.relation.length; i++) - if ( - EcRemoteLinkedData.trimVersionFromUrl(this.relation[i]) == id - ) + for (let relationId of this.relation) + if (EcRemoteLinkedData.trimVersionFromUrl(relationId) == id) return; this.relation.push(id); } @@ -312,12 +301,8 @@ module.exports = class EcFramework extends Framework { removeRelation(id) { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.relation == null) this.relation = []; - for (let i = 0; i < this.relation.length; i++) - if ( - EcRemoteLinkedData.trimVersionFromUrl(this.relation[i]).equals( - id - ) - ) + for (let i = 0;i < this.relation.length;i++) + if (EcRemoteLinkedData.trimVersionFromUrl(this.relation[i]) == id) this.relation.splice(i, 1); } /** @@ -331,8 +316,8 @@ module.exports = class EcFramework extends Framework { addLevel(id) { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.level == null) this.level = []; - for (let i = 0; i < this.level.length; i++) - if (EcRemoteLinkedData.trimVersionFromUrl(this.level[i]).equals(id)) + for (let levelId of this.level) + if (EcRemoteLinkedData.trimVersionFromUrl(levelId) == id) return; this.level.push(id); } @@ -348,7 +333,7 @@ module.exports = class EcFramework extends Framework { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.level == null) this.level = []; for (let i = 0; i < this.level.length; i++) - if (EcRemoteLinkedData.trimVersionFromUrl(this.level[i]).equals(id)) + if (EcRemoteLinkedData.trimVersionFromUrl(this.level[i]) == id) this.level.splice(i, 1); } /** @@ -362,12 +347,8 @@ module.exports = class EcFramework extends Framework { addRollupRule(id) { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.rollupRule == null) this.rollupRule = []; - for (let i = 0; i < this.rollupRule.length; i++) - if ( - EcRemoteLinkedData.trimVersionFromUrl( - this.rollupRule[i] - ).equals(id) - ) + for (let rollupRuleId of this.rollupRule) + if (EcRemoteLinkedData.trimVersionFromUrl(rollupRuleId) == id) return; this.rollupRule.push(id); } @@ -383,11 +364,7 @@ module.exports = class EcFramework extends Framework { id = EcRemoteLinkedData.trimVersionFromUrl(id); if (this.rollupRule == null) this.rollupRule = []; for (let i = 0; i < this.rollupRule.length; i++) - if ( - EcRemoteLinkedData.trimVersionFromUrl( - this.rollupRule[i] - ).equals(id) - ) + if (EcRemoteLinkedData.trimVersionFromUrl(this.rollupRule[i]) == id) this.rollupRule.splice(i, 1); } /** diff --git a/src/org/cass/competency/EcLevel.js b/src/org/cass/competency/EcLevel.js index 7c9f56d5..beaab9c8 100644 --- a/src/org/cass/competency/EcLevel.js +++ b/src/org/cass/competency/EcLevel.js @@ -1,5 +1,7 @@ const EcRepository = require("../../cassproject/ebac/repository/EcRepository.js"); const Level = require("../../cassproject/schema/cass/competency/Level.js"); +const EcRemoteLinkedData = require("../../cassproject/schema/general/EcRemoteLinkedData.js"); +const EcAlignment = require("./EcAlignment.js"); /** * Implementation of a Level object with methods for interacting with CASS * services on a server. @@ -121,7 +123,6 @@ module.exports = class EcLevel extends Level { targetLevel, alignmentType, identity, - serverUrl, success, failure, repo, eim @@ -131,11 +132,11 @@ module.exports = class EcLevel extends Level { a.target = targetLevel.id; a.relationType = alignmentType; a.addOwner(identity.toPk()); - if (repo == null || repo.selectedServer.indexOf(serverUrl) != -1) - a.generateId(serverUrl); - else a.generateShortId(serverUrl); + if (repo?.selectedServer != null) + a.generateId(repo.selectedServer); await a.signWith(identity); - return a.save(success, failure, repo, eim); + await a.save(success, failure, repo, eim); + return a; } /** * Method to set the name of this level diff --git a/src/org/cassproject/schema/general/EcRemoteLinkedData.js b/src/org/cassproject/schema/general/EcRemoteLinkedData.js index b08733a0..d49c7fa5 100644 --- a/src/org/cassproject/schema/general/EcRemoteLinkedData.js +++ b/src/org/cassproject/schema/general/EcRemoteLinkedData.js @@ -26,7 +26,7 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { * @param {string} type JSON-LD Type. * @constructor */ - constructor(context, type) { + constructor(context, type) { //NOSONAR - Required for proper documentation super(context, type); } /** @@ -83,24 +83,24 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { static caching = false; static trimVersionFromUrl(id) { if (id == null) return null; - if (EcRemoteLinkedData.caching == true && EcRemoteLinkedData.tvfuCache[id] != null) + if (EcRemoteLinkedData.caching && EcRemoteLinkedData.tvfuCache[id] != null) return EcRemoteLinkedData.tvfuCache[id]; if ( id.indexOf("/api/data/") == -1 && id.indexOf("/api/custom/data/") == -1 ) { - if (EcRemoteLinkedData.caching == true) + if (EcRemoteLinkedData.caching) EcRemoteLinkedData.tvfuCache[id] = id; return id; } if (!id.substring(id.lastIndexOf("/")).match("^\\/[0-9]+$")) { - if (EcRemoteLinkedData.caching == true) + if (EcRemoteLinkedData.caching) EcRemoteLinkedData.tvfuCache[id] = id; return id; } let rawId = id.substring(0, id.lastIndexOf("/")); if (rawId.endsWith("/")) rawId = rawId.substring(0, rawId.length - 1); - if (EcRemoteLinkedData.caching == true) + if (EcRemoteLinkedData.caching) EcRemoteLinkedData.tvfuCache[id] = rawId; return rawId; } @@ -209,8 +209,9 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { hasOwner(pk) { if (this.owner == null) return false; let pkPem = pk.toPem(); - for (let i = 0; i < this.owner.length; i++) - if (pkPem == EcPk.fromPem(this.owner[i]).toPem()) return true; + for (let ownerPem of this.owner) + if (pkPem == EcPk.fromPem(ownerPem).toPem()) + return true; return false; } /** @@ -225,8 +226,9 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { hasReader(pk) { if (this.reader == null) return false; let pkPem = pk.toPem(); - for (let i = 0; i < this.reader.length; i++) - if (pkPem == EcPk.fromPem(this.reader[i]).toPem()) return true; + for (let readerPem of this.reader) + if (pkPem == EcPk.fromPem(readerPem).toPem()) + return true; return false; } /** @@ -254,8 +256,9 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { canEditAny(ids) { if (this.owner == null || this.owner.length == 0) return true; if (ids == null) return false; - for (let i = 0; i < ids.length; i++) - if (this.hasOwner(ids[i])) return true; + for (let id of ids) + if (this.hasOwner(id)) + return true; return false; } /** @@ -269,18 +272,11 @@ module.exports = class EcRemoteLinkedData extends EcLinkedData { if (ppk instanceof EcPpkFacade) return; let signableJson = this.toSignableJson(); - // let signed = await EcRsaOaepAsync.sign(ppk, signableJson); - // if (this.signature != null) { - // for (let i = 0; i < this.signature.length; i++) - // if (this.signature[i] == signed) return; - // } else { - // this.signature = []; - // } - // this.signature.push(signed); let signedSha256 = await EcRsaOaepAsyncWorker.signSha256(ppk, signableJson); if (this.signatureSha256 != null) { - for (let i = 0; i < this.signatureSha256.length; i++) - if (this.signatureSha256[i] == signedSha256) return; + for (let signatureSha256Actual of this.signatureSha256) + if (signatureSha256Actual == signedSha256) + return; } else { this.signatureSha256 = []; } diff --git a/src/org/json/ld/EcLinkedData.js b/src/org/json/ld/EcLinkedData.js index 7d3a4481..06d89a07 100644 --- a/src/org/json/ld/EcLinkedData.js +++ b/src/org/json/ld/EcLinkedData.js @@ -47,8 +47,9 @@ global.jsonld = require("jsonld"); * @method isAtProperty */ static isAtProperty(key) { - for (let i = 0; i < EcLinkedData.atProperties.length; i++) - if (EcLinkedData.atProperties[i] == key) return true; + for (let atProperty of EcLinkedData.atProperties) + if (atProperty == key) + return true; return false; } /** @@ -107,12 +108,14 @@ global.jsonld = require("jsonld"); let a = []; for (let i = 0; i < o.length; i++) { if (EcObject.isObject(o[i])) { - if (o[i] instanceof EcLinkedData) a[i] = this.atIfyObject(o[i]); - else { + if (o[i] instanceof EcLinkedData) + a[i] = this.atIfyObject(o[i]); + else a[i] = o[i]; - } - } else if (EcArray.isArray(o[i])) a[i] = this.atIfyArray(o[i]); - else a[i] = o[i]; + } else if (EcArray.isArray(o[i])) + a[i] = this.atIfyArray(o[i]); + else + a[i] = o[i]; } return a; } @@ -120,23 +123,24 @@ global.jsonld = require("jsonld"); let keys = []; let me = o; for (let key in me) { - if (me["type"] != null) - if (EcLinkedData.isAtProperty(key)) key = "@" + key; + if (me["type"] != null && EcLinkedData.isAtProperty(key)) + key = "@" + key; //NOSONAR -- Reassignment of non-const variable is intended. keys.push(key); } - keys.sort(function (a, b) { - return a.localeCompare(b); - }); + keys.sort((a, b) => a.localeCompare(b)); let op = {}; - for (let i = 0; i < keys.length; i++) { - let key = keys[i]; + for (let key of keys) { let value = me[key.replace("@", "")]; - if (value != null) - if (value instanceof EcLinkedData) value = value.atIfy(); - else if (EcArray.isArray(value)) value = this.atIfyArray(value); - if (value != null) op[key] = value; - else value = me[key]; - if (value != null) op[key] = value; + if (value != null && value instanceof EcLinkedData) + value = value.atIfy(); + else if (EcArray.isArray(value)) + value = this.atIfyArray(value); + if (value != null) + op[key] = value; + else + value = me[key]; + if (value != null) + op[key] = value; } return op; } @@ -160,11 +164,11 @@ global.jsonld = require("jsonld"); * @return {boolean} True if match, False if not. * @method isAny */ - isAny(type) { + isAny(types) { let computedType = this.getFullType(); - if (type.length == 0) return true; - for (let i = 0; i < type.length; i++) - if (type[i] == computedType || type[i] == this.type) + if (types.length == 0) return true; + for (let type of types) + if (type == computedType || type == this.type) return true; return false; } @@ -204,17 +208,26 @@ global.jsonld = require("jsonld"); * @method copyFrom */ copyFrom(that, eim) { - let me = this; - for (let key in me) { - if (typeof me[key] != "function") delete me[key]; + for (let key in this) { + if (typeof this[key] == "function") + continue; + delete this[key]; } - let you = that; - for (let key in you) { - if (typeof you[key] != "function") { - if (you["@type"] != null) me[key.replace("@", "")] = you[key]; - else me[key] = you[key]; - } + for (let key in that) { + if (typeof that[key] == "function") + continue; + if (that["@type"] != null) + this[key.replace("@", "")] = that[key]; + else + this[key] = that[key]; } + this.swapContext(); + this.upgrade(eim); + if (!this.isAny(this.getTypes())) + throw new Error("Incompatible type: " + this.getFullType() + that); + return this; + } + swapContext() { let stripNamespace = null; let newContext = null; if ( @@ -226,43 +239,31 @@ global.jsonld = require("jsonld"); if (typeParts.length == 2) { newContext = this.context[typeParts[0]]; stripNamespace = typeParts[0]; - if (!newContext.endsWith("/")) newContext += "/"; + if (!newContext.endsWith("/")) + newContext += "/"; } else if (this.context["@vocab"] != null) newContext = this.context["@vocab"]; } if (stripNamespace != null) - for (let key in me) { - if (typeof me[key] != "function") { - if (key.startsWith(stripNamespace + ":")) { - if (EcArray.isArray(me[key])) { - me[ - key.replace(stripNamespace + ":", "") - ] = JSON.parse( - JSON.stringify(me[key]).replace( - stripNamespace + ":", - "" - ) - ); - } else if (EcObject.isObject(me[key])) { - me[ - key.replace(stripNamespace + ":", "") - ] = JSON.parse( - JSON.stringify(me[key]).replace( - stripNamespace + ":", - "" - ) - ); - } else - me[key.replace(stripNamespace + ":", "")] = me[key]; - delete me[key]; - } - } + this.stripNamespace(); + if (newContext != null) + this.context = newContext; + } + stripNamespace() { + for (let key in this) { + if (typeof this[key] != "function" && key.startsWith(stripNamespace + ":")) { + if (EcArray.isArray(this[key]) || EcObject.isObject(this[key])) { + this[key.replace(stripNamespace + ":", "")] = JSON.parse( + JSON.stringify(this[key]).replace( + stripNamespace + ":", + "" + ) + ); + } else + this[key.replace(stripNamespace + ":", "")] = this[key]; + delete this[key]; } - if (newContext != null) this.context = newContext; - this.upgrade(eim); - if (!this.isAny(this.getTypes())) - throw "Incompatible type: " + this.getFullType() + that; - return this; + } } async recast(translationContext, targetContext) { let me = this; @@ -329,7 +330,7 @@ global.jsonld = require("jsonld"); * * @method upgrade */ - upgrade() {} + upgrade() { } //NOSONAR -- Method is a stub. /** * Removes the @ symbol from properties in order to make them more * accessible in Javascript. @@ -339,24 +340,16 @@ global.jsonld = require("jsonld"); * @internal */ deAtify() { - let me = this; - let typeFound = false; - if (me["@type"] != null) typeFound = true; - for (let key in me) { - if (me[key] == null) { + let typeFound = this["@type"] != null; + for (let key in this) { + if (this[key] == null) { if (typeFound) { - let value = me[key]; - if (value != null) - if (value instanceof EcLinkedData) - value = value.deAtify(); - me[key.replace("@", "")] = value; - } else { - let value = me[key]; - if (value != null) - if (value instanceof EcLinkedData) - value = value.deAtify(); - me[key] = value; - } + let value = this[key]; + if (value != null && value instanceof EcLinkedData) + value = value.deAtify(); + this[key.replace("@", "")] = value; + } else if (this?.[key] instanceof EcLinkedData) + this[key] = this[key].deAtify(); } } return this; diff --git a/src/test/EcAes.test.js b/src/test/EcAes.test.js index 68df7376..7952613f 100644 --- a/src/test/EcAes.test.js +++ b/src/test/EcAes.test.js @@ -1,7 +1,6 @@ let chai = require("chai"); const EcAes = require("../com/eduworks/ec/crypto/EcAes"); -let should = chai.should(); let expect = chai.expect; let assert = chai.assert; diff --git a/src/test/EcAesCtr.test.js b/src/test/EcAesCtr.test.js index c977d165..e577b11c 100644 --- a/src/test/EcAesCtr.test.js +++ b/src/test/EcAesCtr.test.js @@ -27,8 +27,8 @@ describe("EcAesCtr", () => { let randomString = EcAes.newIv(1024); let secret = EcAes.newSecret(16); let iv = EcAes.newIv(16); - let encrypted = EcAesCtr.encrypt(randomString, secret, iv); - let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); + let encrypted = EcAesCtr.encrypt(randomString, secret, iv); //NOSONAR + let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); //NOSONAR assert.isTrue(randomString == decrypted); }); it('encryption then decryption (utf-8)', () => { @@ -36,8 +36,8 @@ describe("EcAesCtr", () => { "abc\u16a0\u16c7\u16bb\u16eb\u16d2\u16e6\u16a6\u16eb\u16a0\u16b1\u16a9\u16a0\u16a2\u16b1\u16eb\u16a0\u16c1\u16b1\u16aa\u16eb\u16b7\u16d6\u16bb\u16b9\u16e6\u16da\u16b3\u16a2\u16d7"; let secret = EcAes.newSecret(16); let iv = EcAes.newIv(16); - let encrypted = EcAesCtr.encrypt(randomString, secret, iv); - let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); + let encrypted = EcAesCtr.encrypt(randomString, secret, iv); //NOSONAR + let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); //NOSONAR assert.isTrue(randomString == decrypted); }); it('large encryption then decryption w/caching', () => { @@ -45,17 +45,17 @@ describe("EcAesCtr", () => { let secret = EcAes.newSecret(16); let iv = EcAes.newIv(16); let hrTime = hrtime(); - let encrypted = EcAesCtr.encrypt(randomString, secret, iv); + let encrypted = EcAesCtr.encrypt(randomString, secret, iv); //NOSONAR let elapsed = (hrtime()[0]*1000000 + hrtime()[1]/1000 - hrTime[0] * 1000000 - hrTime[1] / 1000)/1000; console.log(randomString.length/1024+"KB encryption speed: " + elapsed+"ms"); hrTime = hrtime(); EcCrypto.caching = true; - let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); + let decrypted = EcAesCtr.decrypt(encrypted, secret, iv); //NOSONAR elapsed = (hrtime()[0]*1000000 + hrtime()[1]/1000 - hrTime[0] * 1000000 - hrTime[1] / 1000)/1000; console.log("decryption wout/caching speed: " + elapsed+"ms"); hrTime = hrtime(); decrypted = null; - decrypted = EcAesCtr.decrypt(encrypted, secret, iv); + decrypted = EcAesCtr.decrypt(encrypted, secret, iv); //NOSONAR elapsed = (hrtime()[0]*1000000 + hrtime()[1]/1000 - hrTime[0] * 1000000 - hrTime[1] / 1000)/1000; console.log("decryption w/caching speed: " + elapsed+"ms"); assert.isTrue(elapsed < 1); diff --git a/src/test/EcAlignment.test.js b/src/test/EcAlignment.test.js new file mode 100644 index 00000000..abfb0a1e --- /dev/null +++ b/src/test/EcAlignment.test.js @@ -0,0 +1,245 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcAlignment = require('../org/cass/competency/EcAlignment'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); + +const expect = chai.expect; + +describe('EcAlignment', function () { + describe('equals', function () { + it('should return true if obj has same source, target, and relationType', function () { + const alignment1 = new EcAlignment(); + alignment1.source = 'source1'; + alignment1.target = 'target1'; + alignment1.relationType = 'relationType1'; + + const alignment2 = { + source: 'source1', + target: 'target1', + relationType: 'relationType1' + }; + + expect(alignment1.equals(alignment2)).to.be.true; + }); + + it('should return false if obj has different source, target, or relationType', function () { + const alignment1 = new EcAlignment(); + alignment1.source = 'source1'; + alignment1.target = 'target1'; + alignment1.relationType = 'relationType1'; + + const alignment2 = { + source: 'source2', + target: 'target1', + relationType: 'relationType1' + }; + + expect(alignment1.equals(alignment2)).to.be.false; + }); + + it('should return true if obj has same id', function () { + const alignment1 = new EcAlignment(); + alignment1.id = 'id1'; + + const alignment2 = { + id: 'id1' + }; + + expect(alignment1.equals(alignment2)).to.be.true; + }); + }); + + describe('get', function () { + it('should call EcRepository.getAs with correct parameters', function () { + const stub = sinon.stub(EcRepository, 'getAs'); + const id = 'id1'; + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const eim = {}; + + EcAlignment.get(id, success, failure, repo, eim); + + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(id, sinon.match.instanceOf(EcAlignment), success, failure, repo, eim)).to.be.true; + + stub.restore(); + }); + }); + + describe('getBlocking', function () { + it('should call EcRepository.getAs with correct parameters', function () { + const stub = sinon.stub(EcRepository, 'getAs'); + const id = 'id1'; + const repo = {}; + const eim = {}; + + EcAlignment.getBlocking(id, repo, eim); + + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(id, sinon.match.instanceOf(EcAlignment), null, null, repo, eim)).to.be.true; + + stub.restore(); + }); + }); + + describe('search', function () { + it('should call EcRepository.searchAs with correct parameters', function () { + const stub = sinon.stub(EcRepository, 'searchAs'); + const repo = {}; + const query = 'query'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + EcAlignment.search(repo, query, success, failure, paramObj, eim); + + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(repo, query, sinon.match.func, success, failure, paramObj, eim)).to.be.true; + + stub.restore(); + }); + }); + + describe('searchBySource', function () { + it('should call EcAlignment.search with correct query', function () { + const stub = sinon.stub(EcAlignment, 'search'); + const repo = {}; + const sourceId = 'sourceId'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + EcAlignment.searchBySource(repo, sourceId, success, failure, paramObj, eim); + + const expectedQuery = 'source:"sourceId"'; + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(repo, expectedQuery, success, failure, paramObj, eim)).to.be.true; + + stub.restore(); + }); + }); + + describe('searchBySources', function () { + it('should call EcAlignment.search with correct query', function () { + const stub = sinon.stub(EcAlignment, 'search'); + const repo = {}; + const sourceIds = ['sourceId1', 'sourceId2']; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + EcAlignment.searchBySources(repo, sourceIds, success, failure, paramObj, eim); + + const expectedQuery = '(source:"sourceId1" OR source:"sourceId2")'; + expect(stub.calledOnce).to.be.true; + console.log(stub.returnsThis()); + expect(stub.calledWith(repo)).to.be.true; + + stub.restore(); + }); + }); + + describe('searchByCompetency', function () { + it('should call EcAlignment.search with correct query', function () { + const stub = sinon.stub(EcAlignment, 'search'); + const repo = {}; + const competencyId = 'competencyId'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + EcAlignment.searchByCompetency(repo, competencyId, success, failure, paramObj, eim); + + const expectedQuery = ' AND (source:"competencyId" OR target:"competencyId")'; + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(repo, expectedQuery, success, failure, paramObj, eim)).to.be.true; + + stub.restore(); + }); + }); + + describe('save', function () { + it('should throw error if source is missing', function () { + const alignment = new EcAlignment(); + alignment.target = 'target'; + alignment.relationType = 'relationType'; + + expect(() => alignment.save()).to.throw('Source Competency cannot be missing'); + }); + + it('should throw error if target is missing', function () { + const alignment = new EcAlignment(); + alignment.source = 'source'; + alignment.relationType = 'relationType'; + + expect(() => alignment.save()).to.throw('Target Competency cannot be missing'); + }); + + it('should throw error if relationType is missing', function () { + const alignment = new EcAlignment(); + alignment.source = 'source'; + alignment.target = 'target'; + + expect(() => alignment.save()).to.throw('Relation Type cannot be missing'); + }); + + it('should call EcRepository.save if repo is null', function () { + const stub = sinon.stub(EcRepository, 'save'); + const alignment = new EcAlignment(); + alignment.source = 'source'; + alignment.target = 'target'; + alignment.relationType = 'relationType'; + const success = sinon.spy(); + const failure = sinon.spy(); + const eim = {}; + + alignment.save(success, failure, null, eim); + + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(alignment, success, failure, null, eim)).to.be.true; + + stub.restore(); + }); + + it('should call repo.saveTo if repo is not null', function () { + const repo = { + saveTo: sinon.spy() + }; + const alignment = new EcAlignment(); + alignment.source = 'source'; + alignment.target = 'target'; + alignment.relationType = 'relationType'; + const success = sinon.spy(); + const failure = sinon.spy(); + const eim = {}; + + alignment.save(success, failure, repo, eim); + + expect(repo.saveTo.calledOnce).to.be.true; + expect(repo.saveTo.calledWith(alignment, success, failure, eim)).to.be.true; + }); + }); + + describe('_delete', function () { + it('should call EcRepository.DELETE with correct parameters', function () { + const stub = sinon.stub(EcRepository, 'DELETE'); + const alignment = new EcAlignment(); + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const eim = {}; + + alignment._delete(success, failure, repo, eim); + + expect(stub.calledOnce).to.be.true; + expect(stub.calledWith(alignment, success, failure, repo, eim)).to.be.true; + + stub.restore(); + }); + }); +}); \ No newline at end of file diff --git a/src/test/EcAssertion.test.js b/src/test/EcAssertion.test.js new file mode 100644 index 00000000..0daf161f --- /dev/null +++ b/src/test/EcAssertion.test.js @@ -0,0 +1,93 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcAssertion = require('../org/cass/profile/EcAssertion'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); +const EcPpk = require('../com/eduworks/ec/crypto/EcPpk'); +const Assertion = require('../org/cassproject/schema/cass/profile/Assertion'); + +const expect = chai.expect; + +describe('EcAssertion', function () { + let assertion; + + beforeEach(function () { + assertion = new EcAssertion(); + }); + + describe('equals', function () { + it('should return true if ids are equal', function () { + assertion.id = 'testId'; + const obj = { id: 'testId' }; + expect(assertion.equals(obj)).to.be.true; + }); + + it('should return false if ids are not equal', function () { + assertion.id = 'testId'; + const obj = { id: 'differentId' }; + expect(assertion.equals(obj)).to.be.false; + }); + }); + + describe('decrypt', function () { + it('should decrypt and return a new assertion', async function () { + let subject = EcPpk.generateKey(); + let agent = EcPpk.generateKey(); + sinon.stub(assertion, 'getSubject').resolves(subject.toPk()); + sinon.stub(assertion, 'getAgent').resolves(agent.toPk()); + sinon.stub(assertion, 'getAssertionDate').resolves(1234); + sinon.stub(assertion, 'getExpirationDate').resolves(2345); + sinon.stub(assertion, 'getEvidences').resolves('evidences'); + sinon.stub(assertion, 'getNegative').resolves(true); + sinon.stub(assertion, 'getDecayFunction').resolves('decayFunction'); + + const decryptedAssertion = await assertion.decrypt(); + + expect(decryptedAssertion).to.be.an.instanceof(Assertion); + expect(decryptedAssertion.subject).to.equal(subject.toPk().toPem()); + expect(decryptedAssertion.agent).to.equal(agent.toPk().toPem()); + expect(decryptedAssertion.assertionDate).to.equal(1234); + expect(decryptedAssertion.expirationDate).to.equal(2345); + expect(decryptedAssertion.evidence).to.equal('evidences'); + expect(decryptedAssertion.negative).to.equal(true); + expect(decryptedAssertion.decayFunction).to.equal('decayFunction'); + }); + }); + + describe('get', function () { + it('should call EcRepository.getAs with correct parameters', function () { + const getAsStub = sinon.stub(EcRepository, 'getAs'); + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const eim = {}; + + EcAssertion.get('testId', success, failure, repo, eim); + + expect(getAsStub.calledOnce).to.be.true; + expect(getAsStub.calledWith('testId', sinon.match.instanceOf(EcAssertion), success, failure, repo, eim)).to.be.true; + + getAsStub.restore(); + }); + }); + + describe('search', function () { + it('should call EcRepository.searchAs with correct parameters', function () { + const searchAsStub = sinon.stub(EcRepository, 'searchAs'); + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const query = 'testQuery'; + const paramObj = {}; + const eim = {}; + + EcAssertion.search(repo, query, success, failure, paramObj, eim); + + expect(searchAsStub.calledOnce).to.be.true; + expect(searchAsStub.calledWith(repo, query, sinon.match.func, success, failure, paramObj, eim)).to.be.true; + + searchAsStub.restore(); + }); + }); + + // Add more tests for other methods as needed +}); \ No newline at end of file diff --git a/src/test/EcCompetency.test.js b/src/test/EcCompetency.test.js new file mode 100644 index 00000000..69923d3c --- /dev/null +++ b/src/test/EcCompetency.test.js @@ -0,0 +1,144 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcCompetency = require('../org/cass/competency/EcCompetency'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); +const EcAlignment = require('../org/cass/competency/EcAlignment'); +const EcRollupRule = require('../org/cass/competency/EcRollupRule'); +const EcLevel = require('../org/cass/competency/EcLevel'); +const EcPpk = require('../com/eduworks/ec/crypto/EcPpk'); + +const expect = chai.expect; + +describe('EcCompetency', function () { + let competency; + + beforeEach(function () { + competency = new EcCompetency(); + }); + + it('should initialize with template values', function () { + EcCompetency.template = { '@name': 'Test Competency' }; + competency = new EcCompetency(); + expect(competency.name).to.equal('Test Competency'); + }); + + it('should check equality based on id', function () { + competency.id = 'testId'; + const obj = { id: 'testId' }; + expect(competency.equals(obj)).to.be.true; + }); + + it('should retrieve competency asynchronously', function (done) { + const stub = sinon.stub(EcRepository, 'getAs').callsFake((id, obj, success) => { + success(obj); + }); + + EcCompetency.get('testId', function (result) { + expect(result).to.be.instanceOf(EcCompetency); + stub.restore(); + done(); + }, done); + }); + + it('should retrieve competency synchronously', function () { + const stub = sinon.stub(EcRepository, 'getAs').returns(new EcCompetency()); + + const result = EcCompetency.getBlocking('testId'); + expect(result).to.be.instanceOf(EcCompetency); + stub.restore(); + }); + + it('should search for competencies', function (done) { + const stub = sinon.stub(EcRepository, 'searchAs').callsFake((repo, query, factory, success) => { + success([new EcCompetency()]); + }); + + EcCompetency.search(null, 'query', function (results) { + expect(results).to.be.an('array').that.is.not.empty; + stub.restore(); + done(); + }, done); + }); + + it('should add an alignment', async function () { + const stub = sinon.stub(EcAlignment.prototype, 'save').resolves(); + const target = new EcCompetency(); + const owner = EcPpk.generateKey(); + + const alignment = await competency.addAlignment(target, 'alignmentType', owner, 'serverUrl'); + expect(alignment).to.be.instanceOf(EcAlignment); + stub.restore(); + }); + + it('should search for relationships', function (done) { + const stub = sinon.stub(EcAlignment, 'search').callsFake((repo, query, success) => { + success([new EcAlignment()]); + }); + + competency.relations(null, function (result) { + expect(result).to.be.instanceOf(EcAlignment); + stub.restore(); + done(); + }, done); + }); + + it('should add a level', async function () { + const stub = sinon.stub(EcLevel.prototype, 'save').resolves(); + const owner = EcPpk.generateKey(); + + const level = await competency.addLevel('name', 'description', owner, 'serverUrl'); + expect(level).to.be.instanceOf(EcLevel); + stub.restore(); + }); + + it('should search for levels', function (done) { + const stub = sinon.stub(EcLevel, 'search').callsFake((repo, query, success) => { + success([new EcLevel()]); + }); + + competency.levels(null, function (result) { + expect(result).to.be.instanceOf(EcLevel); + stub.restore(); + done(); + }, done); + }); + + it('should add a rollup rule', async function () { + const stub = sinon.stub(EcRollupRule.prototype, 'save').resolves(); + const owner = EcPpk.generateKey(); + + const rollupRule = await competency.addRollupRule('name', 'description', owner, 'serverUrl'); + expect(rollupRule).to.be.instanceOf(EcRollupRule); + stub.restore(); + }); + + it('should search for rollup rules', function (done) { + const stub = sinon.stub(EcRollupRule, 'search').callsFake((repo, query, success) => { + success([new EcRollupRule()]); + }); + + competency.rollupRules(null, function (result) { + expect(result).to.be.instanceOf(EcRollupRule); + stub.restore(); + done(); + }, done); + }); + + it('should set scope', function () { + competency.setScope('testScope'); + expect(competency.scope).to.equal('testScope'); + }); + + it('should save competency', function (done) { + const stub = sinon.stub(EcRepository, 'save').callsFake((obj, success) => { + success(); + }); + + competency.name = 'testName'; + competency.save(function () { + stub.restore(); + done(); + }, done); + }); + +}); \ No newline at end of file diff --git a/src/test/EcDirectedGraph.test.js b/src/test/EcDirectedGraph.test.js new file mode 100644 index 00000000..d6b4a2a5 --- /dev/null +++ b/src/test/EcDirectedGraph.test.js @@ -0,0 +1,161 @@ +const { expect } = require('chai'); +const EcDirectedGraph = require('../com/eduworks/ec/graph/EcDirectedGraph'); + +describe('EcDirectedGraph', () => { + let graph; + let vertex1, vertex2, vertex3, edge1, edge2; + + beforeEach(() => { + graph = new EcDirectedGraph(); + vertex1 = { equals: (v) => v === vertex1 }; + vertex2 = { equals: (v) => v === vertex2 }; + vertex3 = { equals: (v) => v === vertex3 }; + edge1 = { equals: (e) => e === edge1 }; + edge2 = { equals: (e) => e === edge2 }; + }); + + it('should add vertices', () => { + graph.addVertex(vertex1); + graph.addVertex(vertex2); + expect(graph.getVertexCount()).to.equal(2); + expect(graph.containsVertex(vertex1)).to.be.true; + expect(graph.containsVertex(vertex2)).to.be.true; + }); + + it('should add edges', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.getEdgeCount()).to.equal(1); + expect(graph.containsEdge(edge1)).to.be.true; + }); + + it('should find edges between vertices', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.findEdge(vertex1, vertex2)).to.equal(edge1); + }); + + it('should get neighbors of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.getNeighbors(vertex1)).to.have.members([vertex2, vertex3]); + }); + + it('should get incident edges of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.getIncidentEdges(vertex1)).to.have.members([edge1, edge2]); + }); + + it('should get incident vertices of an edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.getIncidentVertices(edge1)).to.have.members([vertex1, vertex2]); + }); + + it('should remove vertices', () => { + graph.addVertex(vertex1); + graph.addVertex(vertex2); + graph.removeVertex(vertex1); + expect(graph.getVertexCount()).to.equal(1); + expect(graph.containsVertex(vertex1)).to.be.false; + }); + + it('should remove edges', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.removeEdge(edge1); + expect(graph.getEdgeCount()).to.equal(0); + expect(graph.containsEdge(edge1)).to.be.false; + }); + + it('should check if vertices are neighbors', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.isNeighbor(vertex1, vertex2)).to.be.true; + expect(graph.isNeighbor(vertex1, vertex3)).to.be.false; + }); + + it('should check if a vertex is incident to an edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.isIncident(vertex1, edge1)).to.be.true; + expect(graph.isIncident(vertex3, edge1)).to.be.false; + }); + + it('should get the degree of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.degree(vertex1)).to.equal(2); + }); + + it('should get the in-degree of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex3, vertex2); + expect(graph.inDegree(vertex2)).to.equal(2); + }); + + it('should get the out-degree of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.outDegree(vertex1)).to.equal(2); + }); + + it('should get the source of a directed edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.getSource(edge1)).to.equal(vertex1); + }); + + it('should get the destination of a directed edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.getDest(edge1)).to.equal(vertex2); + }); + + it('should get predecessors of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex3, vertex2); + expect(graph.getPredecessors(vertex2)).to.have.members([vertex1,vertex3]); + }); + + it('should get successors of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.getSuccessors(vertex1)).to.have.members([vertex2,vertex3]); + }); + + it('should check if a vertex is a predecessor of another vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.isPredecessor(vertex2, vertex1)).to.be.true; + expect(graph.isPredecessor(vertex1, vertex2)).to.be.false; + }); + + it('should check if a vertex is a successor of another vertex', () => { + graph.addEdge(edge1, vertex2, vertex1); + expect(graph.isSuccessor(vertex2, vertex1)).to.be.true; + expect(graph.isSuccessor(vertex1, vertex2)).to.be.false; + }); + + it('should get the predecessor count of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex3, vertex2); + expect(graph.getPredecessorCount(vertex2)).to.equal(2); + }); + + it('should get the successor count of a vertex', () => { + graph.addEdge(edge1, vertex1, vertex2); + graph.addEdge(edge2, vertex1, vertex3); + expect(graph.getSuccessorCount(vertex1)).to.equal(2); + }); + + it('should check if a vertex is the source of an edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.isSource(vertex1, edge1)).to.be.true; + expect(graph.isSource(vertex2, edge1)).to.be.false; + }); + + it('should check if a vertex is the destination of an edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.isDest(vertex2, edge1)).to.be.true; + expect(graph.isDest(vertex1, edge1)).to.be.false; + }); + + it('should get the opposite vertex of an edge', () => { + graph.addEdge(edge1, vertex1, vertex2); + expect(graph.getOpposite(vertex1, edge1)).to.equal(vertex2); + expect(graph.getOpposite(vertex2, edge1)).to.equal(vertex1); + }); +}); \ No newline at end of file diff --git a/src/test/EcFramework.test.js b/src/test/EcFramework.test.js new file mode 100644 index 00000000..c511730f --- /dev/null +++ b/src/test/EcFramework.test.js @@ -0,0 +1,137 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcFramework = require('../org/cass/competency/EcFramework'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); +const expect = chai.expect; + +describe('EcFramework', function () { + let framework; + + beforeEach(function () { + framework = new EcFramework(); + }); + + it('should initialize with template values if template is set', function () { + EcFramework.template = { "@name": "Test Framework" }; + framework = new EcFramework(); + expect(framework.name).to.equal("Test Framework"); + }); + + it('should return true if IDs match in equals method', function () { + framework.id = "testId"; + const obj = { id: "testId" }; + expect(framework.equals(obj)).to.be.true; + }); + + it('should add competency if not already present', function () { + const competencyId = "testCompetency"; + framework.addCompetency(competencyId); + expect(framework.competency).to.include(competencyId); + }); + + it('should not add duplicate competency', function () { + const competencyId = "testCompetency"; + framework.addCompetency(competencyId); + framework.addCompetency(competencyId); + expect(framework.competency.length).to.equal(1); + }); + + it('should remove competency and associated relations and levels', function (done) { + const competencyId = "testCompetency"; + framework.competency = [competencyId]; + framework.relation = ["relation1"]; + framework.level = ["level1"]; + + sinon.stub(framework, 'removeRelationshipsThatInclude').callsFake((id, i, success) => success()); + sinon.stub(framework, 'removeLevelsThatInclude').callsFake((id, i, success) => success()); + + framework.removeCompetency(competencyId, () => { + expect(framework.competency).to.not.include(competencyId); + done(); + }, done); + }); + + it('should add relation if not already present', function () { + const relationId = "testRelation"; + framework.addRelation(relationId); + expect(framework.relation).to.include(relationId); + }); + + it('should not add duplicate relation', function () { + const relationId = "testRelation"; + framework.addRelation(relationId); + framework.addRelation(relationId); + expect(framework.relation.length).to.equal(1); + }); + + it('should remove relation', function () { + const relationId = "testRelation"; + framework.relation = [relationId]; + framework.removeRelation(relationId); + expect(framework.relation).to.not.include(relationId); + }); + + it('should add level if not already present', function () { + const levelId = "testLevel"; + framework.addLevel(levelId); + expect(framework.level).to.include(levelId); + }); + + it('should not add duplicate level', function () { + const levelId = "testLevel"; + framework.addLevel(levelId); + framework.addLevel(levelId); + expect(framework.level.length).to.equal(1); + }); + + it('should remove level', function () { + const levelId = "testLevel"; + framework.level = [levelId]; + framework.removeLevel(levelId); + expect(framework.level).to.not.include(levelId); + }); + + it('should add rollup rule if not already present', function () { + const rollupRuleId = "testRollupRule"; + framework.addRollupRule(rollupRuleId); + expect(framework.rollupRule).to.include(rollupRuleId); + }); + + it('should not add duplicate rollup rule', function () { + const rollupRuleId = "testRollupRule"; + framework.addRollupRule(rollupRuleId); + framework.addRollupRule(rollupRuleId); + expect(framework.rollupRule.length).to.equal(1); + }); + + it('should remove rollup rule', function () { + const rollupRuleId = "testRollupRule"; + framework.rollupRule = [rollupRuleId]; + framework.removeRollupRule(rollupRuleId); + expect(framework.rollupRule).to.not.include(rollupRuleId); + }); + + it('should save framework if name is not empty', function (done) { + framework.name = "Test Framework"; + const stub = sinon.stub(EcRepository, 'save').callsFake((obj, success) => success()); + framework.save(() => { + expect(EcRepository.save.calledOnce).to.be.true; + stub.restore(); + done(); + }, done); + }); + + it('should not save framework if name is empty', function () { + framework.name = ""; + expect(() => framework.save()).to.throw("Framework Name Cannot be Empty"); + }); + + it('should delete framework', function (done) { + const stub = sinon.stub(EcRepository, 'DELETE').callsFake((obj, success) => success()); + framework._delete(() => { + expect(EcRepository.DELETE.calledOnce).to.be.true; + stub.restore(); + done(); + }, done); + }); +}); \ No newline at end of file diff --git a/src/test/EcLevel.test.js b/src/test/EcLevel.test.js new file mode 100644 index 00000000..0aa634c3 --- /dev/null +++ b/src/test/EcLevel.test.js @@ -0,0 +1,156 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); +const EcLevel = require('../org/cass/competency/EcLevel'); +const EcAlignment = require('../org/cass/competency/EcAlignment'); +const EcPpk = require('../com/eduworks/ec/crypto/EcPpk'); +const EcIdentityManager = require('../org/cassproject/ebac/identity/EcIdentityManager'); + +const { expect } = chai; + +describe('EcLevel', () => { + let sandbox; + + beforeEach(() => { + sandbox = sinon.createSandbox(); + }); + + afterEach(() => { + sandbox.restore(); + }); + + describe('get', () => { + it('should retrieve a level by ID', (done) => { + const id = 'testId'; + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const eim = {}; + + sandbox.stub(EcRepository, 'getAs').callsFake((id, level, successCallback, failureCallback, repo, eim) => { + successCallback(level); + }); + + EcLevel.get(id, success, failure, repo, eim); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + done(); + }); + }); + + describe('getBlocking', () => { + it('should retrieve a level by ID synchronously', () => { + const id = 'testId'; + const repo = {}; + const eim = {}; + + sandbox.stub(EcRepository, 'getAs').returns(new EcLevel()); + + const result = EcLevel.getBlocking(id, repo, eim); + + expect(result).to.be.instanceOf(EcLevel); + }); + }); + + describe('search', () => { + it('should search for levels with a query', (done) => { + const repo = {}; + const query = 'testQuery'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + sandbox.stub(EcRepository, 'searchAs').callsFake((repo, query, levelFactory, successCallback, failureCallback, paramObj, eim) => { + successCallback([new EcLevel()]); + }); + + EcLevel.search(repo, query, success, failure, paramObj, eim); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + done(); + }); + }); + + describe('searchByCompetency', () => { + it('should search for levels by competency ID', (done) => { + const repo = {}; + const competencyId = 'testCompetencyId'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + sandbox.stub(EcLevel, 'search').callsFake((repo, query, successCallback, failureCallback, paramObj, eim) => { + successCallback([new EcLevel()]); + }); + + EcLevel.searchByCompetency(repo, competencyId, success, failure, paramObj, eim); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + done(); + }); + + it('should call failure callback if no competency ID is provided', () => { + const repo = {}; + const competencyId = ''; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + EcLevel.searchByCompetency(repo, competencyId, success, failure, paramObj, eim); + + expect(failure.calledOnce).to.be.true; + expect(success.called).to.be.false; + }); + }); + + describe('addRelationship', () => { + it('should add a relationship between levels', async () => { + const level = new EcLevel(); + level.id = 'sourceId'; + const targetLevel = new EcLevel(); + targetLevel.id = 'targetId'; + const alignmentType = 'testType'; + const identity = EcPpk.generateKey(); + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = new EcRepository(); + await repo.init(process.env.CASS_LOOPBACK || 'http://localhost/api/'); + const eim = EcIdentityManager.default; + + let a = await level.addRelationship(targetLevel, alignmentType, identity, success, failure, repo, eim); + + expect(a.source).to.equal(level.id); + expect(a.target).to.equal(targetLevel.id); + expect(a.relationType).to.equal(alignmentType); + }); + }); + + describe('setName', () => { + it('should set the name of the level', () => { + const level = new EcLevel(); + const name = 'testName'; + + level.setName(name); + + expect(level.name).to.equal(name); + }); + }); + + describe('setDescription', () => { + it('should set the description of the level', () => { + const level = new EcLevel(); + const description = 'testDescription'; + + level.setDescription(description); + + expect(level.description).to.equal(description); + }); + }); + +}); \ No newline at end of file diff --git a/src/test/EcRemote.test.js b/src/test/EcRemote.test.js new file mode 100644 index 00000000..aa2996cc --- /dev/null +++ b/src/test/EcRemote.test.js @@ -0,0 +1,51 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcRemote = require('../com/eduworks/ec/remote/EcRemote'); + +const { expect } = chai; +if ((typeof Cypress !== 'undefined') && Cypress != null && Cypress.env != null) + process.env.CASS_LOOPBACK = Cypress.env('CASS_LOOPBACK'); + +describe('EcRemote', function () { + describe('postExpectingObject', function () { + it('should call postInner with correct parameters', async function () { + const server = process.env.CASS_LOOPBACK || 'http://localhost/api/'; + const service = 'ping'; + const fd = new FormData(); + const success = sinon.spy(); + const failure = sinon.spy(); + + await EcRemote.postExpectingObject(server, service, fd, success, failure); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + }); + }); + + describe('getExpectingObject', function () { + it('should call fetch with correct URL', async function () { + const server = process.env.CASS_LOOPBACK || 'http://localhost/api/'; + const service = 'ping'; + const success = sinon.spy(); + const failure = sinon.spy(); + + await EcRemote.getExpectingObject(server, service, success, failure); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + }); + }); + + describe('_delete', function () { + it('should call fetch with DELETE method', async function () { + const url = (process.env.CASS_LOOPBACK || 'http://localhost/api/')+'ping'; + const success = sinon.spy(); + const failure = sinon.spy(); + + await EcRemote._delete(url, null, success, failure); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + }); + }); +}); \ No newline at end of file diff --git a/src/test/EcRollupRule.test.js b/src/test/EcRollupRule.test.js new file mode 100644 index 00000000..38377c05 --- /dev/null +++ b/src/test/EcRollupRule.test.js @@ -0,0 +1,89 @@ +const chai = require('chai'); +const sinon = require('sinon'); +const EcRepository = require('../org/cassproject/ebac/repository/EcRepository'); +const EcRollupRule = require('../org/cass/competency/EcRollupRule'); +const expect = chai.expect; + +describe('EcRollupRule', function () { + describe('get', function () { + it('should retrieve a rollup rule from the server', function (done) { + const id = 'testId'; + const success = sinon.spy(); + const failure = sinon.spy(); + const repo = {}; + const eim = {}; + + sinon.stub(EcRepository, 'getAs').callsFake((id, obj, successCallback, failureCallback, repo, eim) => { + successCallback(obj); + }); + + EcRollupRule.get(id, success, failure, repo, eim); + + expect(success.calledOnce).to.be.true; + expect(failure.called).to.be.false; + EcRepository.getAs.restore(); + done(); + }); + }); + + describe('getBlocking', function () { + it('should retrieve a rollup rule from the server without callbacks', function () { + const id = 'testId'; + const repo = {}; + const eim = {}; + + sinon.stub(EcRepository, 'getAs').returns(new EcRollupRule()); + + const result = EcRollupRule.getBlocking(id, repo, eim); + + expect(result).to.be.an.instanceof(EcRollupRule); + EcRepository.getAs.restore(); + }); + }); + + describe('search', function () { + it('should search for rollup rules with a query', function (done) { + const repo = {}; + const query = 'testQuery'; + const success = sinon.spy(); + const failure = sinon.spy(); + const paramObj = {}; + const eim = {}; + + sinon.stub(EcRepository, 'searchAs').callsFake((repo, query, objFactory, successCallback, failureCallback, paramObj, eim) => { + successCallback([objFactory()]); + }); + + EcRollupRule.search(repo, query, success, failure, paramObj, eim); + + expect(success.calledOnce).to.be.true; + expect(success.args[0][0][0]).to.be.an.instanceof(EcRollupRule); + expect(failure.called).to.be.false; + EcRepository.searchAs.restore(); + done(); + }); + }); + + describe('setName', function () { + it('should set the name of the rollup rule', function () { + const rule = new EcRollupRule(); + const name = 'Test Name'; + + rule.setName(name); + + expect(rule.name).to.equal(name); + }); + }); + + describe('setDescription', function () { + it('should set the description of the rollup rule', function () { + const rule = new EcRollupRule(); + const description = 'Test Description'; + + rule.setDescription(description); + + expect(rule.description).to.equal(description); + }); + }); + +}); \ No newline at end of file