Skip to content
Snippets Groups Projects

Add test for https://gitlab.com/linkahead/linkahead-pylib/-/issues/127

Merged Add test for https://gitlab.com/linkahead/linkahead-pylib/-/issues/127
f-enh-fit-93-pylinkahead-separate-timeouts into dev
1 unresolved thread
Merged I. Nüske requested to merge f-enh-fit-93-pylinkahead-separate-timeouts into dev
1 unresolved thread

Summary

Test for https://gitlab.com/linkahead/linkahead-pylib/-/issues/127
Code MR is here.

Check List for the Author

  • All automated tests pass
  • Reference related issues
  • Up-to-date CHANGELOG.md (or not necessary)
  • Up-to-date JSON schema (or not necessary)
  • Appropriate user and developer documentation (or not necessary)
  • Annotations in code (Gitlab comments)

Check List for the Reviewer

  • I understand the intent of this MR
  • All automated tests pass
  • Up-to-date CHANGELOG.md (or not necessary)
  • Appropriate user and developer documentation (or not necessary)
  • The test environment setup works and the intended behavior is reproducible in the test environment
  • In-code documentation and comments are up-to-date.
  • Check: Are there specifications? Are they satisfied?
Edited by Alexander Schlemmer

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
tests/test_issues_pylib.py
205 connect/read timeouts and timeout None.
206
207 See https://gitlab.com/linkahead/linkahead-pylib/-/issues/127 and
208 https://gitlab.indiscale.com/caosdb/customers/f-fit/management/-/issues/93
209 """
210 # Setup paths and save previous timeout
211 base_dir = Path(__file__).parent.parent
212 temp_pylinkahead_path = base_dir/'.pyla-temp-test.ini'
213 temp_pylinkahead_path.unlink(True)
214 prev_timeout = None
215 if db.get_config().has_option('Connection', 'timeout'):
216 prev_timeout = db.get_config().get('Connection', 'timeout')
217
218 # Parse various timeout strings and check successful connect
219 valid_timeout_strings = ["timeout = None", "timeout=null",
220 "timeout = (4, 40)", "timeout=(4,4)"]
  • Alexander Schlemmer marked the checklist item I understand the intent of this MR as completed

    marked the checklist item I understand the intent of this MR as completed

  • Alexander Schlemmer marked the checklist item All automated tests pass as completed

    marked the checklist item All automated tests pass as completed

  • Alexander Schlemmer marked the checklist item Up-to-date CHANGELOG.md (or not necessary) as completed

    marked the checklist item Up-to-date CHANGELOG.md (or not necessary) as completed

  • Alexander Schlemmer marked the checklist item Appropriate user and developer documentation (or not necessary) as completed

    marked the checklist item Appropriate user and developer documentation (or not necessary) as completed

  • Alexander Schlemmer marked the checklist item The test environment setup works and the intended behavior is reproducible in the test as completed

    marked the checklist item The test environment setup works and the intended behavior is reproducible in the test as completed

  • Alexander Schlemmer marked the checklist item In-code documentation and comments are up-to-date. as completed

    marked the checklist item In-code documentation and comments are up-to-date. as completed

  • Alexander Schlemmer approved this merge request

    approved this merge request

  • I. Nüske added 5 commits

    added 5 commits

    • 60fb5063...7c8a480f - 3 commits from branch dev
    • a3955362 - Merge branch 'refs/heads/dev' into f-enh-fit-93-pylinkahead-separate-timeouts
    • 11709281 - TST: Additional valid test strings for test_gitlab_com_127

    Compare with previous version

  • I. Nüske reset approvals from @salexan by pushing to the branch

    reset approvals from @salexan by pushing to the branch

  • I. Nüske mentioned in commit d362a52a

    mentioned in commit d362a52a

  • I. Nüske merged

    merged

    • Please register or sign in to reply