Skip to content

Commit

Permalink
fix(bazel): fix integration test for bazel building (#38629)
Browse files Browse the repository at this point in the history
Update the API used to request a timestamp.  The previous API we relied on for this
test application, worldclockapi.com no longer serves times and simply 403s on all
requests.  This caused our test to timeout as the HTTP request did not handle a failure
case.  By moving to a new api, the HTTP request responds as expected and timeouts
are corrected as there is not longer a pending microtask in the queue.

PR Close #38629
  • Loading branch information
josephperrott committed Aug 28, 2020
1 parent 2dd29fb commit 930eeaf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions integration/bazel/src/app.component.ts
Expand Up @@ -15,6 +15,6 @@ export class AppComponent {
constructor(private http: HttpClient) {}

time$: Observable<string> =
this.http.get('http://worldclockapi.com/api/json/pst/now')
.pipe(map((result: any) => result.currentDateTime), startWith(['...']));
this.http.get('http://worldtimeapi.org/api/timezone/America/Los_Angeles.json')
.pipe(map((result: any) => result.datetime), startWith(['...']));
}

0 comments on commit 930eeaf

Please sign in to comment.