Skip to content

chore: Stop using httpbin.org for tests#2695

Merged
MSNev merged 1 commit intomicrosoft:betafrom
MSNev:MSNev/RemoveHttpBin
Feb 3, 2026
Merged

chore: Stop using httpbin.org for tests#2695
MSNev merged 1 commit intomicrosoft:betafrom
MSNev:MSNev/RemoveHttpBin

Conversation

@MSNev
Copy link
Collaborator

@MSNev MSNev commented Feb 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 3, 2026 21:28
@MSNev MSNev requested a review from a team as a code owner February 3, 2026 21:28
Copy link
Member

@rads-1996 rads-1996 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request removes the dependency on the external httpbin.org service in tests by replacing all references with localhost:9001 endpoints. The local test server (configured in gruntfile.js on port 9001) provides test resources like README.md as replacements for the httpbin.org endpoints.

Changes:

  • Replaced httpbin.org URLs with http://localhost:9001/README.md in test files
  • Updated test patterns to use localhost:9001 instead of generic localhost
  • Updated npm dependencies (minor version bumps for various packages)

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tools/chrome-debug-extension/src/dataSources/dataSources.ts Changed localhost detection to specifically match localhost:9001 for NoOpDataSource usage
shared/AppInsightsCore/Tests/Unit/src/Common/Util.tests.ts Updated correlation header tests to use localhost:9001 instead of localhost
extensions/applicationinsights-dependencies-js/Tests/Unit/src/ajax.tests.ts Changed exclusion pattern from localhost to localhost:9001
extensions/applicationinsights-dependencies-js/Tests/Unit/src/W3CTraceStateDependency.tests.ts Replaced httpbin.org URLs with localhost:9001 endpoints in W3C trace tests
common/Tests/Framework/src/TestHelper.ts Updated test helper domain from localhost to localhost:9001
AISKU/Tests/Unit/src/applicationinsights.e2e.tests.ts Replaced httpbin.org fetch URLs with localhost:9001 in E2E tests
AISKU/Tests/Unit/src/SnippetInitialization.Tests.ts Replaced httpbin.org URLs with localhost:9001 in snippet initialization tests
common/config/rush/npm-shrinkwrap.json Updated dependency versions (@isaacs/brace-expansion, @nevware21/ts-utils, caniuse-lite, electron-to-chromium) and deprecation messages
Files not reviewed (1)
  • common/config/rush/npm-shrinkwrap.json: Language not supported

@MSNev MSNev enabled auto-merge (squash) February 3, 2026 21:37
@MSNev MSNev merged commit 2d12dfd into microsoft:beta Feb 3, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants