[electrum] add a basic test for setup.py
ClosedPublic
Actions

Authored by PiRK on Jul 12 2023, 16:00.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCcfbb96bfc913: [electrum] add a basic test for setup.py

Summary

This would have caught the bug fixed by D14258.

electrum-tests is run by CI on any change in the electrum/ directory, even if it is not touching python code.

Test Plan

python test_runner.py

Try reintroducing an error in setup.py [README.md -> README.rst) and check that it fails and print the error tracebak.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Jul 12 2023, 16:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 12 2023, 16:00

PiRK requested review of this revision.Jul 12 2023, 16:00

Harbormaster completed remote builds in B24471: Diff 41417.Jul 12 2023, 16:02

@bot check-electrum

teamcity awarded a token.Jul 12 2023, 16:03

PiRK added a child revision: D14261: [electrum] don't run the unit tests twice.Jul 12 2023, 16:03

The build failed due to an unexpected infrastructure outage. The administrators have been notified to investigate. Sorry for the inconvenience.

@bot electrum-tests

teamcity updated a token.Jul 12 2023, 18:14

Fabien accepted this revision.Jul 12 2023, 18:16

Fabien edited the summary of this revision. (Show Details)

This revision is now accepted and ready to land.Jul 12 2023, 18:16

Closed by commit rABCcfbb96bfc913: [electrum] add a basic test for setup.py (authored by PiRK). · Explain WhyJul 12 2023, 20:47

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCcfbb96bfc913: [electrum] add a basic test for setup.py.

Revision Contents
Changeset List

Path

Size

electrum/

test_runner.py

20 lines

Diff 41425

View Options

[electrum] add a basic test for setup.pyClosedPublicActions