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

Fix logo image SHA-256 and size in test #3

Merged
merged 1 commit into from
Jul 27, 2022
Merged

Fix logo image SHA-256 and size in test #3

merged 1 commit into from
Jul 27, 2022

Conversation

aszlig
Copy link
Collaborator

@aszlig aszlig commented Dec 19, 2020

Test failure introduced in 556f7be.

The commit in question has changed the logo image and since the SHA-256 and size of the image is verified in the integration test, the test subsequently fails.

A better way to solve this would be if we'd have a way to properly parameterise the main functionality instead of expecting users to change source files. This could probably involve turning the project into a proper Godot plugin, which I outlined in #2.

For now however, I don't have a lot of time to allocate to dig into Godot plugins, so I'm simply fixing the SHA-256 and size.

Test failure introduced in 556f7be.

The commit in question has changed the logo image and since the SHA-256
and size of the image is verified in the integration test, the test
subsequently fails.

A better way to solve this would be if we'd have a way to properly
parameterise the main functionality instead of expecting users to change
source files. This could probably involve turning the project into a
proper Godot plugin, which I outlined in #2.

For now however, I don't have a lot of time to allocate to dig into
Godot plugins, so I'm simply fixing the SHA-256 and size.

Signed-off-by: aszlig <[email protected]>
@RPicster RPicster merged commit 1763a19 into main Jul 27, 2022
@RPicster RPicster deleted the fix-tests branch July 27, 2022 09:48
@aszlig
Copy link
Collaborator Author

aszlig commented Jul 27, 2022

@RPicster: Hooray :-)

2022-07-27_12-10

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