Skip to content
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

Notifications test not working #1735

Closed
robward-basemap opened this issue Jan 18, 2021 · 5 comments · Fixed by #2323
Closed

Notifications test not working #1735

robward-basemap opened this issue Jan 18, 2021 · 5 comments · Fixed by #2323

Comments

@robward-basemap
Copy link

Expected Behavior

Running "docker run -it --rm --env-file ./dotenv ghcr.io/jef/streetmerchant:latest test:notification:production" should send a test notification

Current Behavior

An error is thrown

Steps to Reproduce

Run "docker run -it --rm --env-file ./dotenv ghcr.io/jef/streetmerchant:latest test:notification:production"

Environment

  • OS: Docker

Logs

streetmerchant@ test:notification:production /app
node build/test/functional/test-notification.js

internal/modules/cjs/loader.js:883
throw err;
^

Error: Cannot find module '/app/build/test/functional/test-notification.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
at Function.Module._load (internal/modules/cjs/loader.js:725:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! streetmerchant@ test:notification:production: node build/test/functional/test-notification.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the streetmerchant@ test:notification:production script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR! /home/appuser/.npm/_logs/2021-01-18T21_11_14_742Z-debug.log

@magicmq
Copy link

magicmq commented Jan 19, 2021

I am also receiving the same issue when I try to test notifications on nightly build 9695e29. I am only using Pushbullet for notifications.

@ryanqyao
Copy link

I'm having the same issue, however when I run npm run test:notification, it works fine. I can get notifications in Telegram. So I assume the code works, just the test-notificaiton not working with Docker.

@PacketShepard
Copy link

Same problem here. The path may be wrong based on the Dockerfile. It looks like build/build/ (duplicate directory) is being created and this is breaking the test notification path. I've noticed something similar with the web-api

@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

@jef
Copy link
Owner

jef commented Mar 17, 2021

I'll test this in the near future. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants