Project

General

Profile

Actions

action #177564

closed

coordination #58184: [saga][epic][use case] full version control awareness within openQA

coordination #88561: [epic] Extend needle version control handling

Ensure test coverage of needle editor size:S

Added by livdywan 2 months ago. Updated about 2 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Feature requests
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Motivation

#162035 was originally covering changes to the needle editor, and questions about existing coverage of the needle editor came up.

Acceptance criteria

  • AC1: Test coverage of needle editor was discussed and increased

Suggestions

  • Review the current state of tests for the needle editor, also with #162035 in mind
  • Consider a NEEDLES_DIR being a different repo/remote entirely (e.g. a pull request) (although this is not strictly required it would be good to ensure we could also support it)
  • Read #56789 again

Related issues 1 (0 open1 closed)

Copied from openQA Project (public) - action #162035: Support development of "Use needles from correct ref of NEEDLES_DIR" https://github.com/os-autoinst/openQA/pull/5175 size:MResolveddheidler2024-06-10

Actions
Actions #1

Updated by livdywan 2 months ago

  • Copied from action #162035: Support development of "Use needles from correct ref of NEEDLES_DIR" https://github.com/os-autoinst/openQA/pull/5175 size:M added
Actions #2

Updated by mkittler 2 months ago

  • Status changed from New to In Progress
Actions #3

Updated by openqa_review 2 months ago

  • Due date set to 2025-03-06

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

Actions #5

Updated by mkittler 2 months ago

  • Status changed from In Progress to Feedback
Actions #6

Updated by mkittler 2 months ago

  • Priority changed from Urgent to High

I know this is an urgent ticket but I actually don't think this is urgent and I'm waiting for feedback on https://github.com/os-autoinst/openQA/pull/6192#discussion_r1962522449 specifically.

Actions #7

Updated by livdywan 2 months ago

  • Subject changed from Ensure test coverage of needle editor to Ensure test coverage of needle editor size:S

Discussed briefly

Actions #8

Updated by livdywan 2 months ago

mkittler wrote in #note-6:

I know this is an urgent ticket but I actually don't think this is urgent and I'm waiting for feedback on https://github.com/os-autoinst/openQA/pull/6192#discussion_r1962522449 specifically.

Merged. Anything else to do here?

Actions #9

Updated by mkittler 2 months ago

  • Status changed from Feedback to Resolved
Actions #10

Updated by okurz about 2 months ago

  • Due date deleted (2025-03-06)
Actions

Also available in: Atom PDF