uhh-lt / autolinks

Automatic Proactive Researching
https://uhh-lt.github.io/autolinks/
Apache License 2.0
1 stars 0 forks source link

System Testing #64

Open remstef opened 6 years ago

remstef commented 6 years ago

This issue for testing system

TypeError: Cannot read property 'name' of undefined
    at module.exports (/opt/project/frontendserver/api/service/call.js:2:21)
    at call (/opt/project/frontendserver/api/service/serviceEndpoint.js:35:21)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/opt/project/frontendserver/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at /opt/project/frontendserver/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/opt/project/frontendserver/node_modules/express/lib/router/index.js:335:12)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/index.js:275:10)
    at Function.handle (/opt/project/frontendserver/node_modules/express/lib/router/index.js:174:3)

TypeError: Cannot read property 'mv' of undefined
    at postDocuments (/opt/project/frontendserver/api/storage/storageEndpoint.js:63:10)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/opt/project/frontendserver/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at /opt/project/frontendserver/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/opt/project/frontendserver/node_modules/express/lib/router/index.js:335:12)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/index.js:275:10)
    at Function.handle (/opt/project/frontendserver/node_modules/express/lib/router/index.js:174:3)
    at router (/opt/project/frontendserver/node_modules/express/lib/router/index.js:47:12)

TypeError: Cannot read property 'name' of undefined
    at module.exports (/opt/project/frontendserver/api/service/call.js:2:21)
    at call (/opt/project/frontendserver/api/service/serviceEndpoint.js:35:21)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/opt/project/frontendserver/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/opt/project/frontendserver/node_modules/express/lib/router/layer.js:95:5)
    at /opt/project/frontendserver/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/opt/project/frontendserver/node_modules/express/lib/router/index.js:335:12)
    at next (/opt/project/frontendserver/node_modules/express/lib/router/index.js:275:10)
    at Function.handle (/opt/project/frontendserver/node_modules/express/lib/router/index.js:174:3)
remstef commented 6 years ago

See also https://github.com/uhh-lt/autolinks/wiki/System-Testing-Workflow

alvinrindra commented 6 years ago

Hi @remstef, my current commits and merges to the master fix some bugs, and add new feature 'merge Nodes To Compound'. At the moment it works in the frontend, but unfortunately, it only saves one node in the backend. What I mean here, if we create new nodes with one compound, the backend only saves the compound one, not with its children. We can discuss further later when I come to the office.

SOLUTION: create a new resource with a list of resources.value: [] first with cid: -1 example: { "before": {"rid":1534,"value":[ {"rid":153456}, {"rid":153345} ], "metadata":{}}, "after": {"cid": 46, "rid":1534,"value":"djzjdt","metadata":{ "label": "asdasf"}} }

remstef commented 6 years ago

@alvinrindra whats the status on the open issues?

remstef commented 6 years ago

Error: ENOENT: no such file or directory, unlink './uploads/Tcell.txt'
at Object.fs.unlinkSync (fs.js:1081:18) at Request._callback (/opt/project/frontendserver/api/storage/storageEndpoint.js:95:12)
at Request.self.callback (/opt/project/frontendserver/node_modules/request/request.js:185:22)
at Request.emit (events.js:159:13) at Request. (/opt/project/frontendserver/node_modules/request/request.js:1157:10)
at Request.emit (events.js:159:13) at IncomingMessage. (/opt/project/frontendserver/node_modules/request/request.js:1079:12) at Object.onceWrapper (events.js:254:19) at IncomingMessage.emit (events.js:164:20) at endReadableNT (_stream_readable.js:1054:12) npm ERR! code ELIFECYCLE npm ERR! errno 1 npm ERR! autolinks@1.0.0 start: node server.js npm ERR! Exit status 1 npm ERR! npm ERR! Failed at the autolinks@1.0.0 start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

╭─────────────────────────────────────╮ │ │ │ Update available 5.5.1 → 6.2.0 │ │ Run npm i -g npm to update │ │ │ ╰─────────────────────────────────────╯

npm ERR! A complete log of this run can be found in:
npm ERR! /root/.npm/_logs/2018-07-19T13_32_23_445Z-debug.log



- Visualiztion test (search)
   - [x] include loading indicator
   - [x] cancel invoked relation adding
   - [x] use rid as "label" for compound nodes without metadata label
   - [x] adding a relation from a node outside of a compund node to a node which is inside a compound node creates a new node outside of the compound node after refreshing ---> expected behavior, show immediately
    - [x] merge rename to "move into", make button on node itself
    - [x] Compound rename to "create new comound" 
    - [x] clear -> also clear any temporary variables 
    - [ ] creating compound from nodes which are within compounds should not move them out of the first compound, #new compound node is created inside the previous parent of current children nodes
alvinrindra commented 6 years ago

Hi @remstef, we still need to keep "CreateCompound" feature, since you said that we cannot change value type of resources. As we know, to create a compound node, we need to have resource values as array [ ] by default. If we create a compound node from an existing node by using "Move Into" feature to another nonparent target node, it means that we are going to change the value of that nonparent target node from "string" to [array], which is not possible by the backend.

-> SOLUTION: Make the compound to be the rid / metadata label if any for the compound, not the first index array anymore!!!!.

whatsapp image 2018-08-09 at 5 21 37 pm

alvinrindra commented 6 years ago

Hi @remstef , I have some issues from the backend. You now change every same label node with different rids.

image

What if a user wants to edit that annotation container? such update its label or even move to another compound nodes?

image