Skip to content

Add automated tests for Go module redirect HTML files#1870

Open
Copilot wants to merge 6 commits intomainfrom
copilot/fix-15859888-44639036-c530c8d6-b734-4ee4-8431-e0c764f5a4a0
Open

Add automated tests for Go module redirect HTML files#1870
Copilot wants to merge 6 commits intomainfrom
copilot/fix-15859888-44639036-c530c8d6-b734-4ee4-8431-e0c764f5a4a0

Conversation

Copy link
Contributor

Copilot AI commented Nov 3, 2025

Plan to create tests for redirect HTML files in the helm-www repository:

  • Create test suite for redirect HTML files (static/helm/index.html, static/helm/v2/index.html, static/helm/v3/index.html, static/helm/v4/index.html)
  • Add test framework dependencies (Jest)
  • Write tests to validate:
    • HTML structure and metadata tags (go-import, go-source)
    • Meta refresh redirects to GitHub
    • Correct repository URLs and branches for each version
  • Add test script to package.json
  • Create GitHub Actions workflow for running tests on pull requests (Node 20 from netlify.toml)
  • Integrate tests into the Makefile build target
  • Verify all tests pass
  • Address code review feedback
  • Resolve yarn.lock conflicts with main branch
Original prompt

This section details on the original issue you should resolve

<issue_title>Create tests for redirects</issue_title>
<issue_description>Create tests for redirects

static/helm/index.html
static/helm/v2/index.html
static/helm/v4/index.html
static/helm/v3/index.html
  • Create a github/workflow for it that runs on pull requests. It should run on node version specified in netlify.toml
  • Run tests when the make build target is run

</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: TerryHowe <104113+TerryHowe@users.noreply.github.com>
@pull-request-size pull-request-size bot added size/L and removed size/XS labels Nov 3, 2025
Co-authored-by: TerryHowe <104113+TerryHowe@users.noreply.github.com>
@TerryHowe TerryHowe marked this pull request as ready for review November 3, 2025 15:25
Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

@TerryHowe TerryHowe changed the title [WIP] Create tests for redirects Create tests for redirects Nov 3, 2025
Co-authored-by: TerryHowe <104113+TerryHowe@users.noreply.github.com>
Copilot AI changed the title Create tests for redirects Add automated tests for Go module redirect HTML files Nov 3, 2025
Copilot AI requested a review from TerryHowe November 3, 2025 15:29
Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

@TerryHowe
Copy link
Contributor

@copilot resolve the yarn.lock conflicts

Copilot AI and others added 2 commits November 11, 2025 10:48
…44639036-c530c8d6-b734-4ee4-8431-e0c764f5a4a0
Co-authored-by: TerryHowe <104113+TerryHowe@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 11, 2025

@copilot resolve the yarn.lock conflicts

Resolved yarn.lock conflicts by merging main branch and regenerating the lockfile. All tests pass. Commit: 8c4467e

Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

@TerryHowe
Copy link
Contributor

https://gh.io/copilot-coding-agent-docs resolve the conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create tests for redirects

2 participants