Skip to content

Commit 303fdc7

Browse files
authored
add solutions to possible error which may occur while running tests (#1008)
* add solutions to possible error while running tests * rectifying a minor mistake * minor change * added one more possible error case * added file path to test scripts for easier navigation * add explicit place to add the flag * added placeholder to both the test script * removed the --project flag error since it has been taken care of in another PR * remove placeholders * changed login URLs in Contributing.md * added extra info for firebase.json issue * added staging-api login * correct url for staging
1 parent 45ac60d commit 303fdc7

File tree

1 file changed

+20
-3
lines changed

1 file changed

+20
-3
lines changed

CONTRIBUTING.md

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,15 +101,19 @@ The following project structure should be followed:
101101
- Authorization header: `Authorization: Bearer <token>`
102102
103103
## Production login:
104-
https://github.com/login/oauth/authorize?client_id=23c78f66ab7964e5ef97
104+
105+
https://api.realdevsquad.com/auth/github/login
105106
106107
### Production login - Cookie:
108+
107109
rds-session
108110
109111
## Staging login:
110-
https://github.com/login/oauth/authorize?client_id=ebb301661e883a85bf9a
111112
112-
### Staging login - Cookie:
113+
https://staging-api.realdevsquad.com/auth/github/login
114+
115+
### Staging login - Cookie:
116+
113117
rds-session-staging
114118
115119
## Testing Guidelines
@@ -155,3 +159,16 @@ export FIRESTORE_EMULATOR_HOST="localhost:<Firebase emulator PORT>"
155159
- Every pull request should have associated issue(s) on our [issue tracker](https://github.com/Real-Dev-Squad/website-backend/issues).
156160
- For any non-trivial fixes and features, unit and integration tests must be added. The PR reviewer should not approve/merge PR(s) that lack these.
157161
- The PR(s) should be merged only after the CI passes.
162+
163+
## Certain issues you may face while running the tests:
164+
165+
- Java version is not above 11
166+
- When we run yarn test, it runs both the unit and integration tests (in this order). So after the unit tests are done, the java process is not killed automatically and when our integration test run it gives error.
167+
- Error: connect ECONNREFUSED ::1:8081
168+
169+
## Possible solutions for above issues (in particular order):
170+
171+
- Java version above 11 is needed for firebase tool version >= 11
172+
- Either manually kill the java process after unit tests are done or run both the tests separately by running the test commands.
173+
- Add 'host : 0.0.0.0' to both firestore and ui object in firebase.json file if it is not added.
174+
for more info refer this : https://github.com/Real-Dev-Squad/website-backend/issues/918

0 commit comments

Comments
 (0)