Skip to content

Comments

Independent test runs#289

Merged
MarcelGeo merged 17 commits intomasterfrom
independent_test_runs
Feb 23, 2026
Merged

Independent test runs#289
MarcelGeo merged 17 commits intomasterfrom
independent_test_runs

Conversation

@DanChov
Copy link
Contributor

@DanChov DanChov commented Feb 2, 2026

Made test independent createting new user and workspace everytime they run and deleting them afterwards
#272

@DanChov DanChov closed this Feb 2, 2026
@DanChov DanChov reopened this Feb 2, 2026
@DanChov DanChov assigned DanChov and unassigned MarcelGeo and harminius Feb 3, 2026
@coveralls
Copy link

coveralls commented Feb 16, 2026

Pull Request Test Coverage Report for Build 22230505888

Details

  • 230 of 237 (97.05%) changed or added relevant lines in 1 file are covered.
  • 7 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.01%) to 82.362%

Changes Missing Coverage Covered Lines Changed/Added Lines %
mergin/test/test_client.py 230 237 97.05%
Files with Coverage Reduction New Missed Lines %
mergin/test/test_client.py 3 97.87%
mergin/client.py 4 87.94%
Totals Coverage Status
Change from base Build 21135416399: -0.01%
Covered Lines: 4254
Relevant Lines: 5165

💛 - Coveralls

USER_PWD = os.environ.get("TEST_API_PASSWORD")
API_USER2 = os.environ.get("TEST_API_USERNAME2")
USER_PWD2 = os.environ.get("TEST_API_PASSWORD2")
PASSWORD_DEFAULT = PASSWORD_DEFAULT = secrets.token_urlsafe(10)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rm one PASSWORD_DEFAULT

@MarcelGeo
Copy link
Contributor

MarcelGeo commented Feb 20, 2026

Fix tests and merge, not sure why failing again....

FAILED mergin/test/test_client.py::test_editor_push - mergin.common.ClientError: Detail: {"password":["Password is not strong enough"]}

@MarcelGeo MarcelGeo merged commit 713a301 into master Feb 23, 2026
4 checks passed
@MarcelGeo MarcelGeo deleted the independent_test_runs branch February 23, 2026 10:57
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.

5 participants