Skip to content

None of the e2e-spec.ts files actually require real GitHub OAuth credentials, making CI's GITHUB_CLIENT_ID/SECRET gate unnecessary #165

Description

@chonilius

Beyond the gate being unmet (filed separately), it's also the wrong gate: grepping test/*.e2e-spec.ts for GITHUB_CLIENT_ID, GITHUB_CLIENT_SECRET, GithubAuthGuard, or any HTTP-mocking library (nock and similar aren't dependencies at all) returns nothing. Every e2e spec (app.e2e-spec.ts, users.e2e-spec.ts, escrow-idempotency.e2e-spec.ts, the three stellar-address-validation-*.e2e-spec.ts files) builds its TestingModule from specific controllers/services with mocked providers and overrideGuard(...)-stubbed auth guards — none of them drive a real passport-github2 OAuth handshake, which is the only thing GITHUB_CLIENT_ID/GITHUB_CLIENT_SECRET are used for (GithubStrategy's constructor). The CI condition should very likely be checking DATABASE_URL reachability (already provided via the postgres service container) rather than GitHub OAuth credentials — as written, it can never actually run the e2e suite in this repository's CI unless someone adds credentials the tests don't need.

Metadata

Metadata

Assignees

Labels

Stellar WaveIssues in the Stellar wave programbugSomething isn't workinghelp wantedExtra attention is neededtestingTesting/QA infrastructurevery hardVery difficult task, expert-level effort required

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions