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

Failing test for peculiar reason: "Uncaught TypeError: Cannot read property 'fil... #44

Closed
todo bot opened this issue Jun 9, 2019 · 0 comments
Closed
Labels
help wanted Extra attention is needed todo 🗒️

Comments

@todo
Copy link

todo bot commented Jun 9, 2019

// @TODO: Failing test for peculiar reason: "Uncaught TypeError: Cannot read property 'filter' of undefined thrown"... similar test with same pattern have no trouble
expect(new InboundMail()).toBeTruthy();
});
});


This issue was generated by todo based on a TODO comment in 694e1f8. It's been assigned to @tboeckmann because they committed the code.
@todo todo bot assigned tboeckmann Jun 9, 2019
@todo todo bot added the todo 🗒️ label Jun 9, 2019
@tboeckmann tboeckmann removed their assignment Jun 9, 2019
@tboeckmann tboeckmann added the help wanted Extra attention is needed label Jun 9, 2019
tboeckmann added a commit that referenced this issue Jul 22, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* add coveralls status

* add coveralls

* add coveralls config

* refactor addresses endpoint into its own service

* remove unneeded import

* add protocols service

* add setNetworkList
remove getPublicNetworks method

* fix testing errors
closes #44

* add more coverage

* add some more unit tests

* add tests and document method

* add tests for address pipe
add method documentation

* require args for pipe

* add gitter badge

* add coverage: pipe tests

* remove changeActiveItem method

* inbox tests

* add test to inbox component;
refactor and review inbox component

* add tests
refactor getMails() and move methods to services
tboeckmann added a commit that referenced this issue Jul 24, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* add coveralls status

* add coveralls

* add coveralls config

* refactor addresses endpoint into its own service

* remove unneeded import

* add protocols service

* add setNetworkList
remove getPublicNetworks method

* fix testing errors
closes #44

* add more coverage

* add some more unit tests

* add tests and document method

* add tests for address pipe
add method documentation

* require args for pipe

* add gitter badge

* add coverage: pipe tests

* remove changeActiveItem method

* inbox tests

* add test to inbox component;
refactor and review inbox component

* add tests
refactor getMails() and move methods to services

* clean up form
add tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed todo 🗒️
Projects
None yet
Development

No branches or pull requests

1 participant