-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
Fix broken surface tests #52
Comments
For webid-provider-tests:v2.0.3 I see 36/36 passed. For solid-crud-tests:v5.1.0 I see: For web-access-control-tests:v6.0.0 I see: I'll create a PR to fix the last few failing tests! |
Adapt to solid-contrib/web-access-control-tests#42 Tested as part of pdsinterop/php-solid-server#52
Let's see which tests pass in #53 [...]
Status: Image is up to date for solidtestsuite/solid-crud-tests:v5.1.0
docker.io/solidtestsuite/solid-crud-tests:v5.1.0
v6.0.0: Pulling from solidtestsuite/web-access-control-tests
Digest: sha256:787d0bb4eee51e9e986a8d4c6c64f0b304cab071429d0f762928f428ba12e7b2
Status: Image is up to date for solidtestsuite/web-access-control-tests:v6.0.0
docker.io/solidtestsuite/web-access-control-tests:v6.0.0
Getting cookie for Alice...
(node:8) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
(Use `node --trace-warnings ...` to show where the warning was created)
/app/node_modules/node-fetch/lib/index.js:1461
reject(new FetchError(`request to ${request.url} failed, reason: ${err.message}`, 'system', err));
^
FetchError: request to https://server/login/ failed, reason: getaddrinfo EAI_AGAIN server
at ClientRequest.<anonymous> (/app/node_modules/node-fetch/lib/index.js:1461:11)
at ClientRequest.emit (node:events:390:28)
at TLSSocket.socketErrorListener (node:_http_client:447:9)
at TLSSocket.emit (node:events:390:28)
at emitErrorNT (node:internal/streams/destroy:157:8)
at emitErrorCloseNT (node:internal/streams/destroy:122:3)
at processTicksAndRejections (node:internal/process/task_queues:83:21) {
type: 'system',
errno: 'EAI_AGAIN',
code: 'EAI_AGAIN'
}
Error obtaining cookie for Alice, stopping. |
Hm, there's definitely something odd going on in the github actions - the solid crud tests pass when I run them locally with Docker, but not when they are run on GitHub Actions. Different Docker version maybe? |
The GitHub Action runs on
|
@mrvahedi68 and I are running [email protected] now |
Result:
|
|
|
Currently all WAC surface tests seem to be broken:
Before a new release is tagged, this needs to be resolved.
In the logs, the following error occurs 83 times (so basically for every test):
The following occurs 94 times:
Where
undefined
is usuallysolidLogicAlice
.@michielbdejong Do you happen to know of the top of your head what would cause this? (Or how to resolve this?)
Full logs
test/surface/update.test.ts
test/surface/create.test.ts
test/surface/read-public.test.ts
test/surface/read.test.ts
test/surface/read-logged_in.test.ts
test/surface/wac-allow.test.ts
test/surface/acl-doc-application.test.ts
test/surface/delete.test.ts
test/surface/fetch-pod-root-acl.test.ts
The webid-provider and solid-crud tests seem stable:
The text was updated successfully, but these errors were encountered: