As it is now, the tests require an actual FN account/connection, which is a bit awkward for a number of reasons:
- Auth
- Rate Limiting
- Flaky Tests
- False Negatives
For example, addressing point 3 and 4, if we happen to change any of the required underlying prerequisites in the account used for testing, the tests might fail because of a false assumption, resulting in false negatives.
I think we should either remove the tests all together, or alternatively mock the endpoints, which might not be reasonable as it would result in the tests testing the mocks more than anything.
As it is now, the tests require an actual FN account/connection, which is a bit awkward for a number of reasons:
For example, addressing point 3 and 4, if we happen to change any of the required underlying prerequisites in the account used for testing, the tests might fail because of a false assumption, resulting in false negatives.
I think we should either remove the tests all together, or alternatively mock the endpoints, which might not be reasonable as it would result in the tests testing the mocks more than anything.