Skip to content

Fix failing integration test on forks#240

Merged
josecelano merged 1 commit into
nautilus-cyberneering:mainfrom
josecelano:issue-175-fix-integration-tests-failing-on-forks
Jun 6, 2022
Merged

Fix failing integration test on forks#240
josecelano merged 1 commit into
nautilus-cyberneering:mainfrom
josecelano:issue-175-fix-integration-tests-failing-on-forks

Conversation

@josecelano

Copy link
Copy Markdown
Member

We have a pretty annoying bug that makes fail all test workflows on pull request events from a forked repository.

As we still need Git Queue action to sign commits I have just fixed it by importing the GPG key we are using for testing instead of importing it from secrets. Anyway, that's the right way to do it.

We agree on removing the signing option from the Git Queue but in this case, the Git Queue is not signing commits. I'm only testing that Git Queue does not interfere with Git global signing configuration.

@josecelano josecelano linked an issue Jun 3, 2022 that may be closed by this pull request
@josecelano josecelano force-pushed the issue-175-fix-integration-tests-failing-on-forks branch from 6229c86 to 3b2bb25 Compare June 3, 2022 14:46
@josecelano

Copy link
Copy Markdown
Member Author

It's not working yet becuase I have to change this line:

gpg_private_key: ${{ secrets.GPG_PRIVATE_KEY }}

To use the ouput from the "Import test GPG key" or read from the temp file test-gpg.key.

In fact, we do not need to import the key only read it from the fixtures folder.

@josecelano josecelano force-pushed the issue-175-fix-integration-tests-failing-on-forks branch 3 times, most recently from b05ac2c to 42b7d29 Compare June 6, 2022 11:03
@josecelano josecelano marked this pull request as ready for review June 6, 2022 11:04
@josecelano josecelano force-pushed the issue-175-fix-integration-tests-failing-on-forks branch from 42b7d29 to 90b5009 Compare June 6, 2022 11:10
@josecelano josecelano merged commit 571a0b6 into nautilus-cyberneering:main Jun 6, 2022
@josecelano josecelano deleted the issue-175-fix-integration-tests-failing-on-forks branch June 6, 2022 11:13
@josecelano

Copy link
Copy Markdown
Member Author

I had to add a line break after the GPG private key header: 94543d4

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.

Integration tests with GPG signature do not work on forks

1 participant