Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • A AQS Pageviews 2.0
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • Eevans
  • AQS Pageviews 2.0
  • Merge requests
  • !20

Draft: Fixes to httptest Server-based approach

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Eevans requested to merge alt-test into main Jan 26, 2022
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 1

There were a couple of things that prevented this from working:

  1. httprouter still needs to be invoked to do its magic (e.g. to populate Params)
  2. Client connections need to use the URL retured by httptest.Newserver
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: alt-test