Skip to content

Commit

Permalink
refactor: change type of parameters
Browse files Browse the repository at this point in the history
Change type of parameters for the constructor of reporters.
  • Loading branch information
gibiw committed Jun 17, 2024
1 parent 2556452 commit 7a1d730
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 9 deletions.
2 changes: 1 addition & 1 deletion qase-python-commons/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"

[project]
name = "qase-python-commons"
version = "3.1.0b1"
version = "3.1.0b2"
description = "A library for Qase TestOps and Qase Report"
readme = "README.md"
authors = [{name = "Qase Team", email = "[email protected]"}]
Expand Down
4 changes: 2 additions & 2 deletions qase-python-commons/src/qase/commons/reporters/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,13 +38,13 @@ def __init__(self, config: ConfigManager):
if mode == Mode.testops:
try:
self._load_testops_plan()
self.reporter = QaseTestOps(config=config, logger=self.logger)
self.reporter = QaseTestOps(config=self.config, logger=self.logger)
except Exception as e:
self.logger.log('Failed to initialize TestOps reporter. Using fallback.', 'info')
self.logger.log(e, 'error')
self.reporter = self.fallback
elif mode == Mode.report:
self.reporter = QaseReport(config=config, logger=self.logger)
self.reporter = QaseReport(config=self.config, logger=self.logger)
else:
self.reporter = None

Expand Down
7 changes: 4 additions & 3 deletions qase-python-commons/src/qase/commons/reporters/report.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,20 +4,21 @@
import json
import re
from ..models import Result, Run, Attachment
from .. import QaseUtils, ConfigManager, Logger
from .. import QaseUtils, Logger
from ..models.config.connection import Format
from ..models.config.qaseconfig import QaseConfig


class QaseReport:
def __init__(
self,
config: ConfigManager,
config: QaseConfig,
logger: Logger
):
self.duration = 0
self.results = []
self.attachments = []
self.config = config.config
self.config = config
self.logger = logger

self.report_path = self.config.report.connection.path
Expand Down
7 changes: 4 additions & 3 deletions qase-python-commons/src/qase/commons/reporters/testops.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,20 @@

from datetime import datetime
from typing import List
from .. import ConfigManager, Logger, ReporterException
from .. import Logger, ReporterException
from ..client.api_v1_client import ApiV1Client
from ..client.base_api_client import BaseApiClient
from ..models import Result
from ..models.config.qaseconfig import QaseConfig

DEFAULT_BATCH_SIZE = 200
DEFAULT_THREAD_COUNT = 4


class QaseTestOps:

def __init__(self, config: ConfigManager, logger: Logger) -> None:
self.config = config.config
def __init__(self, config: QaseConfig, logger: Logger) -> None:
self.config = config
self.logger = logger

self.client = self._prepare_client()
Expand Down

0 comments on commit 7a1d730

Please sign in to comment.