Project

General

Profile

Actions

action #136151

closed

qem-bot unit testing broken with openqa-client==4.2.2

Added by mpagot 7 months ago. Updated 7 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
2023-09-20
Due date:
% Done:

0%

Estimated time:
Tags:

Description

Unit testing, at least as executed in the github action at each PR, are broken.

The workflow is configured to install all the Python dependency from https://github.com/openSUSE/qem-bot/blob/master/requirements-dev.txt that is designed not to have explicit fixed version for each dependency.

The unit test error is like https://github.com/openSUSE/qem-bot/actions/runs/6250035145/job/16968000626#step:5:332

         except Exception as e:
>           (method, url, status_code) = e.args
E           ValueError: too many values to unpack (expected 3)

It is from https://github.com/openSUSE/qem-bot/blob/83c708b55b200f54aae49ab26afe37cbaa652b2a/openqabot/openqa.py#L87

Error is present with openqa-client==4.2.2 and not present with openqa-client==4.2.1

It could be related to https://github.com/os-autoinst/openQA-python-client/pull/35 for https://progress.opensuse.org/issues/120939

Proof for that is https://github.com/openSUSE/qem-bot/pull/136 and passing check in it (PR is only to proof the issue and not a solution for that)

Actions #1

Updated by tinita 7 months ago

Actions #2

Updated by tinita 7 months ago

  • Status changed from New to In Progress
  • Assignee set to tinita
Actions #3

Updated by okurz 7 months ago

Merged, thank you Tina! Rebasing open PRs

Actions #4

Updated by okurz 7 months ago

  • Category set to Regressions/Crashes
  • Target version set to Ready
Actions #5

Updated by openqa_review 7 months ago

  • Due date set to 2023-10-05

Setting due date based on mean cycle time of SUSE QE Tools

Actions #6

Updated by okurz 7 months ago

  • Due date deleted (2023-10-05)
  • Status changed from In Progress to Resolved
Actions #7

Updated by tinita 7 months ago

https://github.com/openSUSE/qem-bot/pull/136 proved it was fine

How did this PR prove it was fine?

# https://patch-diff.githubusercontent.com/raw/openSUSE/qem-bot/pull/136.patch
+++ b/requirements-dev.txt
@@ -4,7 +4,7 @@ coverage
 responses>=0.21.0
 requests
 osc
-openqa-client
+openqa-client==4.2.1

It pinned openqa-client to a version which did not have that new exception parameter.

Actions #8

Updated by okurz 7 months ago

Yeah, the change didn't matter only that CI was ok again

Actions

Also available in: Atom PDF