Skip to content
New issue

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

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

Already on GitHub? # to your account

comment out test #504

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

comment out test #504

wants to merge 3 commits into from

Conversation

michaelkernaghan
Copy link
Contributor

trying to diagnose the source of the error thrown when testing AWS with

out, err = OctezClient("transfer", "1", "from", tz3alias, "to", "alice", "--burn-cap", "0.06425")
assert.NoError(t, err)
require.Contains(t, string(out), "Operation successfully injected in the node")

by commenting out that test.

Copy link

cloudflare-workers-and-pages bot commented Oct 11, 2024

Deploying signatory with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2027370
Status: ✅  Deploy successful!
Preview URL: https://5d992f95.signatory.pages.dev
Branch Preview URL: https://mk-test.signatory.pages.dev

View logs

@michaelkernaghan
Copy link
Contributor Author

also removed "version" from docker-compose.yml to remove the warning and tried comment out the restart signatory call in the first test that fails.

@michaelkernaghan michaelkernaghan changed the title comment out test for diagnosis of integration test fail for merging PRs comment out test Oct 12, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant