Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run tests in Editor type modules #22

Open
PaulHax opened this issue Jul 24, 2020 · 2 comments · May be fixed by #23
Open

Run tests in Editor type modules #22

PaulHax opened this issue Jul 24, 2020 · 2 comments · May be fixed by #23
Assignees

Comments

@PaulHax
Copy link

PaulHax commented Jul 24, 2020

I'ed like to run ue4 test MyEditorTypeModule but no dice. Seems the Unreal -game flag the test command uses means my "Type": "Editor" module does not get loaded.

Perhaps a --with-editor flag that leaves out the -game? Or am I missing something else...

Thanks for this tool.

@adamrehn
Copy link
Owner

Apologies for the delay in responding to this issue, I've been inordinately busy over the past couple of months and I'm only now making my way through my GitHub backlog.

The reason the -game flag is passed is because the Unreal Editor frequently misbehaves when running automation tests from the command-line without that flag, particularly under Windows. A quick test with Unreal Engine 4.25 under Windows confirms that some automation commands result in excessively long wait times before the Editor terminates, and others result in the Editor failing to terminate at all, instead remaining open in the background and leaving the ue4 test command hanging as it waits for the child process to exit.

The need to test code in Editor-only modules is a legitimate use case that's not well-served by the current implementation of the ue4 test command, so I'll investigate methods of invoking the Editor via Unreal AutomationTool (UAT) or Gauntlet to determine whether they might provide a better backend to replace the current direct invocation mechanism.

@adamrehn adamrehn self-assigned this Aug 24, 2020
@dskvr
Copy link

dskvr commented Aug 25, 2020

Thanks for the background information @adamrehn

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 a pull request may close this issue.

3 participants