Skip to content

Add debug e2e harness and export debug metadata for Myslot import/export flow#96

Draft
Copilot wants to merge 4 commits intomasterfrom
copilot/implement-e2e-test-addon
Draft

Add debug e2e harness and export debug metadata for Myslot import/export flow#96
Copilot wants to merge 4 commits intomasterfrom
copilot/implement-e2e-test-addon

Conversation

Copy link
Contributor

Copilot AI commented Dec 26, 2025

Implemented a debug-only e2e path to launch WoW UI, click Export/Import, and validate addon flow via the existing /myslot command, and added debug metadata stamping to exported text.

  • Test hooks: Expose import/export buttons and edit box through MySlot.TestHooks, including dialog identifier, e2e runner, and export debug hook.
  • Debug e2e script: wowtest/myslot-e2e.lua runs export → import automatically, accepts the confirmation popup, and contributes debug metadata; triggered with /myslot e2e.
  • Export debug metadata: Export output now prepends optional debug lines supplied by MySlot.TestHooks.exportDebugInfo, with safe handling for hook errors and non-string entries.
  • Packaging: Include the e2e script in the debug section of Myslot.toc; allow luacheck to recognize the entry points.

Example usage:

-- In WoW client debug build
/myslot e2e  -- opens Myslot, exports current layout, imports it back, confirms dialog, stamps debug metadata
Original prompt

impl e2e test, start wow.exe and test the addon, click import and export button


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
Copilot AI changed the title [WIP] Implement end-to-end test for addon import and export Add debug e2e harness to exercise Myslot import/export flow Dec 26, 2025
Copilot AI requested a review from tg123 December 26, 2025 19:42
end

SlashCmdList["MYSLOTE2E"] = run_e2e
SLASH_MYSLOTE2E1 = "/myslot-e2e"
Copy link
Owner

Choose a reason for hiding this comment

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

@copilot could you please merge to /myslot

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done in 3ef48c4: the e2e runner now lives under the existing /myslot command (/myslot e2e), no separate slash command needed.

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 27, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested a review from tg123 December 27, 2025 18:04
@tg123
Copy link
Owner

tg123 commented Dec 28, 2025

@copilot

  1. resolve conflict
  2. add debug info in export text

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 28, 2025

@copilot

  1. resolve conflict
  2. add debug info in export text

Resolved and added debug metadata into exported text; use /myslot e2e to stamp runs. Changes in fc94f5a.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Add debug e2e harness to exercise Myslot import/export flow Add debug e2e harness and export debug metadata for Myslot import/export flow Dec 28, 2025
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