Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Run tests using greenmail server #98

Merged
merged 1 commit into from Aug 27, 2018
Merged

Conversation

ralphtheninja
Copy link
Member

Closes #69, closes #24

let dc = null

test('setUp dc context', t => {
t.plan(3)
dc = new DeltaChat({
addr: env.DC_ADDR,
mail_pw: env.DC_MAIL_PW,
addr: 'delta1@delta.localhost',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we're not using encrypted environment variables on Travis anymore it means external pull requests should also work since we're no longer required to keep them secret since we're running locally.

cwd: tempy.directory()
})
dc.once('ready', () => t.pass('ready event fired'))
dc.on('DC_EVENT_ERROR', (data1, data2) => {
throw new Error(data1 || data2)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will throw if e.g. the server isn't started. Will update docs later.

@ralphtheninja
Copy link
Member Author

Now tests are working again so we can continue to implement more tests to improve coverage.

@r10s
Copy link
Member

r10s commented Aug 27, 2018

lgtm, however, this does not mean anything since i am neither a travis nor a node expert :)

@ralphtheninja
Copy link
Member Author

lgtm, however, this does not mean anything since i am neither a travis nor a node expert :)

You're an IMAP/SMTP expert. The rest is just details :)

@ralphtheninja ralphtheninja merged commit 37cd7ca into master Aug 27, 2018
@ralphtheninja ralphtheninja deleted the test-with-greenmail-server branch August 27, 2018 12:53
@r10s
Copy link
Member

r10s commented Aug 27, 2018

You're an IMAP/SMTP expert.

i do not think even that.

@r10s
Copy link
Member

r10s commented Aug 27, 2018

The rest is just details :)

this is true, however :)

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

Successfully merging this pull request may close these issues.

Travis having problems (again). Seems to hang on setup. Integration tests
2 participants